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
72.4k stars 30.3k forks source link

Govee BLE (5102) Temperature and Humidity Sensor #85580

Closed rockinonthehubnotkansas closed 1 year ago

rockinonthehubnotkansas commented 1 year ago

The problem

Beginning 2022.12.7 the Govee BLE integration will no longer work for temperatures reading below zero (confirmed on integrated display). Temperature and humidity are correct until temperature goes below zero, then both temperature and humidity entities become unavailable. Battery level and signal strength entities are still reported. Temperature and humidity entities recover when temperature goes above zero.

Prior to 2022.12.7 temperatures at low as -11F were correctly reported and displayed in HA.

What version of Home Assistant Core has the issue?

2023.1.2

What was the last working version of Home Assistant Core?

2022.12.6

What type of installation are you running?

Home Assistant OS

Integration causing the issue

Govee BLE

Link to integration documentation on our website

https://www.home-assistant.io/integrations/govee_ble/

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 1 year ago

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

Code owner commands Code owners of `govee_ble` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign govee_ble` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


govee_ble documentation govee_ble source (message by IssueLinks)

PierreAronnax commented 1 year ago

https://github.com/Bluetooth-Devices/govee-ble/blob/432223ada34def7bb58a1601afdfd6954cc3a152/src/govee_ble/parser.py#L32 MIN_TEMP = -17.7778

@bdraco it seems that this needs to be lowered. Assuming we are talking zero F in this issue.

According to the manual the operating range is -20°C – 60°C (-4°F - 140°F) but the actual range is apparently bigger.

bdraco commented 1 year ago

That value was derived from published specs of the device plus some leeway. I guess we need some more leeway since they actually deliver better than the specs and work with lower temps. Happy to merge a PR to lower the value

rockinonthehubnotkansas commented 1 year ago

I agree that the actual performance of the sensor does not match the documentation. I currently have another HA setup running 2022.12.6 and one of my Govee 5102 sensors is regularly hitting below -11F on the freezer cycle. Based on that data I'm not sure what an appropriate low temperature value should be. Possibly ~ -20F or so (or -30C / -22F for some nice round numbers)?

PierreAronnax commented 1 year ago

Yes, I was thinking -30C myself too.

bdraco commented 1 year ago

Manually reopening this since auto closed when I merged @PierreAronnax 's PR to govee-ble

rsiv commented 1 year ago

Confirming that this fixes missing values with Govee H5075 when temperature is below 0 F. I couldn't figure out why my freezer sensor has been misbehaving in the past few weeks. Was just about to attempt to debug it when I saw this issue.

74jtheriault commented 1 year ago

Has this fix been pushed out to HA? I am still having the same issue.

bdraco commented 1 year ago

Has this fix been pushed out to HA? I am still having the same issue.

There has not been a PR opened to bump the library.

74jtheriault commented 1 year ago

Is there a way to manually update this in my HA instance?

rsiv commented 1 year ago

I'm running HA in a standalone docker container. I just changed MIN_TEMP value to -30 in parser.py in my running container:

docker-compose exec homeassistant bash
vi /usr/local/lib/python3.10/site-packages/govee_ble/parser.py
74jtheriault commented 1 year ago

I'm running mine on Home assistant OS and that location is not there.

rsiv commented 1 year ago

These instructions might help: https://community.home-assistant.io/t/execute-in-home-assistant-container-context/415031

74jtheriault commented 1 year ago

There is no python3.10 folder in that location. The only one I have there is Perl

rsiv commented 1 year ago

You can use find to locate that file: find / -name parser.py | grep govee_ble

74jtheriault commented 1 year ago

it doesn't find that file or location. I followed the instructions in that previous link to connect to ssh and web terminal. Do i need to elevate the prompt before I search?

rsiv commented 1 year ago

Are you running it inside the docker container after running docker exec -it homeassistant bash?

74jtheriault commented 1 year ago

ok that worked. Thanks for your help