Skip to content
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

Update of climate.xxx is taking over 10 seconds #52739

Closed
andrewwa0 opened this issue Jul 8, 2021 · 4 comments
Closed

Update of climate.xxx is taking over 10 seconds #52739

andrewwa0 opened this issue Jul 8, 2021 · 4 comments

Comments

@andrewwa0
Copy link

andrewwa0 commented Jul 8, 2021

The problem

My log is full of these messages for 3 Daikin BRP072C42 units. According to the logs, each update consists of 10 requests to each unit, mostly for querying power consumption.
Any of these single requests can take a couple of seconds to complete when requested from curl (either on my raspberry pi, or my PC), so the warnings in the log come as no surprise.
Personally, I don't care about the power consumption of the units - I just want to be warm! Ideally, if I disable the power consumption entities, the requests for that information should not be sent and the update of the climate entities would be completed in a reasonable time.

What is version of Home Assistant Core has the issue?

2021.7.0

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

daikin

Link to integration documentation on our website

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

Example YAML snippet

No response

Anything in the logs that might be useful for us?

2021-07-08 22:03:05 DEBUG (MainThread) [pydaikin.daikin_base] Updating ['aircon/get_sensor_info', 'aircon/get_control_info', 'aircon/get_day_power_ex', 'aircon/get_week_power', 'aircon/get_day_power_ex', 'aircon/get_week_power', 'aircon/get_day_power_ex', 'aircon/get_week_power', 'aircon/get_day_power_ex', 'aircon/get_week_power']
2021-07-08 22:03:15 WARNING (MainThread) [homeassistant.helpers.entity] Update of climate.bedroom_ac is taking over 10 seconds

Additional information

Screenshot 2021-07-09 090408

@probot-home-assistant
Copy link

daikin documentation
daikin source
(message by IssueLinks)

@probot-home-assistant
Copy link

Hey there @fredrike, mind taking a look at this issue as its been labeled with an integration (daikin) you are listed as a codeowner for? Thanks!
(message by CodeOwnersMention)

@mlemainque
Copy link
Contributor

During the integration initialization we need to perform at least one call to know if the energy consumption is supported by the device, but then if the entities are manually disabled by the user we could disable the HTTP calls in pydaikin
We could keep this behavior for all endpoints - not only power consumption - and keep only the ones needed by the enabled entities

@github-actions
Copy link

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment 👍
This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Oct 13, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Nov 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants