search

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
71.11k stars 29.79k forks source link

Xiaomi Air Purifier/Humidifier update interval was made too high and not configurable #55865

Closed OleksandrBerchenko closed 2 years ago

OleksandrBerchenko commented 3 years ago

The problem

During the recent rework of Xiaomi Purifiers/Humidifiers update interval was changed from 10 seconds to 60 seconds: https://github.com/home-assistant/core/commit/781015fb19297733849db0bf4db2063d9e2d0afe#diff-6d8542b3768a850a8601f2fab8b6ff78fb23b9635055c40e7c38a27904c33293R136

At least for my use case, this value is too high :( From time to time, electricity disappears for a few (up to 30) seconds. Xiaomi Air Purifier doesn't remember its previous state and becomes turned off. I have an automation that turns it on when it went online. With such a high interval, it doesn't work.

It would be great to revert the value back to 10 seconds.

Ideally, if this value is configurable (using scan_interval or something else).

Thanks!

What is version of Home Assistant Core has the issue?

core-2021.9.2

What was the last working version of Home Assistant Core?

core-2021.7.X

What type of installation are you running?

Home Assistant Core

Integration causing the issue

Xiaomi Miio -> Xiaomi Air Purifier, Air Humidifier and Standing Fan

Link to integration documentation on our website

https://www.home-assistant.io/integrations/xiaomi_miio/#xiaomi-air-purifier-air-humidifier-and-standing-fan

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

No response

probot-home-assistant[bot] commented 3 years ago

Hey there @rytilahti, @syssi, @starkillerog, @bieniu, mind taking a look at this issue as it has been labeled with an integration (xiaomi_miio) you are listed as a code owner for? Thanks! (message by CodeOwnersMention)

xiaomi_miio documentation xiaomi_miio source (message by IssueLinks)

jbouwh commented 3 years ago

The update time of 60s is just fine to sync the background. With a smaller interval the io load will increase, especially if you have more devices. When updating the device, it should respond immediately, the state will be cashed until it is refreshed by the backend.

OleksandrBerchenko commented 3 years ago

Yes, I understand that motivation. But that doesn't help in the case that I mentioned above. Did anybody complain about the high load with a smaller interval?

Is it possible to make it configurable? That would cover all possible use cases.

Thanks!

bjornsivertsen commented 2 years ago

I am also saddened to hear that it has been increased. Makes my lovelace card quite unresponsive visually.

Was the io load a big issue to people?

schilea commented 2 years ago

Me too. We are forced to use the unofficial integration MIoT.

snp88 commented 2 years ago

Hi

You all talk about 60s 30s but my PM2.5 sensor in purifier is updated every 30 min yes not 30s but 30min :)

bieniu commented 2 years ago

We are currently discussing about decreasing of the update interval, please be patient.

jbouwh commented 2 years ago

Hi

You all talk about 60s 30s but my PM2.5 sensor in purifier is updated every 30 min yes not 30s but 30min :)

The DataCoordinator current data is refreshed every 60s. If you have an issue with a specific device , please open a separate issue for that and include some logs if you are seeing errors.