bpennypacker / WeatherFlow-Indigo-Plugin

Indigo Plugin for obtaining weather from WeatherFlow
GNU General Public License v2.0
3 stars 2 forks source link

Tempest Rain Accumulated Fails for those outside US/Canada #42

Open Ghawken opened 2 years ago

Ghawken commented 2 years ago

Hi

Well done on the Python3 version. All going well.

Yet - for sometime have been troubled with null/empty daily rainfall totals for Websocket Tempest

Had a look and stems from non-US location and likely affects all the sky devices as well.

Tempest (type="obs_st")
Observation Layout
0 - Epoch (Seconds UTC)
1 - Wind Lull (m/s)
2 - Wind Avg (m/s)
3 - Wind Gust (m/s)
4 - Wind Direction (degrees)
5 - Wind Sample Interval (seconds)
6 - Pressure (MB)
7 - Air Temperature (C)
8 - Relative Humidity (%)
9 - Illuminance (lux)
10 - UV (index)
11 - Solar Radiation (W/m^2)
12 - Rain Accumulation (mm)
13 - Precipitation Type (0 = none, 1 = rain, 2 = hail, 3 = rain + hail (experimental))
14 - Average Strike Distance (km)
15 - Strike Count
16 - Battery (volts)
17 - Report Interval (minutes)
18 - Local Day Rain Accumulation (mm)
19 - [NC Rain](https://help.weatherflow.com/hc/en-us/articles/360024436634) Accumulation (mm)
20 - Local Day [NC Rain](https://help.weatherflow.com/hc/en-us/articles/360024436634) Accumulation (mm)
21 - Precipitation Aanalysis Type (0 = none, 1 = Rain Check with user display on, 2 = Rain Check with user display off)

Index 19 and 20 - currently:

    'report_interval':                   17,
    'rain_accumulated_final':            19,

in obs_tempest_map are the problematic ones.

The [NC Rain] is Neighborhood level rain - only available in US and Canada.

Changing the obs_tempest_map with rain_accumulated_final to index 18 (non NC Rain version) for those of us outside US/Canada fixes the rain_accumulation reporting in my testing.

Thanks

bpennypacker commented 2 years ago

Thanks for the detailed information on this. I'm currently in the middle of preparing to sell our house and move to a new one, so the next month or two is going to be really crazy. I'll try to deal with this when I can, but no promises when I'll be able to get around to it.

Ghawken commented 2 years ago

No problem - stressful times! It’s a very easy fix for anyone reading this - update the rain_accumulated to the non-NCS one (==18)