search

mariusmotea / diyHue

Philips Hue emulator that is able to control multiple types of lights
Other
627 stars 107 forks source link

Remove sleep(1) #435

Closed ticed35 closed 5 years ago

ticed35 commented 5 years ago

Hi, I found that the line 665 with sleep(1) was causing delay when used with real Hue bridge. When removing this line i have no more delay in entertainment mode and dimming my philips hue lights connected to real bridge.

Does this line really necessary ? Cédric

mariusmotea commented 5 years ago

I believe is there because Deconz that use also hue api cannot accept bri + colors in two consecutive requests. Let me check as i use Deconz, maybe is not the case anymore.

mariusmotea commented 5 years ago

Today i had issues with Deconz, scenes don't manage to increase also the brightness. If this is not the case with original hue bridge you can add extra condition 

if bridge_config["lights_address"][light]["protocol"] == "deconz":
    sleep(0.7)