-
-
Notifications
You must be signed in to change notification settings - Fork 30.8k
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
Fix tplink overloading power strips #104208
Conversation
Hey there @rytilahti, @TheGardenMonkey, mind taking a look at this pull request as it has been labeled with an integration ( Code owner commandsCode owners of
|
I just realized that since I've been running this my database average size has dropped 50mib over 10 days as most of the data was useless churn generated by the voltage and wattage sensors fluxing at 0.00..... |
Thanks. leaving as a draft and will merge later after we fix #103150 since issues are building up there and it likely need to be backported (and it will conflict here) |
Going to merge this now since I keep forgetting to merge it into my integration branch while testing #105143 and I was thinking we had regressed the power strips again |
* upstream/dev: (2071 commits) Set zwave_js voltage sensor suggested precision (home-assistant#107116) Bump bluetooth-adapters to 0.17.0 (home-assistant#107195) Disable IPv6 in the opower integration to fix AEP utilities (home-assistant#107203) Fix conversation snapshots (home-assistant#107196) Report missing entities/areas instead of failing to match in Assist (home-assistant#107151) Bump to PyTado 0.17.3 (home-assistant#107181) Fix switch states in AVM FRITZ!Box Tools (home-assistant#107183) Fix tplink overloading power strips (home-assistant#104208) Update sensorpush-ble library to 1.6.1 (home-assistant#107168) Bump aiohomekit to 3.1.2 (home-assistant#107177) Introduce base entity in streamlabs water (home-assistant#107095) Clean up outdated entity replacement logic in Guardian (home-assistant#107160) Add conversation_id parameter to conversation.process service (home-assistant#106078) Pass aiohttp clientsession to tedee integration (home-assistant#107089) Update Home Assistant base image to 2024.01.0 - Python 3.12 (home-assistant#107175) Remove precision in streamlabs water (home-assistant#107096) Pass down language to hassil (home-assistant#106490) Use snapshots in Glances sensor tests (home-assistant#107159) Cache homekit_controller supported features (home-assistant#106702) Migrate AVM FRITZ!Box Call monitor to has entity name (home-assistant#99752) ...
replaces #103668 since I can't reopen that one as I forced pushed while trying to get the CI working from open/close
Breaking change
To prevent overloading the power strips, the energy data is only polled every 60 seconds. If you need to poll the data faster, please use an automation that calls the
homeassistant.update_entity
service.Proposed change
In #87399 we reduced the polling interval to 5s, and this works great for most devices. However power strips do
plugs * N + 1
updates which causes them to get overloaded.Turning on individual sockets is now reliable and doesn't fail ~20% of the time anymore. Also its now possible for the kasa app to be open when setting up the device without it failing.
Type of change
Additional information
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.To help with the load of incoming pull requests: