Skip to content

Fix flux_led only-white controllers (and remove explicit declaration as RGBW in automatic add) #22210

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
balloob merged 13 commits into from
Apr 16, 2019
Merged

Fix flux_led only-white controllers (and remove explicit declaration as RGBW in automatic add) #22210

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
08e9322
Remove explicit declaration of automatic found devices as RGBW
Mar 20, 2019
5f9ccd2
fixes for Magic Home only-white controllers
Apr 4, 2019
e23c366
mode is now set to None instead of removed entirely
Apr 4, 2019
52cb222
flux_led now changes no values when turned on from off state.
Apr 4, 2019
b66a628
Merge 'upstream/dev'
amelchio Apr 9, 2019
05d8200
better checking for changed values in turn_on
Apr 10, 2019
0254abb
Merge branch 'flux_led_fix' of github.com:autinerd/home-assistant int…
Apr 10, 2019
f568097
Reduce waiting time to 1 second
Apr 10, 2019
b46cbbe
Correction of turn on logic
Apr 11, 2019
afd673d
Merge branch 'flux_led_fix' of github.com:autinerd/home-assistant int…
Apr 11, 2019
f780d73
Remove accidentally inserted 'not'
Apr 11, 2019
f104fc0
Remove lint
amelchio Apr 11, 2019
b476390
Remove redundant code
amelchio Apr 15, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion homeassistant/components/flux_led/light.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,6 @@ def setup_platform(hass, config, add_entities, discovery_info=None):
if ipaddr in light_ips:
continue
device['name'] = '{} {}'.format(device['id'], ipaddr)
device[ATTR_MODE] = MODE_RGBW

@amelchio amelchio Mar 25, 2019

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The device[ATTR_MODE] value is always read in __init__() so this will make it always raise an exception. I think it could work if you set it to None instead.

Just a note for others, the mode is set automatically if not explicitly defined: https://github.com/home-assistant/home-assistant/blob/08e93224e0689c9a20a2b6b43057f60348bb7c14/homeassistant/components/flux_led/light.py#L185-L191

(whether that works for all devices, I don't know)

device[CONF_PROTOCOL] = None
device[CONF_CUSTOM_EFFECT] = None
light = FluxLight(device)
Expand Down