Skip to content
This repository has been archived by the owner on Aug 23, 2023. It is now read-only.

Battery charge limit settings not persisting (also resets when running a game) #109

Open
2 of 5 tasks
Speedy-VonGofast opened this issue Jun 7, 2023 · 3 comments
Open
2 of 5 tasks
Labels
enhancement New feature or request

Comments

@Speedy-VonGofast
Copy link

Please confirm

  • I have searched existing issues and found no duplicate
  • I will fill this out to the best of my ability

Extra details

  • I am using a custom pt_oc.json file
  • I am using a custom limits_cache.json file
  • I have submitted a log through the PowerTools UI

Expected Behaviour

Enabling persist settings should keep enabled the battery charge and the value slide to the previous state when restarting the Steam Deck or after launching a game.

Actual Behaviour

The battery charge limit resets to disabled when restarting the Steam Deck or when starting a game.

Steps To Reproduce

  1. Enable persist settings
  2. Enable battery charge limit
  3. Restart the Steam Deck

Anything else?

I think this kind of settings should be global and should not revert back to default settings when starting a game.

I was expecting this setting to always persist between games as this doesn't affect the games performances but only for the battery health.

Version

1.3.1

Platform

SteamDeck

OS

SteamOS 3 (Stable)

@Speedy-VonGofast Speedy-VonGofast added the bug Something isn't working as intended label Jun 7, 2023
@NGnius
Copy link
Owner

NGnius commented Jun 10, 2023

This is working as intended, so I'm closing this. All settings are supposed to be tied to the game profile.

If you'd like me to reopen it and switch the label to feature request, let me know.

@NGnius NGnius closed this as completed Jun 10, 2023
@Speedy-VonGofast
Copy link
Author

Speedy-VonGofast commented Jun 10, 2023 via email

@NGnius NGnius added enhancement New feature or request and removed bug Something isn't working as intended labels Jun 10, 2023
@NGnius NGnius reopened this Jun 10, 2023
@NGnius
Copy link
Owner

NGnius commented Jun 10, 2023

Done.

I'll address the challenges with a change like this, to see if anyone has a solution to them:

Currently, everything is associated with a game profile, which makes the persistent toggle reasonably intuitive to use. Changing one setting to not follow that behaviour is confusing and not immediately obvious. I'd like to not explain that behaviour with text in the UI since many setting descriptions are already long. We need a way that'll allow the user to easily guess the correct answer to

  • How does the battery limit setting behave when I start/finish a game?
  • How does the persistent profile toggle change how the battery limit setting is saved (or not saved)?

There are also technical challenges, since the whole PowerTools codebase is built around the assumption that anything persistable is part of a game profile. But I don't mind doing a reasonably large refactor to accommodate a different way of doing things if it makes sense.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants