[Merged] Power Saver feature
-
@meleu Hehe, i dont take any "prisoners of war". Its bad when bugs start being labelled as features.
@Sano Thank you. I dont understand what you mean by game number on carousel? Are you taking about the number of games in a system shown. That is available in the main branch since a long time.
@felleg Worry not, the screensavers video/dim/blank are working as expected with the PS mod. The logic is becoming very complex on the other hand.
@cyperghost there are some who use it on plasma or retro TV sets, might be for them?
Sent from 20,000 leagues under the sea.
Powersaver Emulation station : https://github.com/hex007/EmulationStation
ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh -
@Sano Thank you. I dont understand what you mean by game number on carousel? Are you taking about the number of games in a system shown. That is available in the main branch since a long time.
Yes, but I realized on your version that the number doesn't appear automatically after a delay. I have to press down direction for it to appear.
I like it, it's more convenient for me to not have this information unless I want it :) -
@Sano Again, bugs are not features. It might be okay for you but for someone who doesnt know about this behaviour will be perplexed and complain on forums.
Also what system are you getting this behaviour in?
-
All of them :) (Mame/FBA/NES/SNES/SMS/Megadrive/Ports/Retropie)
I have to press a button (not only the down direction after some tests) for the number of games to appear. -
System as in Pi1/2/3 Linux PC??
-
Sorry.
RPi3, retropie 4.2.8, 3A Aukru power supply. -
@Hex Okay... That's a point :) So ScreenBurn might be an issue.
Other question. Can you help me out please? I wanted to melt your branch together with @pjft but I wasn't able to do this ... The issue is posted here
I've also uploaded the binary you've provided for RPI1/0 build to my github
-
@Sano are you compiling the binary?
Sent from 20,000 leagues under the sea.
Powersaver Emulation station : https://github.com/hex007/EmulationStation
ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh -
-
Need testers to update and build to verify all changes. Finally please report any bugs (undocumented behaviour)
Final changes are made:
- PS enabled by default
- Removed PS enable warning
- Works nicely with all screensavers
- Gameinfo is shown with PS enabled too
- mVersion no longer shows PS state. Restored to default.
-
@Hex
I've uploaded binary to my github >> https://github.com/crcerror/emulationstation-binary-archive (only PI2/3)This is an merged version with your and @pjft favorite branch!
sadly it's an older release but it works the same way as @sano got -- CPU load is only 0,3%. Thank you mate. I'm sure this will be implented it's better as my suggestion ... the Standard ES branch still got a CPU load of 3% if Screensaver is enabeld.
-
Testers please check if GameInfo (XX Games bar) is properly shown when PS is enabled and all screensavers (video,dim,blank) are properly triggered. Thanks
Sent from 20,000 leagues under the sea.
Powersaver Emulation station : https://github.com/hex007/EmulationStation
ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh -
@Hex Compiled, tested. GameInfo display is back to normal.
Edit : and the screen just dimmed by itself after screensaver timeout. -
@Sano Please set screensaver to 1 min and wait if Dimming, Video and Blank are working
EDIT: Thanks for testing dimming - what about BLANK and Video? -
@cyperghost Black screen is ok too, but this minute is pretty long, I think the timer is buggy.
Currently I don't have any video to test video screensaver. -
@Sano Thank you can you make further tests please? It's important to hunt down possible bugs before a merge to the main branch can be done.
-
@Sano The minute is correct. I have timed it. It is just counting after some delay which you dont know about. The delay is present to finish any remaining animations and delayed displays like game count extra.
-
@felleg said in Testers needed :: Power Saver features :: PR #172:
This is a problem if you want to avoid screen burn-in. I'd much rather have the screensaver be able to start automatically, although, don't get me wrong, being able to start it manually is great in its own right.
I think that playing a video is not a "power saving" action. But now I think I better understood what the @Hex 's change is about.
It seems to be a fix in the logic, and not an addition of a new optional feature.
@Hex am I right saying that?
-
@meleu said in Testers needed :: Power Saver features :: PR #172:
.
Yes, It was a logic problem but such things get overlooked quite often. As I said, Aloshi's work might not have been optimized for Pi but it can be.
I agree Video screensaver is not a power saving feature but it is not intended to be. It is for systems that have a constant source of power and hence not battery friendly. But that does not mean that the feature is useless. A lot of effort and time went into making it worthwhile. :)
The option to enable or disable is just for your satisfaction. The functionality is not compromised at all. It just got better at doing nothing :P
Sent from 20,000 leagues under the sea.
Powersaver Emulation station : https://github.com/hex007/EmulationStation
ES dev script : https://github.com/hex007/es-dev/blob/master/es-tests.sh -
Once you fixed all those "issues" related with screensaver (video/dim/blank), then will it be enabled by default and with no option to toggle it?
Contributions to the project are always appreciated, so if you would like to support us with a donation you can do so here.
Hosting provided by Mythic-Beasts. See the Hosting Information page for more information.