Pirate-Weather / pirateweather

Code and documentation for the Pirate Weather API
Apache License 2.0
634 stars 29 forks source link

Pressure values innacurate by significant amount #269

Closed speedbird182 closed 2 weeks ago

speedbird182 commented 1 month ago

Describe the bug

The Pressure Sensor values are incorrect by a significant amount (~20 hPa).

Expected behavior

This shows the pressure as taken from Pirate Weather: Screenshot 2024-07-18 at 15-57-56 Overview – Home Assistant

Where as this is the actual current pressure from Met.No and the UK Met Office: Screenshot 2024-07-18 at 15-58-27 Overview – Home Assistant

Actual behavior

Current pressure should match (or be very close to) actual readings from the UK met office.

API Endpoint

Production

Location

Liskeard, Cornwall

Other details

No response

Troubleshooting steps

cloneofghosts commented 1 month ago

Thanks for the report. What's happening here is the pressure values displayed in Pirate Weather is surface-level pressure whereas most sites use sea-level pressure for their pressure values which is what Dark Sky used for pressure values as reported in #265

Will ping @alexander0042 to look into this but it should just be a case of switching the data to use sea-level pressure instead of surface-level pressure.

alexander0042 commented 1 month ago

This is a great catch, and I can't believe I've never noticed it before! I'll get pressure switched over to MSLP in the next update, which should fix this problem

cloneofghosts commented 2 weeks ago

This has been fixed in V2.1.1 of the API which released today.