Support for Sensirion STS3X
slimcdk opened this issue · 8 comments
Device name
STS3X
Device description
Device is a digital temperature sensor. It communicates using I2C.
Device product page URL
https://www.sensirion.com/search/products?q=STS3x
Datasheet URL
Can the device be purchased at any of the followings?
- AliExpress
- Amazon
- ebay
- Digi-Key
- Mouser
URLs to purchase the device
Other implementations
No response
Thank you!
Hmm... Did you check your PR on real device?
The datasheet says that the raw data size is 3 bytes. And in your code, the raw data size is 6 bytes.
#define STS3X_RAW_DATA_SIZE 6
typedef uint8_t sts3x_raw_data_t[STS3X_RAW_DATA_SIZE];I'm rolling back the merge of your PR until I get confirmation that the driver works with the actual device.
Sorry for the radio silence! Still have a bit of vacation left, so I'll explain what I have currently tested. You are right, that the size is only 3 bytes due to the lack of humidity compared to the SHT (which this implementation is an exact copy off). I have tested the driver with an ESP32-S3 and STS31 at 100 MHz and didn't encounter any issues for the week I have been using it.
I'll adjust the size allocation next week and test before submitting a new PR.
Had some conflicts with the files were added and then removed from upstream, so I couldn't figure out to sync properly, so I made a new branch.
Driver is tested on real hardware and is reporting correct measurements.
Thank you.
A new branch and a new PR is exactly what is required.