Skip to content

Conversation

@kattni
Copy link
Contributor

@kattni kattni commented Aug 13, 2018

Added per #11

Added dew point calculation using Magnus formula. Checked against online dew point calculator, matches. Updated example to include dew point reading.

@kattni kattni requested review from ladyada and tannewt August 13, 2018 15:42
@ladyada
Copy link
Member

ladyada commented Aug 13, 2018

looks good to me, just need a linty and ill merge!

@kattni
Copy link
Contributor Author

kattni commented Aug 13, 2018

I pylint: disabled it because that's the constant used in the formula.

@ladyada ladyada merged commit ac1bc89 into adafruit:master Aug 13, 2018
tannewt pushed a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Aug 14, 2018
@tannewt
Copy link
Member

tannewt commented Aug 14, 2018

Sorry I didn't get to this sooner. I don't think it should go in this library because its not specific to the BME280 at all. Please remove it so people don't start using it.

Instead, a generic Python library like meteocalc should be used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants