home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
70.47k stars 29.4k forks source link

Airthings BLE pressure differs significantly from other pressure sensors, including itself #110866

Closed jakllsch closed 5 months ago

jakllsch commented 6 months ago

The problem

The airthings_ble integration displays a significantly different pressure reading than the airthings cloud integration does for data coming from the same Airthings Wave Plus device. The airthings integration reading matches what the Airthings Android app shows, as well as what the Home Assistant Companion reading from my Android phone (generally in the same general vicinity) shows.

It's also annoying that both hPa and mbar are being used as units when they are identical, yet are being graphed separately. Is mbar supposed to be Mean Sea Level Pressure and hPa supposed to be locally-apparent pressure? If so, the airthings integration should be switched to hPa or should convert to MSLP; if not, everything should be in hPa and airthings_ble shouldn't try to convert to MSLP in mbar.

image

My altitude is about 1000ft/300m, which seems to match the difference shown in these graphs.

What version of Home Assistant Core has the issue?

core-2024.2.2

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

airthings_ble

Link to integration documentation on our website

No response

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

home-assistant[bot] commented 6 months ago

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

Code owner commands Code owners of `airthings_ble` 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 airthings_ble` Removes the current integration label and assignees on the issue, add the integration domain after the command. - `@home-assistant add-label needs-more-information` Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue. - `@home-assistant remove-label needs-more-information` Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


airthings_ble documentation airthings_ble source (message by IssueLinks)

LaStrada commented 6 months ago

Hi, thanks for reporting this. This will be fixed here:

https://github.com/Airthings/airthings-ble/pull/32

def1149 commented 5 months ago

This gets fixed in two days while the bogus battery level reporting has been ongoing for months!!? My AirThings Plus has been running fine for a week with 0% battery remaining!

LaStrada commented 5 months ago

This gets fixed in two days while the bogus battery level reporting has been ongoing for months!!?

My AirThings Plus has been running fine for a week with 0% battery remaining!

The battery issue was fixed a while ago, but not released yet. This issue was fixed before it was reported here, but also not released yet. There will be a bigger release soon with all the fixes, cleanup etc. We need more testing before we can release since it changes a lot code.

joostlek commented 5 months ago

@def1149 please stop bringing up your issue everywhere. This is not relevant to this issue.

Some issues are difficult, some are unsolvable and that's something we have to deal with. All that we ask from you now is a bit of patience. Thanks.