search

nlrb / com.weather-sensors

Wireless weather sensor app voor Athom Homey
https://apps.athom.com/app/com.weather-sensors
MIT License
14 stars 24 forks source link

Oregon PCR800 presents data in wrong format #67

Closed lgorroe closed 4 years ago

lgorroe commented 5 years ago

The Oregon PCR800 (code 2914) sends rain rate ("rainrate") in inches per hour and total rain ("raintotal") in inches. The data from the PCR800 is however presented in Homey without any data conversion as "Rain" in mm and "Rain Meter" in m3. It would be preferable to have a user selection of inches or mm in the device settings, the rain rate presented in mm per hour or inches per hour, and the total rain presented in mm or inches, depending on the user setting.

nlrb commented 5 years ago

You are right! For 2914 it should be converted. Regarding unit selection, you would have to ask Athom. The default capability measure_rain requires the value to be in mm.

lgorroe commented 5 years ago

I would be perfectly fine with a conversion to mm and mm/h :)

Is the presentation as "mm" instead of "mm/h" and "m3" instead of "mm" an Athom issue or an app issue?

nlrb commented 5 years ago

All the app does is report the value. The unit text/presentation is done by Homey FW.

nlrb commented 4 years ago

Instead of dividing rain total by 1000, I will now divide it by 39.37 (and rain rate 3.937).

nlrb commented 4 years ago

Fixed in 2.5.0.