Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does hls.js support extracting SEI frame from videos? #4421

Closed
dongrixinyu opened this issue Nov 11, 2021 · 5 comments · Fixed by #4459
Closed

Does hls.js support extracting SEI frame from videos? #4421

dongrixinyu opened this issue Nov 11, 2021 · 5 comments · Fixed by #4459
Labels
Feature proposal Needs Triage If there is a suspected stream issue, apply this label to triage if it is something we should fix.
Milestone

Comments

@dongrixinyu
Copy link

dongrixinyu commented Nov 11, 2021

ponse_

@dongrixinyu dongrixinyu added Feature proposal Needs Triage If there is a suspected stream issue, apply this label to triage if it is something we should fix. labels Nov 11, 2021
@dylanjha
Copy link
Contributor

@dongrixinyu would this event help?

  • Hls.Events.FRAG_PARSING_USERDATA - fired when parsing sei text is completed
    data: { id : demuxer id, frag: fragment object, samples : [ sei samples pes ] }

Found it in the list of runtime events

@dongrixinyu
Copy link
Author

Thank you for your answer. and I ll reply after i tried this hook

@dylanjha
Copy link
Contributor

@dongrixinyu I'm not sure the answer to your 2 questions.

Did you try this code above with the Hls.Events.FRAG_PARSING_USERDATA listener? What were the results?

robwalch added a commit that referenced this issue Dec 15, 2021
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
robwalch added a commit that referenced this issue Dec 15, 2021
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
robwalch added a commit that referenced this issue Dec 16, 2021
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
robwalch added a commit that referenced this issue Dec 16, 2021
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
robwalch added a commit that referenced this issue Jan 18, 2022
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
robwalch added a commit that referenced this issue Jan 25, 2022
Resolves #2623 (Add CMAF CC support)
Resolves #4242 (Fix parsing multiple user data unregistered SEI)
Closes #4421 (Does hls.js support extracting SEI frame from videos?)
Fixes #4317 (Progressive mode is broken with fmp4 playback)
@antonm76
Copy link

antonm76 commented Apr 1, 2022

@robwalch I set SEI user data in my stream and then I able to read them with FRAG_PARSING_USERDATA events but it is only possible for HLS streams with MPEGTS segments.
When I try to use FMP4 (with ffmpeg's hls muxer) or M4S (which are generated by the ffmpeg's dash muxer) user data samples are always emty.

@antonm76
Copy link

antonm76 commented Apr 1, 2022

@robwalch I use v.1.1.5

@robwalch robwalch added this to the 1.2.0 milestone Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature proposal Needs Triage If there is a suspected stream issue, apply this label to triage if it is something we should fix.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants