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
72.54k stars 30.35k forks source link

flux_led not working well and complain about unique ID #49130

Closed newbeewan closed 3 years ago

newbeewan commented 3 years ago

The problem

Hi,

I have some light controled by the MagicHome app. This is some RGB bulbs.

I have tried automatic discovery with :

light:

It detects my bulbs but there are in read only because no unique ID.

I tried to define manually each bulb, but no better result

What is version of Home Assistant Core has the issue?

core-2021.4.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Core

Integration causing the issue

MagicHome bulb

Link to integration documentation on our website

https://www.home-assistant.io/integrations/flux_led

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

probot-home-assistant[bot] commented 3 years ago

flux_led documentation flux_led source (message by IssueLinks)

icemanch commented 3 years ago

How many bulbs?

Can you run "flux_led -sSti" on the command line and paste what comes back?

dmwyatt commented 3 years ago

Yeah, there's lots of people complaining on forums about flux_led not assigning unique ids.

It's a problem for me because you can't configure entities and assign them to an area unless they have a unique_id.

shanelord01 commented 3 years ago

I also request the ability to add a unique_id to each - even if manually added ie:

light:
  - platform: flux_led
    devices:
      192.168.1.81:
        name: String LEDs
        mode: "rgb"
        unique_id: “Magic_LED_1”

Can this be added?

Thanks.

PecceG2 commented 3 years ago

+1 to being able to specify a unique ID, even if it must be done manually. Unfortunately the new lamps with the LEDENET protocol do not surprise tasmota or ESPHome like the ones before.