Charge mode is overridden automatically when plugged in #72

New issue

Closed

opened 2023-02-12 01:17:53 +00:00 by NGnius · 1 comment

NGnius commented

2023-02-12 01:17:53 +00:00

(Migrated from github.com)

Please confirm

I have searched existing issues
This issue is not a duplicate of an existing one
I will fill this out to the best of my ability

Expected Behaviour

PowerTools's charge mode override is respected.

Actual Behaviour

Charge mode is set to Normal until the next PowerTools config update (resume from sleep, slider change, etc.)

Steps To Reproduce

Set charge mode to anything except normal in PowerTools
Plug Deck into charger
Watch Deck charge
Update PowerTools config and watch is go back to whatever behaviour PowerTools is set for

Anything else?

Cool project, wish I'd come up with something like this!

But seriously: research is required to figure out how to capture charging events. Ideally without having to hook into the Steam client ones because that's too error prone and it's also horrible design.

Version

1.1.0 (Latest stable)

Platform

Steam Deck

OS

### Please confirm - [X] I have searched existing issues - [X] This issue is not a duplicate of an existing one - [X] I will fill this out to the best of my ability ### Expected Behaviour PowerTools's charge mode override is respected. ### Actual Behaviour Charge mode is set to Normal until the next PowerTools config update (resume from sleep, slider change, etc.) ### Steps To Reproduce 1. Set charge mode to anything except `normal` in PowerTools 2. Plug Deck into charger 3. Watch Deck charge 4. Update PowerTools config and watch is go back to whatever behaviour PowerTools is set for ### Anything else? Cool project, wish I'd come up with something like this! But seriously: research is required to figure out how to capture charging events. Ideally without having to hook into the Steam client ones because that's too error prone and it's also horrible design. ### Version 1.1.0 (Latest stable) ### Platform Steam Deck ### OS _

NGnius commented

2023-03-26 15:52:29 +01:00

(Migrated from github.com)

This should be fixed in f42efab

This should be fixed in [f42efab](https://github.com/NGnius/PowerTools/commit/f42efab0b005a1d389d9089d6d2cd1f7bfa27050)