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
73.31k stars 30.62k forks source link

Broadlink A1 sensor update delay #40985

Closed AsafDAVIDOF closed 3 years ago

AsafDAVIDOF commented 4 years ago

The problem

i integrated broadlink A1 sensor (by broadlink e-control app) and then by home assistant Integrations. the sensor is shown ok, temperature, light and so on the noise sensor have 3 options: 0 (quiet) 1 (regular) 2 noisy most of the time the sensor stuck on 1 after high noise it take forever to change to 2 - and then stay there for a lot of time then return to 1 - and cannot change to 2 with high noise for long time all of that made me feel the the scan interval of the sensor is not good

-->

Environment

home assistant on VM box Home Assistant 0.115.0

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

felipediel commented 4 years ago

Updates are made every minute, so ambient noise is not updated in real time. To do this we would have to reduce the scan interval to one second, which does not seem like a wise choice for a polling integration. In the future I will expose this option, so you will be able to choose the most suitable interval for your need and network capacity.

felipediel commented 4 years ago

Do you think 10s would be ok for this sensor?

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

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

AsafDAVIDOF commented 4 years ago

hi thank you for the answer yes, i think 10 sec will be great start

felipediel commented 4 years ago

I just opened a PR for this. I don't have this device. Would you like to help me test the update? We will only be able to merge the code after someone tests it, and we will hardly find anyone else with that device.

It will be easier if you set up a development environment.