Oregon Scientific THGR810 and unwise sanity validation about rtl_433 HOT 4 CLOSED

Those check were a whole batch of fixes with #2086.

We need to establish:

• Does THN132N possibly report temperature > 70, < -50?
• Does THGR81, THGR810a possibly report temperature > 70, < -50?
• Does THGR81, THGR810a possibly report humidity < 0, humidity > 99? (99 already confirmed now)
• Does WGR800 WGR800a possibly report gust windspeed < 0, > 56,| avg windspeed < 0, > 56
• Does WGR800 WGR800a possibly report quadrant < 0, > 337.5 (e.g. as error codes)

from rtl_433.

You're not going to see temperatures on planet earth of 70 deg. You might get this temperature if you boiled the thermometer but I think it won't send anymore... :-)

This is a slippery slope of validating data. You might find, for example, that another THGR810 cannot send more than 98% humidity. Why? Because the manufacturer only rates it as the range 5% to 95% with a guaranteed error rate +/-. Once it exceeds these limits, its own display consoles only display HH or LL.

It is totally up to yourselves what to do in these circumstance, but my suggestion would be, if you really want to validate the sensors, then use the DATA_COND state to just not send the "bad" reading, rather than disabling the entire sensor.

WGR800

I still have that sensor on a pole, and I know from past usage it never exceeded around 30 m/s. Ever. I have validation software running off an Arduino that reports these if they occur. It has been used for years.
But, is it possible? Who knows? A bad sensor, perhaps? I do know it has sent wrong readings.

Does WGR800 WGR800a possibly report quadrant < 0, > 337.5 (e.g. as error codes)

Looking at my Arduino code, it's byte [4 ] >> 4, so no, it's only ever going to send 0 to 15.

I think validation should be left to the sink of this data. Problem solved.

from rtl_433.

Yes. As a general policy rtl_433 tries to never limit the values with checks unless it's really needed to verify transmission integrity. We should get rid of those checks unless we get false positive otherwise.

Extreme values might be a valueable error code or give more insight to the sensor state than just valid readings.

from rtl_433.

Great job, thank you for that!

I agree, extreme values might give an insight into the sensor state. More often than not, in my experience, they often just fail outright; especially these cheap sensors. YMMV.

But, once again, thank you for your work!

from rtl_433.