search

airgradienthq / arduino

Other
203 stars 108 forks source link

I-9PSL firmware 3.1.4 doesn't recognize SHT3X #203

Open pjcreath opened 1 month ago

pjcreath commented 1 month ago

At power on, the display says:

Sensor init
Error:
SHT not found

The sensor works fine in my older board. I don't know when this regression was introduced.

Debug log ``` ESP-ROM:esp32c3-api1-20210207 Build:Feb 7 2021 rst:0x15 (USB_UART_CHIP_RESET),boot:0xe (SPI_FAST_FLASH_BOOT) Saved PC:0x420184b0 SPIWP:0xee mode:DIO, clock div:1 load:0x3fcd5810,len:0x438 load:0x403cc710,len:0x918 load:0x403ce710,len:0x25f4 entry 0x403cc710 Serial nr: xxxxxxxxxxxx [Configure] Info: Reading file… [Configure] Info: Reading file: success [Configure] Info: Parse file to JSON [Configure] Info: {“country”:“US”,“pmStandard”:“us-aqi”,“ledBarMode”:“co2”,“abcDays”:8,“tvocLearningOffset”:12,“noxLearningOffset”:12,“mqttBrokerUrl”:"",“temperatureUnit”:“f”,“configurationControl”:“both”,“postDataToAirGradient”:true,“ledBarBrightness”:1,“displayBrightness”:1,“offlineMode”:false,“model”:“I-9PSL”} Detected ONE_INDOOR [OledDisplay] Info: begin Firmware Version: 3.1.4 LED bar brightness: 1 Init SGP41 success SHTx sensor not found Set S8 AbcDays successful [ApiClient] Info: begin ... ```
pnt325 commented 1 month ago

@pjcreath Thank for report the issue. we'll check it.

pnt325 commented 1 month ago

@pjcreath I have some code on branch: https://github.com/airgradienthq/arduino/tree/hotfix/sht-sensor-failed to optimize i2c communication that can solve sht3x sensor failed.

pjcreath commented 1 month ago

@pnt325 Do you mean commit e0514e?

Were you able to confirm that this resolved the issue, or do you need me to install the firmware and test? If the latter, where can I download an image?

Why was SGP originally in a separate task? Does this change break ESP32 boards?

pnt325 commented 1 month ago

@pjcreath

Do you mean commit e0514e?

Yes

Were you able to confirm that this resolved the issue, or do you need me to install the firmware and test? If the latter, where can I download an image?

It's optimize I think it's better to reduce the I2C communication error. I still not sure it's solve or not cause I don't have SHT3x sensor to test.

Why was SGP originally in a separate task? Does this change break ESP32 boards?

It's need to run on runtime as second period to calculate the NOx and VOC

pjcreath commented 3 weeks ago

I still not sure it's solve or not cause I don't have SHT3x sensor to test.

Is there an OTA update with this change?