Skip to content
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

Closed
stevemha opened this issue Oct 9, 2023 · 8 comments · Fixed by #101770
Closed

Comments

@stevemha
Copy link

stevemha commented Oct 9, 2023

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

@home-assistant
Copy link

home-assistant bot commented Oct 9, 2023

Hey there @natekspencer, @jeeftor, mind taking a look at this issue as it has been labeled with an integration (weatherflow) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of weatherflow can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign weatherflow Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


weatherflow documentation
weatherflow source
(message by IssueLinks)

@jeeftor
Copy link
Contributor

jeeftor commented Oct 9, 2023

if you modify your config file and turn on debugging for pyweatherflowudp we can see the values I believe and can probalby reverse engineer whats going on...

@stevemha
Copy link
Author

I added the following in my configuration.yaml file and restarted Home Assistant.
logger:
logs:
pyweatherflowudp: debug

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?
Thanks.

@natekspencer
Copy link
Contributor

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

@stevemha
Copy link
Author

Thanks for the wind speed correction--and for the WeatherFlow integration. Great job!
I can get along without the vapor pressure data, but for your information my current conditions (71.7 deg F temperature, 55.17% humidity, 54.8 deg F dew point) should lead to a vapor pressure of 1.466 kPa or 0.4329 inches of Hg according to my calculations. The WeatherFlow integration is reporting a vapor pressure 43.25 inches of Hg--a factor of 100 higher than my calculations indicate. In any case, current air pressure is 29.87 inches of Hg, and vapor pressure cannot possibly be higher than the air pressure.

Thanks!

@stevemha
Copy link
Author

A factor of 100 vapor pressure error could occur if source units were switched between kPa and bar.

@natekspencer
Copy link
Contributor

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!

@jeeftor
Copy link
Contributor

jeeftor commented Oct 10, 2023

👏🏼

@github-actions github-actions bot locked and limited conversation to collaborators Nov 9, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants