ruuvi / com.ruuvi.station

BSD 3-Clause "New" or "Revised" License
72 stars 39 forks source link

2.7 Sensor is showing 0C although last measurement 6 days ago shows -19.88C #1204

Open markoaamunkajo opened 7 months ago

markoaamunkajo commented 7 months ago

Description:

This sensor has some issue and it's being investigated on Cloud side. https://github.com/ruuvi/ruuvi.cloud.js/issues/430

Last measurement is from 6 days ago.

Sensor card showing 0C when last measurement was -19.88. User should see all last measured sensor values. History view shows correct last measurement for temp.

Mökki Kitchen_20240215T111403+0700.csv

markoaamunkajo commented 7 months ago

More details to this

The owner sees

Sharee sees

andreevdenis commented 7 months ago

Currently the app is not supporting nullable temperature for the sensor. Supporting it is tricky and time consuming because in almost all of the screens and classes where we display and operate temperature we expect it to be not null. So we need to code behaviour for null temperature cases in many many places.

andreevdenis commented 7 months ago

For now we can skip measurements that come with null temperature to avoid showing wrong value (0 C). Or postpone the changes. @markoaamunkajo

laurijamsa commented 7 months ago

Let's postpone for now I would say. But we may need to fix this properly later.

I think not showing measurements that may not be correct somehow can just make it harder to spot issues or bugs in the future.

laurijamsa commented 4 weeks ago

I assume the issue https://github.com/ruuvi/com.ruuvi.station/issues/1289 requires this discussed null support and if so, it will allow to fix also this issue.