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

Release 0.23.0 (please TEST!) #8230

Closed
12 tasks done
Stypox opened this issue Apr 16, 2022 · 21 comments · Fixed by #8249
Closed
12 tasks done

Release 0.23.0 (please TEST!) #8230

Stypox opened this issue Apr 16, 2022 · 21 comments · Fixed by #8249

Comments

@Stypox
Copy link
Member

Stypox commented Apr 16, 2022

Testing for regressions

Debug APK (built by our CI in #8231): NewPipe_0.23.0_RC2.zip
Release APK (signed with Stypox's keys): NewPipe_0.23.0_RC2.zip
Please report only regressions (i.e. new issues) here, not issues that were already present in the previous release!

TODO

NewPipeExtractor version

This version of NewPipe will ship with NewPipeExtractor version 0.22.1

App changelog

New

Improved

Fixed

Development

@Stypox Stypox changed the title Release 0.23.0 Release 0.23.0 (please TEST!) Apr 16, 2022
@Stypox Stypox pinned this issue Apr 16, 2022
@Stypox
Copy link
Member Author

Stypox commented Apr 16, 2022

@tsiflimagas reported this crash on IRC, it happens in the release build when tapping on Channels in the New stream notifications settings.
Edit: fixed by #8236

Crash log

Exception

  • User Action: ui error
  • Request: ACRA report
  • Content Language: en-GB
  • Service: none
  • Version: 0.23.0
  • OS: Linux Android 10 - 29
androidx.fragment.app.Fragment$InstantiationException: Unable to instantiate fragment org.schabi.newpipe.settings.notifications.NotificationModeConfigFragment: make sure class name exists
	at androidx.fragment.app.FragmentFactory.loadFragmentClass(FragmentFactory.java:97)
	at androidx.fragment.app.Fragment.instantiate(Fragment.java:611)
	at androidx.fragment.app.FragmentContainer.instantiate(FragmentContainer.java:57)
	at androidx.fragment.app.FragmentManager$3.instantiate(FragmentManager.java:483)
	at org.schabi.newpipe.settings.SettingsActivity.instantiateFragment(SettingsActivity.java:181)
	at org.schabi.newpipe.settings.SettingsActivity.onPreferenceStartFragment(SettingsActivity.java:174)
	at androidx.preference.PreferenceFragmentCompat.onPreferenceTreeClick(PreferenceFragmentCompat.java:422)
	at androidx.preference.Preference.performClick(Preference.java:1208)
	at androidx.preference.Preference.performClick(Preference.java:1182)
	at androidx.preference.Preference$1.onClick(Preference.java:182)
	at android.view.View.performClick(View.java:7259)
	at android.view.View.performClickInternal(View.java:7236)
	at android.view.View.access$3600(View.java:801)
	at android.view.View$PerformClick.run(View.java:27892)
	at android.os.Handler.handleCallback(Handler.java:883)
	at android.os.Handler.dispatchMessage(Handler.java:100)
	at android.os.Looper.loop(Looper.java:214)
	at android.app.ActivityThread.main(ActivityThread.java:7356)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:491)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:940)
Caused by: java.lang.ClassNotFoundException: org.schabi.newpipe.settings.notifications.NotificationModeConfigFragment
	at java.lang.Class.classForName(Native Method)
	at java.lang.Class.forName(Class.java:454)
	at androidx.fragment.app.FragmentFactory.loadClass(FragmentFactory.java:53)
	at androidx.fragment.app.FragmentFactory.loadFragmentClass(FragmentFactory.java:94)
	... 20 more
Caused by: java.lang.ClassNotFoundException: org.schabi.newpipe.settings.notifications.NotificationModeConfigFragment
	... 24 more

@litetex
Copy link
Member

litetex commented Apr 16, 2022

Possibly see if #7630 can be solved in this release

I think we have some kind of buffering problem which now affects all streams (until 0.23.0 it only affected YT 60fps streams). I created a new issue for that: #8238

@shom13 shom13 mentioned this issue Apr 17, 2022
4 tasks
@VaclavTrpisovsky

This comment was marked as resolved.

@Stypox

This comment was marked as resolved.

@VaclavTrpisovsky

This comment was marked as resolved.

@karyogamy
Copy link
Contributor

Found another issue in #8150: After selecting No Caption before closing a player instance, open a new next player instance, and its track selector will automatically select a caption, even if caption is disabled as mentioned.

@SameenAhnaf
Copy link
Collaborator

SameenAhnaf commented Apr 18, 2022

  1. When system dark mode is enabled, a lot of components lack contrast. This isn't the case when theme is changed from app settings.
Screen recording
Screenrecorder-2022-04-18-23-47-32-352.mp4

First bug is removed as it was reproducible in previous app versions

@peat80

This comment was marked as resolved.

@AudricV

This comment was marked as resolved.

@gardockt
Copy link

After a new upload I received a notification not only about this video, but also about 2 other videos on different channels, both uploaded months ago - one of which was not even a last upload on the channel. That happened on RC2, on a first automatic refresh.

@Stypox
Copy link
Member Author

Stypox commented Apr 22, 2022

When system dark mode is enabled, a lot of components lack contrast. This isn't the case when theme is changed from app settings.

@SameenAhnaf sorry, I don't understand what you mean. Can you take a screenshot and make a circle around what you think is not colored correctly? Or could you tell me which icons are wrong? I don't see a problem in your video

After a new upload I received a notification not only about this video, but also about 2 other videos on different channels, both uploaded months ago - one of which was not even a last upload on the channel. That happened on RC2, on a first automatic refresh.

@gardockt This can happen on the first refresh, since the database might miss some videos the first time. Please let us know if this keeps happening even after the first refresh, as that would really be a problem.

@smailyliams

This comment was marked as off-topic.

@AudricV

This comment was marked as off-topic.

@Stypox
Copy link
Member Author

Stypox commented Apr 25, 2022

I would like to release tomorrow, almost everything should be ready.

Possibly including @TiA4f8R's extractor improvement to solve potential future throttling issues

@TiA4f8R could you open a PR or delegate that to me? If I understood correctly it is just a simple change (replacing a HashMap with a List), so I can do that myself in a few minutes (unless I'm not understanding something correctly)

DEFINITELY

The only reason I said "maybe" was because the fastlane changelog does not include that change, but we would like users to know about such an important change. Maybe we should include it in the next release changelog, so that at least people can know by looking at the changelog?

When system dark mode is enabled, a lot of components lack contrast. This isn't the case when theme is changed from app settings.

This is still not fixed, but I still don't understand what needs to be solved. I don't see anything badly-themed, except maybe the brightness/volume, but I am not sure. @SameenAhnaf

@SameenAhnaf
Copy link
Collaborator

SameenAhnaf commented Apr 25, 2022

@Stypox Extremely sorry for the late reply. Following components lack contrast.

  1. Brightness and Volume sliders
  2. Subscribe icon
  3. Feed update bar
Stable Version Version 0.23.0_RC2
Before.mp4 After.mp4

@Stypox
Copy link
Member Author

Stypox commented Apr 26, 2022

@SameenAhnaf ok, thank you, I could see the differences by pausing the videos at the same time. What is happening is really strange, since I cannot reproduce on an emulator. Maybe there is a bug in your ROM or something like that, since NewPipe's code does the same thing (i.e. calling setTheme(BlackTheme)) both when the theme selection is done automatically or not. So I would not focus on this really small bug and release even in its presence. But thank you @SameenAhnaf for noticing, please open an issue and we can ask @mauriciocolli to look into this (he is the author of #7518) :-)

@opusforlife2
Copy link
Collaborator

Maybe we should include it in the next release changelog, so that at least people can know by looking at the changelog?

@Stypox Can the current changelog not be edited?

@Stypox
Copy link
Member Author

Stypox commented Apr 28, 2022

@opusforlife2 the English one, yes, the translated ones no, since we don't know all languages. I would just include that change in the next changelog.

@Stypox Stypox closed this as completed Apr 29, 2022
@moriel5
Copy link

moriel5 commented May 1, 2022

I'll test streaming in a bit, however on version 0.23 downloads are still throttled for me in the same manner as on 0.22.2, regardless of which stream is being requested.

@Stypox
Copy link
Member Author

Stypox commented May 1, 2022

There has been no changes about throttling between 0.22.2 and 0.23.0

@moriel5
Copy link

moriel5 commented May 1, 2022

@Stypox Ah, sorry for misunderstanding then, it simply sounded as though a solution was found on the blog post.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.