plexinc / plex-media-player

Next generation Plex Desktop/Embedded Client
GNU General Public License v2.0
1.17k stars 170 forks source link

CEC behaviour on TV Power off #583

Open edrikk opened 7 years ago

edrikk commented 7 years ago

Hi,

Running PMP on a Pi 3, if the User presses the power off button while a show is playing, the current behavior appears to be that the show continues (with the TV off) until it completes.

I believe that the power off button should stop the video/audio/etc that is playing.

My setups: PMP on a Raspberry Pi 3 --> AVR (Yamaha YSP2200) --> Samsung LED TV (J6300)

vincegre commented 7 years ago

If you use CEC to control PMP, you can do that by tweaking CEC control in it to do something when it receives Power off from TV ! By default it does nothing :(

edrikk commented 7 years ago

Thanks @vincegre

The UI obviously doesn't allow for tweaking, but I have tried putting the CEC XML in the directory one level above the provided sample, and well... That basically resulted in no keys being recognized (with no changes).

I'm assuming the devs have to update the code... In all honesty, the "stop on power off" seems like something a UI made for running a TV should do...

OttoKerner commented 7 years ago

This is what I saw when pressing Power Off on the TV during a playback:

2017-08-24 23:30:23 [ DEBUG ] InputCEC.cpp @ 366 - CecCommand : Got a standby Request
2017-08-24 23:30:23 [ DEBUG ] InputCEC.cpp @ 378 - Unhandled CEC command  160 ,  "7 parameter(s) :[0]=8[1]=0[2]=46[3]=0[4]=9[5]=0[6]=1"
2017-08-24 23:30:25 [ DEBUG ] InputCEC.cpp @ 378 - Unhandled CEC command  132 ,  "3 parameter(s) :[0]=20[1]=0[2]=4"
LongChair commented 6 years ago

RPI3 : doesn't support standby, it can only support power off.

edrikk commented 6 years ago

Fair, but Plex can at least stop playback if it gets any of the above codes.

LongChair commented 6 years ago

Right, is that still an issue in current builds though ?

edrikk commented 6 years ago

There hasn’t been an emebeded build for over half a year... And none in the 2.X branch.

LongChair commented 6 years ago

@edrikk please check the test build here https://nightlies.plex.tv/public-test/plexmediaplayer/embedded-testing/ and let me know if that issue is still relevant :)

LongChair commented 6 years ago

@edrikk : could you please give us a status update with the build above ? Thanks.

edrikk commented 6 years ago

Hi, I now use an AppleTV, but brought out my Pi to test.... The issue was still present with PlexMediaPlayer-2.9.0.1

LongChair commented 6 years ago

@edrikk : thanks for taking the time to check this. I will investigate this shortly