I read it a little differently.

I think they're asking that the underlying code use plural for altitude 
because it is unlike the altitude would ever be 1 foot/meter regardless of 
whether it's a pip/apt/yum installation.   (it's always 0 or 2 or 123 
feet/meters etc.)

Alternately you could just fix just their garbage-in under the hood in 
configuring the station.   Looks like you could do so around line 167 in 
station_config.py if you were so inclined with just a few lines for 
'foot/feet' and 'meter/meters'.  Folks do seem to mess this one up a lot.

Maybe something like this (untested) ?

    if unit in ['feet']:
       unit = 'foot'
    elif unit in ['meters']:
       unit = 'meter'
    else:
       raise ValueError(f"Unknown altitude unit {unit}")

-- 
You received this message because you are subscribed to the Google Groups 
"weewx-development" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/weewx-development/3ca06e69-b13e-4517-9dda-c4af4403f7e1n%40googlegroups.com.

Reply via email to