-
-
Notifications
You must be signed in to change notification settings - Fork 29.9k
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
WeatherFlow integration: Incorrect values for vapor pressure(inHg) & wind speed (mph) #101681
Comments
Hey there @natekspencer, @jeeftor, mind taking a look at this issue as it has been labeled with an integration ( Code owner commandsCode owners of
(message by CodeOwnersMention) weatherflow documentation |
if you modify your config file and turn on debugging for |
I added the following in my configuration.yaml file and restarted Home Assistant. Is that what you had in mind? Under Settings > System > Logs, I do not see any log entries mentioning weatherflow. I repeated with "pyweatherflowudp: info" in configuration.yaml and still don't see any log entries mentioning weatherflow. Am I doing the wrong thing or looking in the wrong place? |
FYI, The wind speed values were corrected in 2023.10.1. There was an incorrect native unit assigned to them. I've not seen an issue with vapor pressure values though |
Thanks for the wind speed correction--and for the WeatherFlow integration. Great job! Thanks! |
A factor of 100 vapor pressure error could occur if source units were switched between kPa and bar. |
I was able to find the issue in the library. Thanks! |
👏🏼 |
The problem
The new WeatherFlow integration in Home Assistant core has a couple of problems possibly involving unit conversions.
(1) Vapor pressure is currently reporting values 50.75752 inHg, much larger than the reported air pressure of 29.84979 inHg. It is physically impossible for vapor pressure to exceed atmospheric air pressure. (Other related values reported to Home Assistant are: Dew point = 59.2 deg F; Humidity = 77.19%)
(2) Wind speed, gusts, and lulls reported to Home Assistant in miles/hour (mph) are lower than those reported in the Tempest App (also in mph) by a factor of approximately 3.6.
What version of Home Assistant Core has the issue?
core-2023.10.0
What was the last working version of Home Assistant Core?
none. WeatherFlow integration is new with version 2023.10
What type of installation are you running?
Home Assistant OS
Integration causing the issue
WeatherFlow
Link to integration documentation on our website
https://www.home-assistant.io/integrations/weatherflow/
Diagnostics information
No error messages observed
Example YAML snippet
No response
Anything in the logs that might be useful for us?
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: