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

VideoPlayer: hls, select best resolution closest to desired bandwidth #10801

Merged
merged 1 commit into from Nov 1, 2016

Conversation

FernetMenta
Copy link
Contributor

see title

@popcornmix that fixes the issue for the url you gave me

@FernetMenta FernetMenta added Type: Fix non-breaking change which fixes an issue v17 Krypton Component: Video labels Oct 29, 2016
@FernetMenta FernetMenta added this to the Krypton 17.0-beta6 milestone Oct 29, 2016
@popcornmix
Copy link
Member

Yes, fixes the URL. However now playing that URL takes between 6 and 12 seconds to start stream.
With #10766 reverted the stream starts in 1 or 2 seconds.

Any reason why it's much slower?

@FernetMenta
Copy link
Contributor Author

It is much faster here on my Mac. Stream also starts within 1-2 secs. The old function did parse the m3m8 playlist file to select a particular url. But it did not properly and as a result on some playlists audio was missing. Now we let ffmpeg do this.

Maybe the self-selected stream was identified as mpegts and demuxer used the "fast-channel-switch" path by reducing time for analysis in find_stream_info.

jenkins build this please

@MartijnKaijser
Copy link
Member

good for merge?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Video Type: Fix non-breaking change which fixes an issue v17 Krypton
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants