-
Notifications
You must be signed in to change notification settings - Fork 28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
visibility unit shows nmi? #235
Comments
That probably was a home assistant change. When I get a chance, I will change the units km and mi from nautical mile. It may actually be in miles but did not get the units updated. I know I based the calculation off a NOAA survey of visibility compared to humidity and air pressure (I did not utilize the air pressure portion); then later I took into account station elevation for visibility. |
Awesome, thanks man, I don't see a problem with nautical miles myself and yeah it might actually be in miles but I guess HA doesn't like it haha. |
How's it going Glenn? Just wondering if there is anything else I can do to help. :) |
It is returning the value at km or mi; I missed updating the imperial unit from nautical mile to mile. The value is miles thou. I should have a PR to fix it soon. |
Awesome.
…On Fri, Sep 8, 2023 at 3:59 PM Glenn ***@***.***> wrote:
It is returning the value at km or mi; I missed updating the imperial unit
from nautical mile to mile. The value is miles thou. I should have a PR to
fix it soon.
—
Reply to this email directly, view it on GitHub
<#235 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/A47NQ2BABEJ55PF3PV623B3XZN2JBANCNFSM6AAAAAA4DTTZVQ>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
PR is in |
Home Assistant Version?
2023.8.4
Add-On or Self Hosted?
HA Supervised (Add-On)
Weatherflow2MQTT version?
3.2.0
What happened?
I am getting an error in my log that I don't think I saw before the update to 3.2.0 of the addon. I cannot find a way to change the unit of measurement for visibility myself or if I should even need to.
Relevant log output
The text was updated successfully, but these errors were encountered: