Skip to content

Fixed bug setting auto mode incorrectly - #261

Open
martinchoto420 wants to merge 1 commit into
PXDiv:mainfrom
martinchoto420:main
Open

Fixed bug setting auto mode incorrectly#261
martinchoto420 wants to merge 1 commit into
PXDiv:mainfrom
martinchoto420:main

Conversation

@martinchoto420

Copy link
Copy Markdown

So on my machine and 2 more users using CachyOS since the new kernel update 7.2, for some reason setting the fans to auto both at the same time it didn't work correctly. Sometimes it works the first time, sometimes it doesn't. Also when 1 of the fans is on auto it says correctly but the fan still spins not on auto speed. Using the Linuwu-Sense documentation this command works flawlessly echo "0,0" | sudo tee /sys/module/linuwu_sense/drivers/platform:acer-wmi/acer-wmi/nitro_sense/fan_speed but the daemon sets it incorrectly.

This is the latest build 1.0.2-h1 logs

2026-08-28 17:28:42,956 - DAMXDaemon - INFO - Processing command: set_thermal_profile with params: {'profile': 'performance'}
2026-08-28 17:28:43,959 - DAMXDaemon - INFO - Processing command: get_all_settings with params: {}
2026-08-28 17:28:44,827 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 100, 'gpu': 100}
2026-08-28 17:28:48,407 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 50, 'gpu': 50}
2026-08-28 17:28:49,690 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 50}
2026-08-28 17:28:55,226 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 40, 'gpu': 40}
2026-08-28 17:28:58,127 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:28:58,127 - DAMXDaemon - ERROR - Failed to write to /sys/module/linuwu_sense/drivers/platform:acer-wmi/acer-wmi/nitro_sense/fan_speed: [Errno 22] Invalid argument
2026-08-28 17:29:04,252 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:29:04,253 - DAMXDaemon - ERROR - Failed to write to /sys/module/linuwu_sense/drivers/platform:acer-wmi/acer-wmi/nitro_sense/fan_speed: [Errno 22] Invalid argument
2026-08-28 17:29:05,478 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:29:05,478 - DAMXDaemon - ERROR - Failed to write to /sys/module/linuwu_sense/drivers/platform:acer-wmi/acer-wmi/nitro_sense/fan_speed: [Errno 22] Invalid argument

This is after this fix

2026-08-28 17:31:35,562 - DAMXDaemon - INFO - Processing command: get_supported_features with params: {}
2026-08-28 17:31:35,562 - DAMXDaemon - INFO - Processing command: set_thermal_profile with params: {'profile': 'quiet'}
2026-08-28 17:31:35,569 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:31:36,575 - DAMXDaemon - INFO - Processing command: get_all_settings with params: {}
2026-08-28 17:31:40,248 - DAMXDaemon - INFO - Processing command: set_thermal_profile with params: {'profile': 'performance'}
2026-08-28 17:31:41,249 - DAMXDaemon - INFO - Processing command: get_all_settings with params: {}
2026-08-28 17:31:43,573 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 100, 'gpu': 100}
2026-08-28 17:31:49,956 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 50, 'gpu': 50}
2026-08-28 17:31:51,470 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 50, 'gpu': 0}
2026-08-28 17:31:53,953 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:31:55,451 - DAMXDaemon - INFO - Processing command: set_fan_speed with params: {'cpu': 0, 'gpu': 0}
2026-08-28 17:31:55,457 - DAMXDaemon - INFO - Processing command: get_all_settings with params: {}

I have tested it with every other option in the app and it works but before merging I suggest looking at it more deeply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant