Temperature and humidity sensor support #18910

roman-dvorak · 2021-12-25T11:47:58Z

Hi,
i created a driver for the SHT3x sensor by Senserion. Thanks to this, meteorological conditions can be measured in flight. There are several temperature sensors, but most of them are housed in some packaging and the temperature stabilization takes a very long time or is heated by the surrounding electronics.

Humidity is another important factor in some flights.

Describe problem solved by this pull request
The problem is solved by creating an I2C driver for the sensor and a message (temperature, humidity) with enabled logging. The driver can re-establish the connection and re-initialize the sensor after accidentally disconnecting the sensor.

A new message has been created for sending and logging this data. In the next PR, I would like to create a translation into the MAVLink message.

Test
The functionality was verified with the SHT31 sensor in the TFHT01 board.

dagar · 2021-12-25T15:22:19Z

Did you see #18474?

I was thinking adding this as sensor_hygrometer would be consistent.

roman-dvorak · 2021-12-25T15:39:07Z

I didn't know that. I only knew the 'hygrometer' mavlink message.

Mentioned PR receives data from a UAVCAN source. I would therefore suggest to use the uORB message from PR #18474 (sensor_hygrometer.msg) and fill it with the temperature/humidity data from my SHT3x I2C driver.

Is there any assumption when it will be merged? To just make a rebase.

dagar · 2021-12-25T22:56:53Z

I didn't know that. I only knew the 'hygrometer' mavlink message.

Mentioned PR receives data from a UAVCAN source. I would therefore suggest to use the uORB message from PR #18474 (sensor_hygrometer.msg) and fill it with the temperature/humidity data from my SHT3x I2C driver.

Is there any assumption when it will be merged? To just make a rebase.

It was waiting for the DSDL to move to DroneCAN. We should be able to bring it in now.

dagar · 2021-12-27T15:22:57Z

@roman-dvorak sensor_hygrometer will be merged shortly. #18913

msg/humidity_temperature.msg

src/drivers/atmospheric_quantities/sht3x/sht3x.cpp

src/drivers/drv_sensor.h

src/modules/logger/logged_topics.cpp

dagar · 2021-12-27T15:28:57Z

How do you start this? Do you want to add a new parameter?

PX4-Autopilot/src/drivers/differential_pressure/ms4525/parameters.c

Line 41 in 3f17acd

PARAM_DEFINE_INT32(SENS_EN_MS4525, 0);

PX4-Autopilot/ROMFS/px4fmu_common/init.d/rc.sensors

Lines 146 to 150 in 3f17acd

    
           # TE MS4525 differential pressure sensor external I2C 
        
           if param compare -s SENS_EN_MS4525 1 
        
           then 
        
           	ms4525_airspeed start -X 
        
           fi

boards/px4/fmu-v5/default.px4board

src/drivers/hygrometer/CMakeLists.txt

src/drivers/hygrometer/sht3x/sht3x.cpp

roman-dvorak · 2022-01-07T13:47:46Z

@dagar, can you review again?

mrpollo · 2022-01-26T16:09:06Z

@roman-dvorak please rebase. We can merge as soon as GitHub Actions completes

SHT3x driver, clean code changes leading to merge TFHT with CUAV hygrometer Delete humidity_temperature.msg Update CMakeLists.txt rename 'atmosphefir_quantities' to 'hygrometers' fix logging removed cmake files fix PR issues

roman-dvorak · 2022-01-27T10:54:24Z

@mrpollo @dagar
Hi, I did a rebase and I tested it again with a real sensor that everything works. Thank you!

Can you enable CI tests for this PR