dresden-elektronik / deconz-rest-plugin

deCONZ REST-API plugin to control ZigBee devices
BSD 3-Clause "New" or "Revised" License
1.9k stars 498 forks source link

Fix state of Hue lights not always updated correctly #7704

Closed ebaauw closed 5 months ago

ebaauw commented 5 months ago

Damn, #7652 is not working as intended. Bloody operator precedence in Javascript.

I missed this during testing, since, apparently, my lights were configured with bindings and attribute reporting for Color Control. Apparently, deCONZ won't remove old bindings no longer defined in the DDF?

manup commented 5 months ago

Ha good catch.

deCONZ won't remove old bindings no longer defined in the DDF?

I was thinking about adding this initially since it can be easily done. However I was afraid of breaking some stuff if done without context. However on the todo list is an addition to specify in the DDF that unknown bindings should be removed (default: false as it is now): This way we will be able to control exactly how things are done. Note there are dozens of devices where it can fail since removing bindings does strange things or even impacts groups (Silabs stack based devices are really nasty here).

woytekbode commented 5 months ago

Just to let you guys know I am currently experiencing improper state updates in my HA + Deconz set-up for my Hue lights.