gjr80 / weewx-realtime_gauge-data

Near realtime support for updating of SteelSeries Weather Gauges by WeeWX
GNU General Public License v3.0
9 stars 5 forks source link

Remove option to set windrun units #1

Closed gjr80 closed 7 years ago

gjr80 commented 7 years ago

SteelSeries Weather Gauges derive windrun units from wind speed units whereas with rtgd the user can specify the windrun units independent of wind speed. Similarly, rain rate units are tied to rain units. This could cause a conflict if the user set, say, group_speed=km_per_hour and group_distance=mile or group_rain=mm and group_rainrate=inch_per_hour.

Solution is for rtgd to derive windrun and rainrate units in the same manner as the SteelSeries Weather Gauges. Need to make sure that formatting options are sufficiently flexible though.

gjr80 commented 7 years ago

Fixed at commits https://github.com/gjr80/weewx-realtime_gauge-data/commit/36e5c0a3ee0268d55294980202912b03c3ef9777 and https://github.com/gjr80/weewx-realtime_gauge-data/commit/99fc70fd951965935b2c84d345584719e6a1a47c