[Merged] Power Saver feature

pjft

Agreed, good call. Any text that's larger than the container will scroll, I think.

But if it scrolls once and then stops at the beginning, I don't have much of an issue for the moment.

Hex

Game names were working well before. Descriptions were not. Now they are working but it is a hack as i had some work and needed it tested before that. Can everyone who can test it, GIT pull and then build.

pjft

@Hex I'll look into it in the coming days, if not sooner. Thanks!

dankcushions

tested (pi3). builds and looks fine to me! descriptions and game titles scroll fine. i don't use a video theme, so can't comment on that.

idle cpu went from 6-7% (dimmed) to 3% (dimmed/not)

cyperghost

@Hex
There is still issue with removing favorite by pressing Y button if you are alredy in favorite - even with your newest branch and pjfts newest commit.

All your features... Dimming, Blanking and CPU savings work
Scrolling I didn't test
Gamelist shows number of game entries without pressing button.

I load binary to my github repo: This is a merged branch @pjft favorite and your newest commits (4 times cherry pick). Is there a better way to reload all cherry picks with only one command? I used the command git cherry-pick abcdef12345678790 4 times

https://github.com/crcerror/emulationstation-binary-archive

Hex

@cyperghost you should use git pull --rebase to update your repository. You dont need to cherry pick every time

I have not tested my changes with the branch you are testing. If that PR gets merged first then i can test it. I will not test it on unmerged code.

pjft

@Hex I agree. That's the wise approach to take.

cyperghost

@pjft said in Testers needed :: Power Saver features :: PR #172:

@Hex I agree. That's the wise approach to take.

Is it rather about:

Don't trust a binary compiled by someone else
Don't do working on a fix that may never be needed because the branch isn't merged into main release?
Annother codex or good practice I don't know about it?

@Hex
I think fix is easy ... but needs some work in looking @pjfts routines.
Anyway, I hope that my posting can be treated as alpha test.

ah.. and thanks for helping in using gitcommands

pjft

@cyperghost No, no, don't get this wrong :)

It's just that:
a) he won't have the source code to exactly debug it, by just using the binary -- we trust you that it crashes, but without the exact code, it's a bit of guesswork on his end as he doesn't know what code to look at exactly.
b) given that my current branch (and his branch) are both currently being developed, it may be that any time invested at this stage will be rendered obsolete - either because the bug may disappear, or because it may resurface while we keep developing.

As such, after one of them is merged, it's easier for everyone to have a stable code-base to use for actual runtime debugging. The best he could attempt to do right now would only be to try to replicate it on his current code base, seeing if deleting an entry from the list would trigger the same error. That way he would be able to narrow it down to the code to remove an entry from the list.

But if that doesn't work, he's kind of flying blind :)

Thanks for testing!

Hex

@cyperghost #3 definitely. You see features should inherently be isolated. Convoluted features should either be worked in a single PR or sequentially . This maintains the stability of the system. Simultaneous testing two PRs is very unwise.

Consider you are cooking two dishes. What do you think is better, making them one after another or at the same time? The prior is easier if you are making elaborate dishes that need great attention and timing. The later is easier if you are making scrambled eggs and toast because they are not convoluted; as in they dont require the same resources. Got it now?

cyperghost

@Hex @pjft
Sounds like you both are really know what you are doing. Thank you for clearing out :)

Hex

@pjft said in Testers needed :: Power Saver features :: PR #172:

@cyperghost No, no, don't get this wrong :)

It's just that:
a) he won't have the source code to exactly debug it, by just using the binary -- we trust you that it crashes, but without the exact code, it's a bit of guesswork on his end as he doesn't know what code to look at exactly.

@pjft I can easily reproduce the code that he is working with. I know which commands were used to get a working copy. Problem is if that doesnt exhibit the bug.

b) given that my current branch (and his branch) are both currently being developed, it may be that any time invested at this stage will be rendered obsolete - either because the bug may disappear, or because it may resurface while we keep developing.

@cyperghost This is the main reason and my post above elaborates on this idea with an example ;)

As such, after one of them is merged, it's easier for everyone to have a stable code-base to use for actual runtime debugging. The best he could attempt to do right now would only be to try to replicate it on his current code base, seeing if deleting an entry from the list would trigger the same error. That way he would be able to narrow it down to the code to remove an entry from the list.

But if that doesn't work, he's kind of flying blind :)

Thanks for testing!

Also I am completely unaware on what they are developing and what is the expected functionality. So for me everything is unexplored territory.

If i am able to find a stable solution how would you recommend me going back to my code to implement it? I might not have the interface with his code to apply the fix.

Hex

BUG : PS should be disabled while Scrapper is running

Hex

Guys I thought of a solution. Instead of having a switch how about having options(List)

PowerSaver Mode :-

None ~~~~~~No power Saving/Full performance
Minimal ~~~Power savings kicks in after Description has scrolled once, same time-out for Systems view)
Full ~~~~~~~PS kicks in after Game name has scrolled, description may not scoll
MAX ~~~~~~Experimental; Works well with Instant transition and no carousel animation

All of these will work fluidly with all Screensavers and with Video Previews.

This is the only way I can think of that will allow proper PowerSavings settings and allow user to decide till which level is the compromise acceptable.

Zigurana

@hex
I would ask, what is the difference?
Not the difference in text elements scrolling or whatnot, but what is the difference in power saved when going from min to max?
How much battery - life do users gain?

I have the feeling that the bulk of the improvements are already achieved when usin PS=min, and now you are trying to squeeze out some more improvements that have little impact in day-to-day use.
80-20 pareto, perfect being the enemy of the good, and all that.

Despite all our work on ES, remember that users will spend most of their time (and power) while running actual emulators, not just the front-end.

Hex

@Zigurana Minimal is the same as None if you are browsing the gamelist (with or without VideoP). This is because PS is not triggered till 10-12 seconds after last key pressed as description is scrolling. If someone is not interested in description then Full saves ~10 seconds of unnecessary processing per event.

How much battery life is saved, who knows. Havent profiled yet. Remember that some builds use Pi zero and lesser the processing lesser the heat. For those who are on Pi3 Min might be ideal. For those looking to squeeze every drop, MAX might be an alternative :P

MAX is total bonkers. Its branded experimental and might not be implemented.
This is to know your opinion before i implement it.

ES without PS heats pi more than Emulators. You see Emulators are meant for performance and likely so take more CPU usage. ES is an interface and its power consumption should be as low as possible dont you agree?

Zigurana

@hex You are of course correct in all your points. I wonder not if it matters, but how much, this needs to be considered as well, especially if one solution is more restrictive than the other.

For instance if the effective gain of max vs min is only 3% (because in the long run, these initial 12 sec do not matter) I would not introduce an additional user option. On the other hand, if this brings a significant improvement, albeit at the cost of some penalty, then a user option would be the way to go.

Now, we might never find out hard data on ES vs emulator usage, and therefore the precise savings that this will realize. But at least we can try to come up with a very rough approximation:

Assume about 50% usage of ES, of which the user would scroll through games about 50% of the time, and 50% is spend idling.
With PS=min we are impacting 25% of the power usage, while PS = max impacts all ES usage (50%).
Is it fair to assume that the idling will be mostly condensed in long stretches, because someone is leaving their machine running while having fun outside in the sunshine? Maybe, maybe not.
The power savings are significant, a factor of 10, I believe?

Using PS = min, we arrive at a power reduction of 22.5% (averaged over time), while PS = max gives a whopping 45%!

If this seems sensible, I would say implement PS = min as the default implementation and allow the user to enable PowerSaving (which is PS = Max) if they are willing to play the price in functionality. So no PS = NO, as PS after 12 second is basically painless, right? So the UI becomes a simple boolean, where we can also communicate the consequences of using PS in the extreme sense (no scrolling text, no videos, etc).

Hex

@Zigurana There is not much difference between Boolean and list. That much complexity is fine with me. List allows us to add further options later if necessary. Full allows PS for people who dont care about Description. I use Full all the time. My theme does not even have Description. Why should I have to be restricted to using Minimal Mode?

Remember all video stuff works well with PS. Omx does not even need PS to be disabled. So it makes sense. I would like to default PS to Minimal and allow users to decide if they want more power savings or not. It doesnt matter how much Mins you can squeeze out of this. I am against not having option to not care for description. Just as many users read description, many users dont. I dont want to punish them for that.

pjft

@hex said in Testers needed :: Power Saver features :: PR #172:

My theme does not even have Description.

Should that not be optimizable then in itself, regardless of PS being minimal or max? Same for empty descriptions, or descriptions that fit the container?

Hex

@pjft Well PS trigger is timer based so even if description is not present PS will wait full 12 secs after last event for (Missing) description to scroll. Currently PS and Scrolling is sped up for testing in my branch. Check out the stock for how long the scrolling takes.

If i have to code PS to factor in so many things as Video playing or not, Screensaver working or not, description scrollin or not, It will become too complex to code, manage and make changes to.

Description is scrollable text view and has fixed timeouts for start scrolling and repeat from start. It has no way of knowing how much time it actually scrolls. That depends on the size of text. So if description is too long then scrolling is also time consuming. 10 secs is lower bound to this.