search

Slimbook-Team / slimbookbattery

Slimbook Battery 4
https://slimbook.es/en/tutoriales/aplicaciones-slimbook/398-slimbook-battery-3-application-for-optimize-battery-of-your-laptop
GNU General Public License v3.0
252 stars 18 forks source link

Regarding changing settings on connection to AC #90

Open siddhpant opened 2 years ago

siddhpant commented 2 years ago

Originally posted by me in https://github.com/Slimbook-Team/slimbookbattery/issues/88#issuecomment-1305484243.

Since it was too off-topic for that issue, hence created a new one. This is a continuation of that conversation.

once you connect the ac, governor will switch to performance , and stop using the bat settings (1st column). You can check it with sudo tlp stat -p

It doesn't change when I connect AC. I will use monitor mode of auto-cpufreq to show this (since it uses an update loop and shows battery status).

I set energy saving mode, and the output is as below which is expected (ignore suggestion):

image

Now when I connect battery, the output is as below:

image

So it didn't change.

If I change the mode to maximum performance, the output still remains the same, because the 1st column settings don't apply when on AC:

image

Here is the max performance setting for reference:

image

Also, it seems the "Disable animation" switch has negative logic. If I turn it on, then animations are enabled. Maybe it was intended to be yes/no, but then implemented as a switch.

slimbook commented 2 years ago

I see, in my tests does not happen, can you attach the file /etc/tlp.conf please?

siddhpant commented 2 years ago

I see, in my tests does not happen, can you attach the file /etc/tlp.conf please?

https://gist.github.com/siddhpant/86b13502b7787307baf526797b9297ff

slimbook commented 2 years ago

Try using the restore default values button, accept changes, and check that CPU_SCALING_GOVERNOR_ON_AC parameter is not empty.

slimbook commented 2 years ago

I've detected that this happens only on low config mode, I'm working to fix this.

slimbook commented 2 years ago

If you don't want to wait for the update, you can update the default configuration locally as in the commit, and restore values. /home/(user)/.config/slimbookbattery/default/ahorrodeenergia

siddhpant commented 2 years ago

Hello, sorry for replying late.

Try using the restore default values button, accept changes, and check that CPU_SCALING_GOVERNOR_ON_AC parameter is not empty.

I restored the default values, and checked. It is indeed empty.

$ grep CPU_SCALING_GOVERNOR_ON_AC /etc/tlp.conf
CPU_SCALING_GOVERNOR_ON_AC=

I've detected that this happens only on low config mode, I'm working to fix this.

I'm not sure what this means. If you are saying it won't happen if I am on other modes like balanced and max perf, then that is incorrect since it happens in it too.

$ grep CPU_SCALING_GOVERNOR_ON_AC .config/slimbookbattery/default/*
.config/slimbookbattery/default/ahorrodeenergia:CPU_SCALING_GOVERNOR_ON_AC=
.config/slimbookbattery/default/equilibrado:CPU_SCALING_GOVERNOR_ON_AC=
.config/slimbookbattery/default/maximorendimiento:CPU_SCALING_GOVERNOR_ON_AC=

And yeah, I am on an AMD laptop.