From b546ee35bf20759a4c7af1e07cc3d4319cb0176f Mon Sep 17 00:00:00 2001 From: Anders Melchiorsen Date: Thu, 14 Sep 2017 22:50:25 +0200 Subject: [PATCH] LIFX: fix color restore after effects on multi-zone lights The aiolifx 0.6.0 release fixes an issue where an effect color could remain set after stopping the effect. This only affected multi-zone lights (i.e. LIFX Z) and only if the effect was stopped by setting the light brightness or the color (but not both). The aiolifx 0.6.0 release also defaults end_index to start_index+7, so we can remove that argument. Finally, aiolifx_effects 0.1.2 adds support for aiolifx 0.6.0. --- homeassistant/components/light/lifx.py | 5 ++--- requirements_all.txt | 4 ++-- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/homeassistant/components/light/lifx.py b/homeassistant/components/light/lifx.py index 6b57a1c5146319..93412710987e48 100644 --- a/homeassistant/components/light/lifx.py +++ b/homeassistant/components/light/lifx.py @@ -33,7 +33,7 @@ _LOGGER = logging.getLogger(__name__) -REQUIREMENTS = ['aiolifx==0.5.4', 'aiolifx_effects==0.1.1'] +REQUIREMENTS = ['aiolifx==0.6.0', 'aiolifx_effects==0.1.2'] UDP_BROADCAST_PORT = 56700 @@ -684,8 +684,7 @@ def update_color_zones(self): # Each get_color_zones can update 8 zones at once resp = yield from AwaitAioLIFX().wait(partial( self.device.get_color_zones, - start_index=zone, - end_index=zone+7)) + start_index=zone)) if resp: zone += 8 top = resp.count diff --git a/requirements_all.txt b/requirements_all.txt index e23c6aaaf5537f..76900e9896f779 100644 --- a/requirements_all.txt +++ b/requirements_all.txt @@ -55,10 +55,10 @@ aiodns==1.1.1 aiohttp_cors==0.5.3 # homeassistant.components.light.lifx -aiolifx==0.5.4 +aiolifx==0.6.0 # homeassistant.components.light.lifx -aiolifx_effects==0.1.1 +aiolifx_effects==0.1.2 # homeassistant.components.scene.hunterdouglas_powerview aiopvapi==1.4