Slimbook-Team / slimbookamdcontroller

GNU General Public License v3.0
35 stars 8 forks source link

Slimbook hardware shipping with incomplete config #14

Closed BlueNalgene closed 1 year ago

BlueNalgene commented 2 years ago

Vitals

Problem

Default config in ~/.config/slimbookamdcontroller/slimbookamdcontroller.conf was incomplete on new installation. It read:

[CONFIGURATION]
mode = medium
autostart = on
show-icon = on
cpu-parameters = 10000-10000-10000-15000-15000-25000-60000-65000-70000

which does not match https://github.com/slimbook/slimbookamdcontroller/blob/02f673770b734808aab93921ae5f01712918da0a/src/configuration/slimbookamdcontroller.conf . This causes the application to open requesting user input on settings, yet it does not save them. Output in terminal looks like:

/usr/share/slimbookamdcontroller/src
Loading configuration:

- Autostart enabled
- Indicator enabled
- Medium
Found thermal zone!
en
Setting cpu TDP values
Could not find your proc in .conf
[redacted]/.config/slimbookamdcontroller/slimbookamdcontroller.conf
Ryzen-9-5900HX
Creates values.
CPU-Parameters: 10000-10000-10000/15000-15000-25000/70000-80000-100000/ 35/45 w
CPU not added

Solution

Manually replacing the default config with the one on this git appears to have fixed it.

Notes

This is nearly the same problem as https://github.com/slimbook/slimbookrgbkeyboard/issues/5#issue-1197628177 with an incomplete or incorrect config file installation causing problems with the programs. Y'all may want to check that your install folks have the correct files on hand.

slimbook commented 2 years ago

Thanks for reporting this bug, we will update a soon as possible.

slimbook commented 1 year ago

Version 0.3.8 fixed this.