-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
feat: Dynamic target latency #6858
base: main
Are you sure you want to change the base?
Conversation
This PR supercedes #6193. I marked it as a draft because while the functionality is there and the build passes locally, I hadn't yet gotten a chance to confirm that it's working as intended. It's possible there may be tweaks necessary. I'll get back to it on Thursday. |
Incremental code coverage: 42.03% |
@gkatsev please fix the tests! Thanks! |
I'll take a look. At first glance they seemed unrelated, but who knows. |
…, it ends up being a bit over 6000
c9d3042
to
ff2b547
Compare
This adds a new feature for adjusting the target latency closer to the live edge when playback is stable and back off away from the live edge when playback rebuffers. It is off by default and will only run a default of 5 times before it will stop trying to adjust the target latency.
Closes #6131.