rain accumulation logic breaks when more than one sensor reporting, as they are not tracked separately. Current code might mistake this for rain counter wrap-around.
My vote is to just report the raw value (0.01 or whatever to get the units right). Then clients can do what ever they want. In general I think rtl_433, should hand the RF bits, parse/validate athe framing to know when valid messages are received and then get out of the way.
If you are going to keep state and report deltas, then it should be as accurate and handle the edge cases such as multiple stations being in range, wraparounds, sensor reset, etc.
Please remove the state tracking in this case. I agree with @rct.
Addressed in #893 now. Pending on discussion in #827.
Most helpful comment
My vote is to just report the raw value (0.01 or whatever to get the units right). Then clients can do what ever they want. In general I think rtl_433, should hand the RF bits, parse/validate athe framing to know when valid messages are received and then get out of the way.
If you are going to keep state and report deltas, then it should be as accurate and handle the edge cases such as multiple stations being in range, wraparounds, sensor reset, etc.