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

verify that we fully support the segment template spec #52

Open
friday opened this issue Feb 8, 2024 · 0 comments
Open

verify that we fully support the segment template spec #52

friday opened this issue Feb 8, 2024 · 0 comments

Comments

@friday
Copy link
Member

friday commented Feb 8, 2024

Currently we string replace $Number$ with the segment index, but there is also for example a zero-pad syntax $Number%0[width]d$ where you should zero-pad with the number specified as the width. I have fix for this coming up, but there seems to be more to it. For example if you look at dash.js issue tracker for PR 32 and 196, they have further details.

https://ottverse.com/structure-of-an-mpeg-dash-mpd/#Segment_Templates

#53 fixes the known one, but the dash.js pr 196 supports a lot more variants, and it would be good to also add tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant