Skip to content

Comments

Fan controller task#1357

Merged
WantClue merged 3 commits intobitaxeorg:masterfrom
mutatrum:fan-controller-task
Feb 5, 2026
Merged

Fan controller task#1357
WantClue merged 3 commits intobitaxeorg:masterfrom
mutatrum:fan-controller-task

Conversation

@mutatrum
Copy link
Collaborator

This is a part of #1217 where only the fan controller is extracted. It increased the PID controller frequency from 1.8 seconds to 100ms, which results in a much smoother fan curve. Especially on older devices that uses the EMC2101 internal temp with a 1° C resolution, it's a much more restful soundscape.

I also decreased the PID tuning values quite a bit, but this needs further testing. Maybe the PID parameters need to be exposed under an advanced tab, so people can tune the controller themselves.

@github-actions
Copy link

github-actions bot commented Nov 16, 2025

Test Results

41 tests  ±0   41 ✅ ±0   0s ⏱️ ±0s
 1 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit ae612ff. ± Comparison against base commit b797ea7.

♻️ This comment has been updated with latest results.

@WantClue WantClue added this to the 2.12.0 milestone Nov 17, 2025
@WantClue WantClue added enhancement New feature or request cleanup Code cleanup labels Nov 17, 2025
@0xf0xx0
Copy link
Contributor

0xf0xx0 commented Nov 21, 2025

auto fan control isnt capped at 100, and will slowly over an hour or two go up to 255 before wrapping around and re-starting the fan

image

@WantClue WantClue self-requested a review November 27, 2025 18:17
@WantClue WantClue modified the milestones: 2.12.0, 2.13.0 Nov 29, 2025
@mutatrum
Copy link
Collaborator Author

mutatrum commented Feb 5, 2026

auto fan control isnt capped at 100, and will slowly over an hour or two go up to 255 before wrapping around and re-starting the fan

Apparently the PID max output is a soft limit and it can overshoot, reproduced it by setting the temp to 35C. Added clamping of the PID output.

@WantClue WantClue merged commit bd048ae into bitaxeorg:master Feb 5, 2026
3 checks passed
@mutatrum mutatrum deleted the fan-controller-task branch February 5, 2026 11:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Code cleanup enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants