-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Update AndroidX Media library to 1.4.x #7148
Conversation
How can I test this change / Where can I see it? |
I do not know if it is visible anywhere or can be tested, which is why I'm asking if it really needs to be kept. Perhaps it could be utilized in some way. However in hindsight I probably should've just asked in the PR before making that change. |
Documentation/Use cases for this feature are really rare. I think all the available actions are for the notification (I don't know what it's called correctly) that is visible when e.g. using the background-player: As there is no "change playback speed action" I would strongly push for removing the |
79bfca0
to
a953aab
Compare
@litetex Makes sense to me, thanks! Will remove, then. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think there is something weird happening with this dependency update.
Every time I installed the apk/cleared all data/(sometimes)started the apk it had massive sound stuttering/speed up (while video was normal) until I set the speed/pitch of a video or the audio run out of buffer.
Tested on the following devices:
- Emulator - Pixel 3a Android 11 (API 30) ❌ Bug occurred
- Emulator - Pixel 3a Android 12 (API 31) ❌ Bug occurred
- Huawei P20lite Android 9 (API 28) ✔️ Couldn't reproduce problem
The latest release (0.21.10) is also working fine on the emulators.
Maybe we have to update ExoPlayer?
Can someone cross-check if this is also happening on other devices?
I couldn't reproduce the problem on my device, Xiaomi Redmi note 8. |
As a note for future reference, it is pretty much standard protocol to also share the Android version of the device :) what was yours? |
Oh, excuse me, I'll keep it in mind :). I'm on a11 (custom ROM). |
@TacoTheDank If then also other people get problems we can simply revert the commit and everything is fine 😄 |
@litetex Sounds good to me. I don't think I noticed any problems myself on my Android 10 device. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested it again and was unable to reproduce any problems now.
Thanks :) |
Seems like ExoPlayer is moving into AndroidX Media:
|
What is it?
Description of the changes in your PR
Add a new action from version 1.3.0-beta01, which adds support for the set playback speed command. Please advise on whether this change should be kept.APK testing
The APK can be found by going to the "Checks" tab below the title. On the left pane, click on "CI", scroll down to "artifacts" and click "app" to download the zip file which contains the debug APK of this PR.
Due diligence