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

Regression: choosing lowest-bandwidth codec leads to always picking audio-only variant for HLS #861

Closed
ismena opened this issue Jun 8, 2017 · 2 comments
Assignees
Labels
component: HLS The issue involves Apple's HLS manifest format status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Milestone

Comments

@ismena
Copy link
Contributor

ismena commented Jun 8, 2017

This was introduced by 434f1bf (CL that fixed #841). The fact that we started grouping variants by codecs and choosing the group with the smallest average bandwidth led to the player always choosing audio-only variants for HLS content that has at least one.
We can't switch from audio-only variants to audio+video, so we end up being stuck with audio-only.
A manifest to repro: https://nexxplayplus6.streaming.mediaservices.windows.net/32de1b7d-822c-43b2-b3a5-1cd97f0e9ae7/242391_src.ism/manifest(format=m3u8-aapl)

(Discovered by investigating #860).

@ismena ismena added the type: bug Something isn't working correctly label Jun 8, 2017
@joeyparrish joeyparrish added this to the v2.2.0 milestone Jun 8, 2017
@joeyparrish
Copy link
Member

Given #824, where we can't switch to audio-only variants anyway, we should probably solve both problems at one by filtering out audio-only variants in cases where there are audio+video variants available.

@theodab theodab self-assigned this Jun 14, 2017
@theodab theodab added the component: HLS The issue involves Apple's HLS manifest format label Jun 21, 2017
joeyparrish pushed a commit that referenced this issue Jul 17, 2017
In situations where there are variants with video and audio, and variants
with only one of the two, this filters out the less-complete variants.
This removes the possibility of the player switching to an audio-only
variant and breaking MediaSource. It also solves the regression where
audio-only variants have the lowest bandwidth and thus are chosen
preferentially on weak connections.

Closes #824
Closes #861

Change-Id: Ic37543d98962ddd83a85224e26e5225d142b844e
@joeyparrish
Copy link
Member

The fix for this has been cherry-picked and will be released in v2.1.5.

@shaka-project shaka-project locked and limited conversation to collaborators Mar 22, 2018
@shaka-bot shaka-bot added the status: archived Archived and locked; will not be updated label Apr 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: HLS The issue involves Apple's HLS manifest format status: archived Archived and locked; will not be updated type: bug Something isn't working correctly
Projects
None yet
Development

No branches or pull requests

4 participants