-
Notifications
You must be signed in to change notification settings - Fork 510
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
Add Support to SCTE - 35 #2158
Comments
I have on my plate to extract the SCTE-35 markers from MPEG-TS and forward them to the DASH/HLS muxer. Which use-cases do you want to cover? |
Exactly that one! also I start to code that! |
But i have a lot of problems/questions regarding the arch of the project |
The first step is to demux the messages. You'll need to modify a bit the M2TS demuxer code :
Once you have a new SCTE35 message extracted, you will need to forward it to the user (here the m2ts demux filter) through a new event type passing the parsed info, as done for TEMI. We can then map this info to a packet property Not sure we need to parse the entire set of SCTE35 descriptors, maybe a subset will do. It would be nice to have one or two public reference TS with SCTE35 descriptors that we could include in the testsuite, would you have any ? |
I found this in a FFMPEG question but I don't know if is legal https://www.mediafire.com/folder/zv20csqeo1ixt/ |
Hello @jeanlf @rbouqueau
once again thank you for your help |
I think that's a good approach yes. You may want to extend the GF_TEMIInfo structure to differentiate between TEMI and SCTE35 messages and change packet prop name in m2tsdmx_merge_temi.
Nothing specific to do, the data will be passed along as packet properties, so all you need to do is check for the desired properties when processing a packet in the dasher
What I would do (at least initially) is format the EXT-X-DATERANGE tag in the dasher, and add that info to the segment state, then serialize it after or before the segment crypt info |
@JuanMorenoS Is there any progress or questions here? |
1 similar comment
@JuanMorenoS Is there any progress or questions here? |
For those reading: WiP in #2748 |
#2748 merged. Closing. |
Hi,
Do you have in your roadmap add support to SCTE-35?
If not.... I can do it, but maybe I need help to know the project structure.
Thanks and awesome project!
The text was updated successfully, but these errors were encountered: