NewPipe icon indicating copy to clipboard operation
NewPipe copied to clipboard

No fullscreen possible with Android TV

Open dolorosus opened this issue 2 years ago • 59 comments

Checklist

  • [X] I am able to reproduce the bug with the latest version.
  • [X] I made sure that there are no existing issues - open or closed - which I could contribute my information to.
  • [X] I have read the FAQ and my problem isn't listed.
  • [X] I have taken the time to fill in all the required details. I understand that the bug report will be dismissed otherwise.
  • [X] This issue contains only one bug.
  • [X] I have read and understood the contribution guidelines.

Affected version

v0.24.0

Steps to reproduce the bug

  1. Start any video and try to switch to fullscreen mode results in an error message (see screenshot).
  2. No chance to continue the video, 'cause only an remote control is attached, but got no focus to the play button. Only chance is going back and restart the video which brings us back 1.

Expected behavior

entering fullscreen mode

Actual behavior

see attached [screenshot]

Screenshots/Screen recordings

Screenshot 2022-09-25 210614

Logs

No response

Affected Android/Custom ROM version

AndroidTV 9 and FireTV

Affected device model

No response

Additional information

No response

dolorosus avatar Sep 25 '22 19:09 dolorosus

Thank you. Being polite never killed anyone.

opusforlife2 avatar Sep 25 '22 19:09 opusforlife2

The screenshot you've posted is of the toast. Can you check your notifications to fetch the actual error?

opusforlife2 avatar Sep 25 '22 19:09 opusforlife2

I know the screenshot give no useful information, but how should I access notifications AndroidTV?

dolorosus avatar Sep 25 '22 20:09 dolorosus

Okay, it seems like providing app notifications on Android TV is a no-no: https://scribe.rip/android-tv-application-notifications-on-o-c6820ada2086

@Stypox Can it be made so that errors which would generate error notifications on handhelds are made to open the error activity on TV instead?

opusforlife2 avatar Sep 25 '22 20:09 opusforlife2

Same issue on Android TV. Videos in non-full screen play fine in v0.24 but when clicking on the full screen icon, the screen is filled with green except the video window stays the same size. At that point, the video freezes, but the audio continues. When I back out of full screen, it then crashes. (previous recent versions of NewPipe worked well with no issues) Screenshot_20220928-091531 Screenshot_20220928-092145 Screenshot_20220928-092156 I don't have time right now to figure out how to submit this report in live text format from Android TV, so I made some screenshots for now.

j-seeley avatar Sep 28 '22 13:09 j-seeley

Same Issue on a FireTV 4K stick. Videos do not open in Fullscreen and when manually clicking on Full-Screen newpipe crashes...

fliegentoeter avatar Sep 28 '22 16:09 fliegentoeter

I have the same issue on Sony Android TV., now I reinstalled the 23.3 version, that works. may I suggest to add a shortcut to switch into/out of fullscreen that can be used from a tv remote?

migbox avatar Sep 30 '22 15:09 migbox

may I suggest to add a shortcut to switch into/out of fullscreen that can be used from a tv remote?

Is the full screen button non-selectable or invisible?

opusforlife2 avatar Sep 30 '22 16:09 opusforlife2

I have the same issue on Sony Android TV., now I reinstalled the 23.3 version, that works. may I suggest to add a shortcut to switch into/out of fullscreen that can be used from a tv remote?

I'm afraid that won't help. As soon as you switch to full screen mode, you get an error message. After that, no further operation is possible, because none of the controls get the focus anymore.

It would make more sense to eliminate the cause. in versions <23.0 it worked, only since V23.0 there are problems with the fullscreen mode.

dolorosus avatar Sep 30 '22 16:09 dolorosus

What i've learned on AndroidTV: Do not update newpipe when everything is working, wait until youtube makes changes and you are forced to update :) Same goes for a lot of other apps that are simply not designed for AndroidTV...

fliegentoeter avatar Sep 30 '22 17:09 fliegentoeter

What i've learned on AndroidTV: Do not update newpipe when everything is working, wait until youtube makes changes and you are forced to update :) Same goes for a lot of other apps that are simply not designed for AndroidTV...

What your actual learning should be is that you should help us test upcoming changes on your TV device so that our entire TV userbase doesn't have to suffer when a new version is released.

No one in the dev team has an Android TV. If no user steps up to help test, then issues like this are bound to come up again and again.

opusforlife2 avatar Oct 01 '22 00:10 opusforlife2

I will be glad to do tests on your behalf.

The Sony Android TV has been the best TV I ever bought, I strongly recommend it to people who need to buy a new tv

migbox avatar Oct 01 '22 07:10 migbox

... No one in the dev team has an Android TV. If no user steps up to help test, then issues like this are bound to come up again and again.

Maybe you should go a little bit more into detail how to get a test version?

dolorosus avatar Oct 01 '22 08:10 dolorosus

I get that. But I have no idea how to test these versions and/or install them with a regular newpipe version installed. It is quite complicated to downgrade by itself and copy/import my old settings with only a remote control by itself. So my solutuion is simply to run the version until something breaks and upgrade then when I have to. There are no notifications, so I do not even get a notice when a new version would be available in Neo-Store (only Fdroid-Store I found usable with a remote control). As mentioned before, the old v23 from fdroid works great.

fliegentoeter avatar Oct 01 '22 08:10 fliegentoeter

I will be glad to do tests on your behalf.

@migbox Shall I add your name to https://github.com/TeamNewPipe/NewPipe/wiki/Testers#android-tv?

Maybe you should go a little bit more into detail how to get a test version?

@dolorosus At the bare minimum, we need our release candidates tested to catch any regressions (bugs that are not present in the current release, but were added by new changes since then). You can grab these from the "RC [version number]" issues that we pin for at least a week before releasing a new version, precisely for this reason.

Apart from that, every PR that could possibly affect TV users should be tested. For the most part, this includes PRs that affect the player code. The PR description has a "testing" section which describes how to obtain the needed debug APKs for this.

But I have no idea how to test these versions and/or install them with a regular newpipe version installed.

@fliegentoeter Stypox did some magic years ago which lets you install several different versions of Newpipe side by side with the current release. The APKs you download from different PRs or from the RC issues have separate package names. This means that if you have 2 APKs from 2 different PRs, they will install side-by-side. But if you download a newer APK from one of the PRs, one of the 2 APKs will get updated. This means you can continue to test newer changes in PRs as they get updated, and also that you can test multiple PRs in parallel.

You should never need to touch your official release version for any reason. One of our developers used to keep a record 60+ Newpipe builds on their device at a time.

And there is no need at all to import/export anything. You should definitely keep a database export of your current release build as a safety net in any case. Just make sure to never mess with database backups with development builds, because any changes there could potentially corrupt your "true" database of the release version.

opusforlife2 avatar Oct 01 '22 10:10 opusforlife2

I use to download new versions to my gdrive and install it from there using the app Solid explorer . However it should be interesting a way to save easily bookmarks and settings when I will need to downgrade.. Does anyone of you have simple instructions?

migbox avatar Oct 01 '22 17:10 migbox

No one in the dev team has an Android TV....

No problem, just add a FireTV Stick to your Amazon wishlist and send me a link to your Amazon wishlist.

dolorosus avatar Oct 01 '22 18:10 dolorosus

I'm using a Sony Android tv 9 and have the same problem with the lastest f-droid's version. full screen in 4k and 1080p doesn't work. An extra is kind of difficult to move arroud the app with the remote control.

ingmarcosp avatar Oct 01 '22 19:10 ingmarcosp

No problem, just add a FireTV Stick to your Amazon wishlist and send me a link to your Amazon wishlist.

You just quoted the same comment again and had a different 'advice' to give. The dev team isn't full of paupers. If you're not interested in testing, then have patience and wait for a fix. Don't complain if you're not ready to be a part of the solution.

opusforlife2 avatar Oct 01 '22 20:10 opusforlife2

@migbox Read my big comment again closely. You will never need to downgrade at all. Keep your release version installed. Keep a fresh database export ready in case of emergencies. Then install the APK from the RC issue for version 0.24.0 as a test. You will see that it installs as a completely separate app with its own database and settings. It won't interact with your main app at all.

opusforlife2 avatar Oct 01 '22 20:10 opusforlife2

Sure, you can add me. Of course I need to know where I can get the APK to test and how do I have to report the tests results

migbox avatar Oct 02 '22 09:10 migbox

@migbox Thanks! We'll ping you and help you figure out how to provide relevant info when needed.

opusforlife2 avatar Oct 02 '22 16:10 opusforlife2

@opusforlife2

I am a heavy Firestick user and would be happy to do some testing as well to help the effort. I have noticed some substantial differences between the way the 4k and 4k Max behave so I can test on both.

IMMalebolgia avatar Oct 07 '22 16:10 IMMalebolgia

@IMMalebolgia Thanks! To be sure, are you referring to this specific issue or do you wish to be a TV tester for us in general?

opusforlife2 avatar Oct 07 '22 18:10 opusforlife2

Happy to be a tester in general.

I have been an EE for 26+ years now specializing in post silicon debug so plenty of experience tracking down wierd SW/HW bugs so figured I might be of service to an app I rely heavily on. I previously assumed you had plenty of testers or else I would have volunteered sooner 🙂

IMMalebolgia avatar Oct 07 '22 18:10 IMMalebolgia

Hi, below is the error I get in logcat when trying to play video fullscreen on Android TV. Visible symptoms are the same as the other guys are getting. Android version 11, vendor Philips, uname -a says Linux localhost 4.19.182-ab2437 #1 SMP PREEMPT Fri Jul 15 10:35:27 CST 2022 armv8l.

10-07 22:11:36.164 12647 12647 E Player  : ExoPlayer - onPlayerError() called with:
10-07 22:11:36.164 12647 12647 E Player  : com.google.android.exoplayer2.ExoPlaybackException: Unexpected runtime error
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:616)
10-07 22:11:36.164 12647 12647 E Player  : 	at android.os.Handler.dispatchMessage(Handler.java:102)
10-07 22:11:36.164 12647 12647 E Player  : 	at android.os.Looper.loop(Looper.java:223)
10-07 22:11:36.164 12647 12647 E Player  : 	at android.os.HandlerThread.run(HandlerThread.java:67)
10-07 22:11:36.164 12647 12647 E Player  : Caused by: java.lang.IllegalStateException
10-07 22:11:36.164 12647 12647 E Player  : 	at android.media.MediaCodec.native_setSurface(Native Method)
10-07 22:11:36.164 12647 12647 E Player  : 	at android.media.MediaCodec.setOutputSurface(MediaCodec.java:2146)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.mediacodec.SynchronousMediaCodecAdapter.setOutputSurface(SynchronousMediaCodecAdapter.java:189)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.video.MediaCodecVideoRenderer.setOutputSurfaceV23(MediaCodecVideoRenderer.java:1451)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.video.MediaCodecVideoRenderer.setOutput(MediaCodecVideoRenderer.java:646)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.video.MediaCodecVideoRenderer.handleMessage(MediaCodecVideoRenderer.java:584)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.ExoPlayerImplInternal.deliverMessage(ExoPlayerImplInternal.java:1546)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.ExoPlayerImplInternal.sendMessageToTarget(ExoPlayerImplInternal.java:1510)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.ExoPlayerImplInternal.sendMessageInternal(ExoPlayerImplInternal.java:1485)
10-07 22:11:36.164 12647 12647 E Player  : 	at com.google.android.exoplayer2.ExoPlayerImplInternal.handleMessage(ExoPlayerImplInternal.java:521)
10-07 22:11:36.164 12647 12647 E Player  : 	... 3 more
10-07 22:11:36.166   397   397 E Layer   : [SurfaceView - org.schabi.newpipe/org.schabi.newpipe.MainActivity#0] No local sync point found

ghost avatar Oct 07 '22 20:10 ghost

I am Android TV Box user( a low end device) so maybe i can give some feedback on testing .My device is B860H V5.0(Google certified Android Tv OS) I already tested 0.24.0 but different from the others. When i click fullscreen just upper half get the video player and the lower half get Title. when Fullscreen icon clicked again it's not responding on meda player.

NeloKa61 avatar Oct 08 '22 12:10 NeloKa61

Yes, the button is visible, but to act on it, you can only use the arrows on the remote walking around the screen trying to have the focus on full screen icon. Unfortunately when you try to go there using the down arrows, the focus goes on the time slider; the full screen is on his right and if you try to move on it, what you obtain is moving the time slider on the right. To reach the full screen icon, you have to move first to the top screen, then on the right until you reach the next title column, 3 times down to align with the full screen icon, then once on the left and you will be on the icon and can press enter

Il giorno ven 30 set 2022 alle ore 18:38 opusforlife2 < @.***> ha scritto:

may I suggest to add a shortcut to switch into/out of fullscreen that can be used from a tv remote?

Is the full screen button non-selectable or invisible?

— Reply to this email directly, view it on GitHub https://github.com/TeamNewPipe/NewPipe/issues/9023#issuecomment-1263789217, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB5BHJ2L7VBXERYGS2M2D7LWA4JR5ANCNFSM6AAAAAAQVGWPY4 . You are receiving this because you commented.Message ID: @.***>

migbox avatar Oct 11 '22 08:10 migbox

I uninstalled v24 but before doing so, I saved/copied four files to a save place: 1) newpipe.db 2) newpipe.db-shm 3) newpipe.db-wal and 4) org.schabi.newpipe_preferences.xml (not sure if all four of these are needed but it was just hit or miss)

After uninstalling v24, I then installed v23.3 and ran the app to get the db and xml files to populate so I know where to then replace those with the four above. I got all of my settings back as well as search and view history. Good enough for me.

I have a simple and old Android TV Box (rooted) running Android 6. Because it's rooted, I had access to those files.

j-seeley avatar Oct 11 '22 21:10 j-seeley

Almost nobody seems to know that if you press down on your remote another time when the progressbar is focused it will then select the fullscreen button. That should be noted somewhere. 🙄

peat80 avatar Oct 14 '22 18:10 peat80