dbrgn/sht2x-rs

WIP Platform agnostic Rust driver for the Sensirion SHT2x temperature/humidity sensors.

sht2x-rs's Issues

TODO list prior to uploading to crates.io?

I think the minimum necessary for releasing to crates.io is probably to fix the temp type (#2) and add humidity measurement. I've got working code for that which returns an i32 for the RH value in thousands of a %RH (which I believe can go negative due to measurement errors, hence the i32).

I could also add an example based on the stm32f103 testrig which I'm using, which would probably be useful.

There are a few other possible features, such as reading the device serial number and implementing (optional) CRC checking which would probably be useful, as well as reset.

Also, a number of other devices have been created by other manufacturers which implement the same protocol as the Sensirion SHT2x, such as the Silicon Labs Si7021 & the TE (previously Measurement Specialties) HTU21D, so it would be useful if this at least showed up in crates.io searches (by adding it to the description) but alternatively the name could reflect this I suppose?

Not usable since published without a suitable license

Hello,

I'd like to use this code (and carry on development of it - I have a number of SHT21 which I'd like to use in a couple of projects), but there's no license currently. According to https://choosealicense.com/no-permission/ if there's no license, then:

"Although a code host such as GitHub may allow you to view and fork the code, this does not imply that you are permitted to use, modify, or share the software for any purpose."

I wonder if you'd mind licensing it? The information at https://choosealicense.com/ might be useful.

FWIW, the rust embedded-hal is:

Licensed under either of

Apache License, Version 2.0 (LICENSE-APACHE or http://www.apache.org/licenses/LICENSE-2.0)
MIT license (LICENSE-MIT or http://opensource.org/licenses/MIT)

at your option.

Thanks!

Tim.

confirmation - working with esp_idf_hal

👋🏽

Just wanted to say thanks for building this.
It works as advertised on a SHT20 sensor running on the ESP32c3-12F

i16 temperature (milli-degrees celsius) return type insufficient for device operating range

The temperature method currently returns an i16 (which is documented as being milli-degrees Celsius), so will overflow at temperatures over 32.767°C, or below -32.768°C. The SHT21 operating range is -40°C to +125°C

Possible solutions would be to reduce the number of decimal places (e.g. to 2) or use an i32.