adafruit / adafruit_si7021 Goto Github PK

...\Arduino\libraries\Adafruit_Si7021_Library\Adafruit_Si7021.cpp: In member function 'float Adafruit_Si7021::readHumidity()':
...\Arduino\libraries\Adafruit_Si7021_Library\Adafruit_Si7021.cpp:88:15: warning: unused variable 'chxsum' [-Wunused-variable]
uint8_t chxsum = _wire->read();
^
...\Arduino\libraries\Adafruit_Si7021_Library\Adafruit_Si7021.cpp: In member function 'float Adafruit_Si7021::readTemperature()':
...\Arduino\libraries\Adafruit_Si7021_Library\Adafruit_Si7021.cpp:121:15: warning: unused variable 'chxsum' [-Wunused-variable]
uint8_t chxsum = _wire->read();

https://github.com/adafruit/Adafruit_BusIO

I would like to unify this with bme280
like so
/*!
* @brief Sets up the HW by reseting It, reading serial number and reading
* revision.
* @return Returns true if set up is successful.
*/
bool Adafruit_Si7021::begin(uint8_t addr, TwoWire *theWire) {
delete i2c_dev;
i2c_dev = new Adafruit_I2CDevice(addr, theWire);
if (!i2c_dev->begin())
return false;
return init();
}
`/*!` `* @brief Initialise sensor with given parameters / settings` `* @returns true on success, false otherwise` `*/` `bool Adafruit_Si7021::init() {`
if (_readRegister8(SI7021_READRHT_REG_CMD) != 0x3A)
_writeRegister8(SI7021_WRITERHT_REG_CMD, 0x3A);
if (_readRegister8(SI7021_READRHT_REG_CMD) != 0x3A) return false;
readSerialNumber();
_readRevision();
``
return true;
`}`

and this check
if (_readRegister8(SI7021_READRHT_REG_CMD) != 0x3A)
why?
Copyright?

Lines 166 and 175 seem to be copied. Line 175 should read

sernum_b = uint32_t(buffer[0]) << 24 | uint32_t(buffer[1]) << 16 |

Otherwise sernum_a is overwritten with both parts of the serial number, and sernum_b remains 0.

I use multi I2C sensors board (CCS811 + SI7021 + BMP280)
(like this https://arduino.ua/prod3768-modyl-datchikov-kachestva-vozdyha-ccs811--si7021--bmp280)

First I'm try using Marcus Sorensen [email protected] library, but it is not work stable for my sensors board.

Secondary I try to using this Adafruit_Si7021 it is work properly for I2C address 0x40, only after I re-code begin() method like this

`
bool Adafruit_Si7021::begin() {
_wire->begin();

_wire->beginTransmission(_i2caddr);
_wire->endTransmission();

reset();

_readRegister8(SI7021_READRHT_REG_CMD);

readSerialNumber();
_readRevision();

return true;
}
`
I know it is all wrong way! But like temporary solution it is possible to use and I hope it is helps to developers.

Additional:
After some loops counts the sensor is lost bus... so if temperature or humidity values is NAN, just call begin() method.

like this
float temperature = sensor.readTemperature(); if (temperature == NAN) { Serial.println("Si7120 reset"); sensor.begin(); float temperature = sensor.readTemperature(); } Serial.print("Humidity: "); Serial.print(sensor.readHumidity(), 2); Serial.print("\tTemperature: "); Serial.println(temperature, 2);

Im just using latest library as it is with ESP8266. What i observed is humidity is crossing 100 value and is being reported as 101 and above.

Its currently raining outside but I guess RH value is never beyond 100.
Please help.

• Arduino board: Arduino Mega

• Arduino IDE version (found in Arduino -> About Arduino menu): 1.8.19

• List the steps to reproduce the problem below (if possible attach a sketch or
  a_serre_logger.zip

  copy the sketch code in too): LIST REPRO STEPS BELOW
  [[email protected]]
  [[email protected]]
  [[email protected]]

[[email protected]] / [[email protected]]

all other libraries are in the zip file.

Polling the sensor(humidity or temperature) eats 8 bytes of RAM each poll that doesn't get released each loop.
This will cause the program to crash in a few minutes (out of RAM)

Using Adafruit version (of the Si7021) 1.4.0 fixes the RAM leak problem, the problem exist with version 1.5.0 and 1.5.1.

code also in zip, as it is unreadable as text here ( brackets don't seem to work very well with my code?)

Is there a python library for this, if now how would i make one?

I am using ESP-12S and struggling with this library, as its not getting detected.
I want to change the i2c pins as the default pins for the esp-12s board have different pins assigned.