Add fan speed sensor - Githubissues

Sennevds / system_sensors

Logging of system sensor specific for the RPI and sending them to a MQTT broker

MIT License

361 stars 112 forks source link

Add fan speed sensor #178

Closed blacknell closed 2 months ago

blacknell commented 2 months ago

Now that the Raspberry Pi 5 comes with fan enabled it seems obvious we should expose this as a sensor

alump commented 2 months ago

I assume the number it gives in "revolutions per minute", so maybe units could be also defined for 'fan_speed' in sensors.py: 'unit': 'rpm', Anyway used this on my own custom version (with extra i2c sensors), and it worked. I only added the missing units. Thanks for it.

blacknell commented 2 months ago

I assume the number it gives in "revolutions per minute", so maybe units could be also defined for 'fan_speed' in sensors.py: 'unit': 'rpm', Anyway used this on my own custom version (with extra i2c sensors), and it worked. I only added the missing units. Thanks for it.

The only problem is that Home Assistant doesn't have a unit of measurement for rotational speed

alump commented 2 months ago

The only problem is that Home Assistant doesn't have a unit of measurement for rotational speed

I still accepts unknown units, and shows those correctly. Only conversion logic to other units will be missing, right? At least it's happy to show rpm as unit on my entities.