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

UI: add clearBuffer option to configure #1733

Closed
Kogoruhn opened this issue Dec 18, 2018 · 6 comments
Closed

UI: add clearBuffer option to configure #1733

Kogoruhn opened this issue Dec 18, 2018 · 6 comments
Assignees
Labels
component: UI The issue involves the Shaka Player UI status: archived Archived and locked; will not be updated type: enhancement New feature or request
Milestone

Comments

@Kogoruhn
Copy link

This will allow the player to change the quality without playback blinking.

ui/controls.js:

shaka.ui.Controls.prototype.onTrackSelected_ = function(track) {
  // Disable abr manager before changing tracks.
  let config = {abr: {enabled: false}};
  this.player_.configure(config);
  let clearBuffer = this.player_.getConfiguration().clearBuffer; // <-- changed
  this.player_.selectVariantTrack(track, clearBuffer); // <-- changed
};
@ismena
Copy link
Contributor

ismena commented Dec 18, 2018

Hi @Kogoruhn,
shaka.extern.PlayerConfiguration does not have a clearBuffer field.
Could you walk me through the use case for this? Is it that playback stalls for a sec while we're buffering the new resolution?

@ismena ismena added type: question A question from the community component: UI The issue involves the Shaka Player UI and removed needs triage labels Dec 18, 2018
@ismena
Copy link
Contributor

ismena commented Dec 18, 2018

Oh, are you proposing we add a clearBuffer config?
Sorry, I'm slow today :)

It would make more sense to be part of the UI config, but can certainly do it if that was the suggestion.

@ismena ismena added type: enhancement New feature or request and removed type: question A question from the community labels Dec 18, 2018
@shaka-bot shaka-bot added this to the Backlog milestone Dec 18, 2018
@Kogoruhn
Copy link
Author

@ismena Yep, just an enhancement request, sorry for unclear start message... Im slow anytime X)
Playback is blinking now, as on youtube. But in our project we want soft transition after resolution changes.
So we should keep current buffer info and wait for the next chunks with new resolution.

@ismena
Copy link
Contributor

ismena commented Dec 18, 2018

Gotcha!
Spasibo, we'll work on it.

@ismena
Copy link
Contributor

ismena commented Jun 5, 2019

Ahh, we completely lost track of this :(
@joeyparrish Could you schedule? I don't think this is a UI enhancement also, but a player one.

@ismena
Copy link
Contributor

ismena commented Aug 16, 2019

Sorry this took forever :(
uiConfig.clearBufferOnQualityChange setting is at your service!

@joeyparrish joeyparrish modified the milestones: Backlog, v2.6 Aug 16, 2019
TheModMaker pushed a commit that referenced this issue Aug 22, 2019
Resolutions menu calls player.selectVariantTrack() when
a user selects a resolution. SelectVariant track has a
clearBuffer argument that we used to always set to true.
However, an app may want it to be false. This change
makes this behavior configurable.

Closes #1733.

Change-Id: I6a0dbcca99a8322d68abd0151afd66af31530856
@shaka-project shaka-project locked and limited conversation to collaborators Oct 15, 2019
@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: UI The issue involves the Shaka Player UI status: archived Archived and locked; will not be updated type: enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants