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

"Multi-variant playlist", not "variant stream" #360

Open
wabiloo opened this issue Apr 17, 2024 · 0 comments
Open

"Multi-variant playlist", not "variant stream" #360

wabiloo opened this issue Apr 17, 2024 · 0 comments

Comments

@wabiloo
Copy link

wabiloo commented Apr 17, 2024

Hi,

Just a point of detail (and not critical) but I believe that the section https://github.com/globocom/m3u8?tab=readme-ov-file#variant-playlists-variable-bitrates is not technically correct. The "main" playlist is called multi-variant, not variant stream (see https://datatracker.ietf.org/doc/html/draft-pantos-hls-rfc8216bis-14#section-4.4.6.2).

It seems to me therefore that the property should ideally be called .is_multi_variant

This would however be a breaking change, and may annoy a lot of people. Maybe just adding something to the documentation to avoid confusion with the spec?

@wabiloo wabiloo changed the title Multi-variant, not variant "Multi-variant playlist", not "variant stream" Apr 17, 2024
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