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.15k stars 30.19k forks source link

TP-Link KE100 unavailable if heating and low battery #121335

Closed karl-the-llama closed 2 months ago

karl-the-llama commented 3 months ago

The problem

I just installed the TP-Link Smart Home integration because of the added support for the Kasa KE100 thermostat. I only installed it in core 2024.7.1 . My first test was to set one thermostat to heat which made it unavailable. It was a thermostat with low battery. When I switched it off outside HA it immediately became available again. My two other KE100s with full battery are not affected. The log says there are 2 states (heating, low bat) and that it chose low battery.

What version of Home Assistant Core has the issue?

2024.7.1

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

TP-Link Smart Home

Link to integration documentation on our website

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

Diagnostics information

config_entry-tplink-01J2246E2BPXHQS6P3KYZEWYJ9.json

Example YAML snippet

No response

Anything in the logs that might be useful for us?

2024-07-05 23:05:27.743 WARNING (MainThread) [kasa.smart.modules.temperaturecontrol] Got multiple states (['low_battery', 'heating']), using the first one: low_battery
2024-07-05 23:05:27.744 WARNING (MainThread) [kasa.smart.modules.temperaturecontrol] Got unknown state: low_battery
2024-07-05 23:05:27.745 WARNING (MainThread) [homeassistant.components.tplink.entity] Unable to read data for <DeviceType.Thermostat SZ Thermo (KE100) of <DeviceType.Hub at 192.168.178.118 - Smart Hub (KH100)>> climate.sz_thermo: <ThermostatState.Unknown: 'unknown'>

Additional information

No response

home-assistant[bot] commented 3 months ago

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

Code owner commands Code owners of `tplink` 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 tplink` 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)


tplink documentation tplink source (message by IssueLinks)

rytilahti commented 3 months ago

Thanks for the report & the logs! This will require changes to the upstream library to handle this correctly, so I added a label for it.

rytilahti commented 2 months ago

Fixed by #121791 so this should work with 2024.7.3 when it's released.