happyleavesaoc / python-limitlessled

MIT License
33 stars 22 forks source link

Throttle v6 commands #38

Closed amelchio closed 6 years ago

amelchio commented 6 years ago

Unfortunately, #37 has turned out to cause some reliability issues. Messages are now sent back to back and this is apparently too fast for the bulbs so random messages end up being ignored.

Adding a delay solves the issue (though it negates some of the gains in #37).

Quite some people are affected by this (in home-assistant/home-assistant#15844) so it would be great if you could push a new update soon.

Sorry for missing this the first time around :-/

amelchio commented 6 years ago

I just got the ESP8266 DIY bridge and this has allowed me to test a multi-bridge scenario.

Other bridges can apparently disturb a bulb so it misses a message. Thus, I have changed the lock to be global so that multiple bridges will not be sending right after each other. This further improves the reliability.

Sorry if I have been spamming you with notifications ... I forgot that this was the branch that I had pushed to you.

amelchio commented 6 years ago

I will close this since we missed the HA update and I am still trying things out.