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

Added liveUpdateTimeThreshold to control time syncing during live streams #4382

Merged

Conversation

matvp91
Copy link
Contributor

@matvp91 matvp91 commented Feb 2, 2024

On low end devices, we noticed an increasing burden on the CPU during live streams. This setting adds a threshold that must be passed in order for PlaybackController to push another time sync instruction during dynamic manifests.

This is very comparable to wallclockTimeUpdateInterval but it does not interfere with other logic based on wall clock such as gap jumping (which is a way less heavy instruction for low end CPU's to handle).

@matvp91 matvp91 changed the title Added liveUpdateTimeThreshold for fine grained control on time syncing during live streams Added liveUpdateTimeThreshold to control time syncing during live streams Feb 2, 2024
@dsilhavy dsilhavy added this to the 4.7.4 milestone Feb 2, 2024
src/core/Settings.js Outdated Show resolved Hide resolved
src/core/Settings.js Outdated Show resolved Hide resolved
@dsilhavy dsilhavy merged commit 9e3da3c into Dash-Industry-Forum:development Feb 6, 2024
1 of 3 checks passed
eirikbjornr pushed a commit to bbc/dash.js that referenced this pull request Feb 14, 2024
…eams (Dash-Industry-Forum#4382)

* Added liveUpdateTimeThresholdInMilliseconds in order to control the frequency of time updates during live streams
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

Successfully merging this pull request may close these issues.

2 participants