-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
No way to check if automatic adaptation enabled #29
Comments
Hrm... It's easy enough to add, and I don't mind doing it. But out of curiosity, why is it useful? I had thought that there would be two basic use cases:
When would you want to query the state? How would your application lose track of whether or not adaptation was enabled? |
I agree with you that it is easy enough to keep track of but if it's ever ambiguous what the starting state is or just not known what the starting state is, it seems like there is the potential to get out of sync. To me it seems like it's the responsibility of the player to manage its own state and that way whoever is using it can't ever get out of sync or make the wrong assumption about what state it is in. Also, I'm happy to submit a pull request so you can concentrate on higher priority stuff. Maybe this won't be generally considered useful. |
I'm not sure it would get out of sync, though. It's easy to set explicitly as soon as Player is created. In any case, this debate is purely philosophical. I don't object to the feature. Please feel free to submit a pull request. It may be a bit more pain than contributing to the typical open-source project, but Google does require a one-time contributor agreement as outlined in CONTRIBUTING.md. And thanks for offering to contribute. You're the first! |
this is the way of checking if abr is enabled or not
it returns true or false accordingly. |
@Awais-cb, this thread was closed 2.5 years ago. Please try not to reopen very old threads. Shaka Player looks very different today than it did in 2015. Thanks! |
I can't see a way to check if adaptation is enabled or not. I think it would be useful to have a method player.getAdaptationEnabled() that just returns the value of adaptationEnabled_.
The text was updated successfully, but these errors were encountered: