search

bruhautomation / ESP-MQTT-JSON-Multisensor

(OBSOLETE) ESP MQTT JSON Multisensor for Home Assistant. Supported sensors include the TEMT6000 light, AM312 PIR, DHT22 temperature/humidity sensors. RGB led supports flash, fade, and transition. Over-The-Air (OTA) uploading, too!
https://youtu.be/jpjfVc-9IrQ
Apache License 2.0
340 stars 152 forks source link

Change to HeatIndex function using built in DHT library plus set C or F #47

Closed cbrherms closed 5 years ago

cbrherms commented 6 years ago

Now using the built in dht.computerHeatIndex function as found in the arduino DHT library.

This now allows celsius to work with HeatIndex (did not previously) so added definition to make setting fahrenheit or celsius simple, incorperating @rdbahm idea in to my own fix. Fixes #28

Added note and fixed a few typos

rdbahm commented 6 years ago

I think this is a better way to implement the change than my way to do it - puts the logic in the upstream library and shrinks the size and complexity of the multisensor's code.

theetron commented 6 years ago

Thanks for this addition.

bruhautomation commented 5 years ago

Many apologies for the delay getting back to these repos! I've migrated away from this codebase after the esphomeyaml project was announced. I plan on marking these repo's deprecated soon because that project is vastly more capable.