PlusPlus-ua / ha_mobilealerts

Moble-Alerts integration for Home Assistant
MIT License
18 stars 13 forks source link

Rain Sensor - Entity "Last hour rain" duplicate #14

Open janoschbatschi opened 1 year ago

janoschbatschi commented 1 year ago

First of all, thanks for your work! I was looking for that integration for a long time.

There seems to be an issue with the rain sensor. There are two entries for "last hour rain" but there is no one for "last day rain". I didn't have the chance to check the values - could be a naming issue and the second "last hour rain" has the data for "last day rain".

image
janoschbatschi commented 1 year ago

Obviously, there is not only a naming issue, but there is also no data on last hour rain or last day rain. Today we had rain twice and there was no data reported (the app reported the data and also the accumulated rain data was updated - compare screenshot)

image
hiatsu0 commented 1 year ago

I have wondered this as well, and I recall reading somewhere that there was some naming issue, hopefully fixed in an upcoming version.

But indeed the functionality of these entries might have some issues as well: I recall seeing just once some data in the "Last rain" entity, and even that disappeared shortly after the actual rain (maybe an hour or so). I think something like "Last rain" should have a little longer timespan..maybe like a few days (or even no time limit at all, and the date & time of the last detected rainfall as an attribute, in addition to the millimeters)