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

Support 4k content on bowsers with limited MSE buffer #192

Closed
gesinger opened this issue Aug 9, 2018 · 8 comments
Closed

Support 4k content on bowsers with limited MSE buffer #192

gesinger opened this issue Aug 9, 2018 · 8 comments
Labels

Comments

@gesinger
Copy link
Contributor

gesinger commented Aug 9, 2018

Tracking PRs (which should eventually get merged into http-streaming):
"handle dynamic buffer sizes (4k support)"
videojs/videojs-contrib-hls#1416
"Graceful handling of QuotaExceededError"
videojs/videojs-contrib-media-sources#178

Related:
https://github.com/videojs/videojs-contrib-hls/issues/1435

@welcome
Copy link

welcome bot commented Aug 9, 2018

👋 Thanks for opening your first issue here! 👋

If you're reporting a 🐞 bug, please make sure you include steps to reproduce it. We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.
To help make it easier for us to investigate your issue, please follow the contributing guidelines.

@leonhandreke
Copy link

leonhandreke commented Mar 20, 2019

For anybody facing the Failed to execute 'appendBuffer' on 'SourceBuffer': The SourceBuffer is full error with high-bitrate videos: I cranked down videojs.Hls.MAX_GOAL_BUFFER_LENGTH and videojs.Hls.GOAL_BUFFER_LENGTH to 30. I couldn't really make sense of the code in

goalBufferLength() {
const currentTime = this.tech_.currentTime();
const initial = Config.GOAL_BUFFER_LENGTH;
const rate = Config.GOAL_BUFFER_LENGTH_RATE;
const max = Math.max(initial, Config.MAX_GOAL_BUFFER_LENGTH);
return Math.min(initial + currentTime * rate, max);
}
but that fixed it, now the buffer is shorter.

It's an ugly hack but it works ¯_(ツ)_/¯

@asadmalik3
Copy link

asadmalik3 commented Mar 28, 2019

@leonhandreke is there a neat way to change value of MAX_GOAL_BUFFER_LENGTH from video js?

@leonhandreke
Copy link

@asadmalik3 yes, you can just set videojs.Hls.MAX_GOAL_BUFFER_LENGTH = 30; in your code before initializing your player. Hope that helps!

@johneisenheim
Copy link

Hello! Thanks for this fixes...Is there any way to apply the fixes above on MPEG-Dash? On older browser (like Chrome 55) the error is still reproduced with mpd files. Thank you again!

@jeopard
Copy link

jeopard commented Oct 6, 2020

Hey! Thank you for all the great work! I just wanted to let you know that the videojs.Hls.MAX_GOAL_BUFFER_LENGTH = 30; doesn't work for Firefox, as I still get the append buffer error. I'm using video.js 7.8.4 with http-streaming 1.13.2.

I'm guessing this issue is a duplicate of #344 ?

@gkatsev
Copy link
Member

gkatsev commented Oct 11, 2020

Yup, they're very likely the same issue.

@gkatsev
Copy link
Member

gkatsev commented Feb 22, 2021

Closing in favor #344.

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

No branches or pull requests

7 participants