{"action":"created","issue":{"url":"https://api.github.com/repos/mangui/flashls/issues/471","labels_url":"https://api.github.com/repos/mangui/flashls/issues/471/labels{/name}","comments_url":"https://api.github.com/repos/mangui/flashls/issues/471/comments","events_url":"https://api.github.com/repos/mangui/flashls/issues/471/events","html_url":"https://github.com/mangui/flashls/issues/471","id":124245661,"number":471,"title":"608/708 CC SEI parsing issue","user":{"login":"zuzzurro","id":6942457,"avatar_url":"https://avatars.githubusercontent.com/u/6942457?v=3","gravatar_id":"","url":"https://api.github.com/users/zuzzurro","html_url":"https://github.com/zuzzurro","followers_url":"https://api.github.com/users/zuzzurro/followers","following_url":"https://api.github.com/users/zuzzurro/following{/other_user}","gists_url":"https://api.github.com/users/zuzzurro/gists{/gist_id}","starred_url":"https://api.github.com/users/zuzzurro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zuzzurro/subscriptions","organizations_url":"https://api.github.com/users/zuzzurro/orgs","repos_url":"https://api.github.com/users/zuzzurro/repos","events_url":"https://api.github.com/users/zuzzurro/events{/privacy}","received_events_url":"https://api.github.com/users/zuzzurro/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-12-29T17:48:43Z","updated_at":"2016-01-08T20:24:33Z","closed_at":null,"body":"Hi  @mangui and @jlacivita \r\n\r\nWe tested the 608/708 CC implementation using streams from two different encoders and we are able to display them fine (using OSMFCClib) in one case but not the other one.\r\nWe spent some time analyzing, researching and debugging and we think we know the reason.\r\nFirst of all here: https://en.wikipedia.org/wiki/CEA-708 you will find this note:\r\nNOTE: the SEI depending on the encoder can contain more payloads than just the captions, so one would need to navigate all payloadTypes contained within.\r\nSecondly you can see that exoplayer probably had a similar issue:\r\n\r\nhttps://github.com/google/ExoPlayer/issues/295\r\n\r\nand implemented a fix for it:\r\n\r\nhttps://github.com/google/ExoPlayer/commit/b5100886896ad35a32e6105a9d005c3209a81a9b\r\n\r\n(reading the SEI happen in a while loop after the fix).\r\n\r\nhttps://github.com/google/ExoPlayer/blob/master/library/src/main/java/com/google/android/exoplayer/extractor/ts/SeiReader.java#L47\r\n\r\nWe have implemented a similar fix and it seems to improve the situation a lot (we start seeing the captions) but not completely. Sometimes we seem to still miss some text (the byte array seems to be corrupted), but it is a very promising start.\r\n\r\nHere you can find a test stream that has 608 CC that shows the issue.\r\n\r\nhttp://demo.deltatre.it/cc-test/QualityLevels(1400000)/Manifest(video,format=m3u8-aapl).m3u8\r\n"},"comment":{"url":"https://api.github.com/repos/mangui/flashls/issues/comments/170113874","html_url":"https://github.com/mangui/flashls/issues/471#issuecomment-170113874","issue_url":"https://api.github.com/repos/mangui/flashls/issues/471","id":170113874,"user":{"login":"mangui","id":1798553,"avatar_url":"https://avatars.githubusercontent.com/u/1798553?v=3","gravatar_id":"","url":"https://api.github.com/users/mangui","html_url":"https://github.com/mangui","followers_url":"https://api.github.com/users/mangui/followers","following_url":"https://api.github.com/users/mangui/following{/other_user}","gists_url":"https://api.github.com/users/mangui/gists{/gist_id}","starred_url":"https://api.github.com/users/mangui/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mangui/subscriptions","organizations_url":"https://api.github.com/users/mangui/orgs","repos_url":"https://api.github.com/users/mangui/repos","events_url":"https://api.github.com/users/mangui/events{/privacy}","received_events_url":"https://api.github.com/users/mangui/received_events","type":"User","site_admin":false},"created_at":"2016-01-08T20:24:33Z","updated_at":"2016-01-08T20:24:33Z","body":"Great. Just bake a PR "}}