driftkingtw / raspberry-pi-pwm-fan-control

Raspberry Pi PWM fan contorl (for Notcua PWM fans)

raspberry-pi-pwm-fan-control's Issues

fan_control.py speed step calculation

In python2 integer divided by another integer results integer.
Fix:
step = float(FAN_HIGH - FAN_LOW)/float(MAX_TEMP - MIN_TEMP)
setFanSpeed(FAN_LOW + round( temp * step ))

Script eats a lot of CPU

not sure why, but running this script with python2 oder python3 shows me about 13%-21% CPU ussage in htop no matter what refreshrate i configure.

can someone confirm this bug?

Consider adding a license

Please consider adding a license.

ansible-deploy script

If you are interested I have created a ansible role to deploy this script as systemd service on an Ubuntu 20.04. I can publish the role and you can refer to it in your documentation.

Question: FAN_MAX & MIN_TEMP <-> OFF_TEMP

Hi there,

why is the constant FAN_MAX to be defined? It is not used in the code or do I miss something?

I have anther question regarding the MIN_TMEP and OFF_TEMP. What is the reason to define the OFF_TEMP constant? There is no case made if the temperature is between OFF_TEMP and MIN_TEMP which is 40 / 45 at default. Wouldnt it make sense to not use OO_TEMP and just stop the fan if the temp is below MIN_TEMP?

Thanks and best regards
Kai

How can I read speed from my Argon fan?

Hi! I hope you're doing okay! I'm working on my Raspberry Pi project and I came across your repository. I have an Argon mini fan (link) and I just can figure out why read_fan_speed.py comes back 0 every time, even when I hear the fan spinning. Could you help me out find a way to make my Raspberry Pi 4 (running Dietpi) read and display this fan's speed?

[Bug] Ctrl+C does not set fan to High

Raspberry-Pi-PWM-Fan-Control/fan_control.py

Line 82 in 79e1917

setFanSpeed(FAN_HIGH)

As the title says, when testing the script, when I kill the script with Ctrl+C, the fan shuts off instead of getting set to high. Can you adjust this to instead use its own variable and make it so if the script is killed by any means it'll get set to this new variable?