Skip to content
This repository has been archived by the owner on Dec 8, 2019. It is now read-only.

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

Merged
merged 1 commit into from
Jan 11, 2019

Conversation

cbrherms
Copy link
Contributor

@cbrherms cbrherms commented Nov 27, 2017

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

Now using dht.computerHeatIndex function, definition for fahrenheit or celsius, plus notes and typo fixes
@rdbahm
Copy link

rdbahm commented Nov 29, 2017

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
Copy link

Thanks for this addition.

@bruhautomation
Copy link
Owner

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.

@bruhautomation bruhautomation merged commit 9d60362 into bruhautomation:master Jan 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Temp in C not F?
4 participants