home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
73.55k stars 30.72k forks source link

[LimitlessLED] Unable to control second Wifi Bridge #6452

Closed philhawthorne closed 7 years ago

philhawthorne commented 7 years ago

Home Assistant release (hass --version): 0.39.3

Component/platform: LimitlessLED

Description of problem: Since the upgrade to 0,39, Home Assistant loses the ability to control my second LimitlessLED Wifi Bridge.

When restarting Home Assistant, all my LimitlessLED lights turn off during the reboot phase. After the reboot, I am able to control my lights on the first bridge as expected.

However, at random points in time, Home Assistant loses the ability to control my second LimitlessLED bridge. This means the lights attached to them are stuck off/on depending at what time Home Assistant loses the ability to control that bridge.

Expected: I should always be able to control my lights. Previous versions of Home Assistant did not have this issue.

Problem-relevant configuration.yaml entries and steps to reproduce:

- platform: limitlessled
    bridges:
    - host: 192.168.1.24
      version: 5
      port: 8899
      groups:
      - number: 2
        type: rgbw
        name: Bookcase Left
      - number: 1
        type: rgbw
        name: Bookcase Right
    - host: 192.168.1.21
      version: 5
      port: 8899
      groups:
      - number: 1
        type: rgbw
        name: WIR
      - number: 2
        type: rgbw
        name: Dining
      - number: 3
        type: rgbw
        name: Living

Traceback (if applicable):

17-03-06 21:57:02 INFO (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_off, service_data=entity_id=['light.bookcase_left'], domain=light, service_call_id=140231920600624-299>
17-03-06 21:57:02 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: new_state=<state light.bookcase_left=off; icon=mdi:glassdoor, emulated_hue_name=bookcase left, emulated_hue=True, friendly_name=Bookcase Left, supported_features=61 @ 2017-03-06T21:57:02.586014+11:00>, old_state=<state light.bookcase_left=on; rgb_color=[255, 255, 255], icon=mdi:glassdoor, emulated_hue=True, supported_features=61, emulated_hue_name=bookcase left, friendly_name=Bookcase Left, brightness=155 @ 2017-03-06T21:56:36.606067+11:00>, entity_id=light.bookcase_left>
17-03-06 21:57:02 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140231920600624-299>
17-03-06 21:57:02 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140231920600624-298>
17-03-06 21:57:03 INFO (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_off, service_data=entity_id=light.bookcase_right, domain=homeassistant, service_call_id=140231920600624-300>
17-03-06 21:57:03 INFO (MainThread) [homeassistant.core] Bus:Handling <Event call_service[L]: service=turn_off, service_data=entity_id=['light.bookcase_right'], domain=light, service_call_id=140231920600624-301>
17-03-06 21:57:03 INFO (MainThread) [homeassistant.core] Bus:Handling <Event state_changed[L]: new_state=<state light.bookcase_right=off; icon=mdi:glassdoor, emulated_hue_name=bookcase right, emulated_hue=True, friendly_name=Bookcase Right, supported_features=61 @ 2017-03-06T21:57:03.151253+11:00>, old_state=<state light.bookcase_right=on; rgb_color=[255, 255, 255], icon=mdi:glassdoor, emulated_hue=True, supported_features=61, emulated_hue_name=bookcase right, friendly_name=Bookcase Right, brightness=155 @ 2017-03-06T21:56:39.439734+11:00>, entity_id=light.bookcase_right>
17-03-06 21:57:03 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140231920600624-301>
17-03-06 21:57:03 INFO (MainThread) [homeassistant.core] Bus:Handling <Event service_executed[L]: service_call_id=140231920600624-300>
janLo commented 7 years ago

Do you have a transition specified somewhere? See https://github.com/home-assistant/home-assistant/issues/6295

philhawthorne commented 7 years ago

@janLo yes, yes I do. Thank you!

So, looks like if you have a transition anywhere for that light, the exception is thrown and you can't use it. Looks like I'll wait for 0,40 or apply your patch for now :)

Thanks again!

philhawthorne commented 7 years ago

Hey @janLo after upgrading my version to the latest limitlessled library (1.0.5) I'm still seeing problems controlling lights that contain a transition. Wondering if the fix hasn't worked.

janLo commented 7 years ago

I havn't upgraded my limitlessled as I let HA pull its needed dependencies. I applied the patch as I wrote earlier and the problem is gone. Can you post the error/exception you get?

@balloob can we please release a bugfix release as more and more people run into this regression? Also the clarification for the docs need still to be merged.

janLo commented 7 years ago

@philhawthorne The patch is definitely in the 1.0.5 tarball.

philhawthorne commented 7 years ago

Yeah that's what I thought too.

Okay I'll dig some more. Maybe my instance has reverted for some reason. I'll double check when I can to make sure it's on the right version and retest everything.