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

Update AndroidX Media library to 1.4.x #7148

Merged
merged 1 commit into from
Oct 11, 2021

Conversation

TacoTheDank
Copy link
Member

@TacoTheDank TacoTheDank commented Sep 22, 2021

What is it?

  • Bugfix (user facing)
  • Feature (user facing)
  • Codebase improvement (dev facing)
  • Meta improvement to the project (dev facing)

Description of the changes in your PR

  • Update AndroidX Media 1.3.1 -> 1.4.2 (changelog)
  • 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

@TacoTheDank TacoTheDank changed the title Update AndroidX Media to 1.4.x Update AndroidX Media library to 1.4.x Sep 22, 2021
@litetex litetex self-requested a review September 25, 2021 08:38
@litetex
Copy link
Member

litetex commented Sep 25, 2021

  • 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.

How can I test this change / Where can I see it?
Is this only available for specific android versions?

@TacoTheDank
Copy link
Member Author

How can I test this change / Where can I see it? Is this only available for specific android versions?

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.

@litetex
Copy link
Member

litetex commented Sep 30, 2021

Documentation/Use cases for this feature are really rare.
I tried to search for "ACTION_SET_PLAYBACK_SPEED" but only got ~1k results when googling... no stackoverflow post etc with a practical implementation.
The only helpful result is this one from ExoPlayer: google/ExoPlayer#8229

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:
grafik

As there is no "change playback speed action" I would strongly push for removing the ACTION_SET_PLAYBACK_SPEED.

@TacoTheDank
Copy link
Member Author

@litetex Makes sense to me, thanks! Will remove, then.

Copy link
Member

@litetex litetex left a 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?

@tsiflimagas
Copy link
Contributor

I couldn't reproduce the problem on my device, Xiaomi Redmi note 8.

@TacoTheDank
Copy link
Member Author

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?

@tsiflimagas
Copy link
Contributor

Oh, excuse me, I'll keep it in mind :). I'm on a11 (custom ROM).

@litetex
Copy link
Member

litetex commented Oct 3, 2021

@TacoTheDank
I also had the problem yesterday in when using another git-branch. So I guess it's either the emulator or something got cached while building.
Anyway, if no further problems pop-up and nobody else manages to reviews this PR, I will re-test it on my real device and eventually merge it after the next release is out.

If then also other people get problems we can simply revert the commit and everything is fine 😄

@TacoTheDank
Copy link
Member Author

@litetex Sounds good to me. I don't think I noticed any problems myself on my Android 10 device.

Copy link
Member

@litetex litetex left a 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.

@litetex litetex merged commit 048743c into TeamNewPipe:dev Oct 11, 2021
@TacoTheDank
Copy link
Member Author

Thanks :)

@TacoTheDank TacoTheDank deleted the androidxMediaBump branch October 12, 2021 01:32
@Pentaphon
Copy link

Pentaphon commented Oct 29, 2021

Seems like ExoPlayer is moving into AndroidX Media:

Exoplayer was previously hosted in a separate ExoPlayer GitHub project. In AndroidX Media its package name is androidx.media3.exoplayer. We plan to continue to maintain and release the ExoPlayer GitHub project for a while to give apps time to migrate.

This was referenced Nov 30, 2021
@TacoTheDank TacoTheDank added the codequality Improvements to the codebase to improve the code quality label Jan 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
codequality Improvements to the codebase to improve the code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants