Incorrect light type for LEDVANCE A19 RGBW

[Thomas-Vos]

Describe the bug

A user reported that their LEDVANCE A19 RGBW has the wrong light type. It is reported as a Color temperature light but it also supports setting a color. So it should be Extended color light instead.

https://community.hueessentials.com/t/ledvance-a19-rgbw-bulb-no-color-controls/4579

Steps to reproduce the behavior

N/A

Expected behavior

Extended color light as light type

Screenshots

https://community.hueessentials.com/t/ledvance-a19-rgbw-bulb-no-color-controls/4579

Environment

N/A

deCONZ Logs

N/A

Additional context

#3721
#3365

[Mimiix]

Hi Thomas

Can you (or ask the user to) provide the rest api response ?

Kind regards,

[Thomas-Vos]

@Mimiix already discussed the issue with the devs on Discord, and I asked the user to provide the DDF so the bug can be fixed. It seems deCONZ is overriding the light type incorrectly here:

deconz-rest-plugin/de_web_plugin.cpp

Line 3260 in 5d136ae

item->setValue(QVariant("Color temperature light"));

I will make a PR to fix this code, and once the user reports the DDF it can be fixed in a DDF as well.

[Mimiix]

Aah okay. Good to hear you figured it out!

[Smanar]

Have found capture here #3365 (comment)
Will be nice if he can share the "auto generated" DDF, when selecting a node and making "Edit DDF", the DDF is on the third tab "preview"

In theorie just need to replace

"type": "$TYPE_COLOR_TEMPERATURE_LIGHT",
by
"type": "$TYPE_EXTENDED_COLOR_LIGHT",

In the "auto generated" DDF, but you can miss some fields too, like state/x or state/y, can need too to freeze for exemple the state/colormode.

Will be usefull if we can have the attribute list in the cluster 0x0300.

[Thomas-Vos]

Created a PR to fix the legacy code here: #6592

[Ltek]

I changed the type in the DDF and Hue Essentials now correctly controls the bulb... though HE still shows it as 'unknown'. I also set the DDF to 'gold'

stock DDF...

{
"schema": "devcap1.schema.json",
"manufacturername": "LEDVANCE",
"modelid": "A19 RGBW",
"product": "A19 RGBW",
"sleeper": false,
"status": "Draft",
"subdevices": [
{
"type": "$TYPE_COLOR_TEMPERATURE_LIGHT",
"restapi": "/lights",
"uuid": [
"$address.ext",
"0x01"
],
"items": [
{
"name": "attr/id"
},
{
"name": "attr/lastannounced"
},
{
"name": "attr/lastseen"
},
{
"name": "attr/manufacturername"
},
{
"name": "attr/modelid"
},
{
"name": "attr/name"
},
{
"name": "attr/swversion"
},
{
"name": "attr/type"
},
{
"name": "attr/uniqueid"
},
{
"name": "config/colorcapabilities",
"description": "The supported color modes as bitmap.",
"refresh.interval": 84000
},
{
"name": "config/ctmax",
"description": "Maximum supported color temperature.",
"refresh.interval": 84000
},
{
"name": "config/ctmin",
"description": "Minimum supported color temperature.",
"refresh.interval": 84000
},
{
"name": "state/alert",
"description": "The currently active alert effect.",
"default": "none"
},
{
"name": "state/bri",
"description": "The current brightness.",
"refresh.interval": 5
},
{
"name": "state/colormode",
"description": "The currently active color mode.",
"refresh.interval": 300
},
{
"name": "state/ct",
"description": "The current Mired color temperature of the light. Where Mired is 1000000 / color temperature (in kelvins).",
"refresh.interval": 5
},
{
"name": "state/effect",
"description": "The currently active effect.",
"default": "none"
},
{
"name": "state/hue",
"description": "The current color hue."
},
{
"name": "state/on",
"description": "True when device is on; false when off.",
"refresh.interval": 5
},
{
"name": "state/reachable"
},
{
"name": "state/sat",
"description": "The current color saturation.",
"refresh.interval": 5
},
{
"name": "state/x",
"description": "The current color x coordinate.",
"refresh.interval": 5
},
{
"name": "state/y",
"description": "The current color y coordinate."
}
]
}
],
"bindings": [
{
"bind": "unicast",
"src.ep": 1,
"dst.ep": 1,
"cl": "0x0006",
"report": [
{
"at": "0x0000",
"dt": "0x10",
"min": 1,
"max": 300
}
]
},
{
"bind": "unicast",
"src.ep": 1,
"dst.ep": 1,
"cl": "0x0008",
"report": [
{
"at": "0x0000",
"dt": "0x20",
"min": 1,
"max": 300,
"change": "0x00000001"
}
]
},
{
"bind": "unicast",
"src.ep": 1,
"dst.ep": 1,
"cl": "0x0300",
"report": [
{
"at": "0x0007",
"dt": "0x21",
"min": 1,
"max": 300,
"change": "0x00000001"
},
{
"at": "0x0003",
"dt": "0x21",
"min": 1,
"max": 300,
"change": "0x0000000A"
},
{
"at": "0x0004",
"dt": "0x21",
"min": 1,
"max": 300,
"change": "0x0000000A"
},
{
"at": "0x0008",
"dt": "0x30",
"min": 1,
"max": 300
}
]
}
]
}

[Thomas-Vos]

Thanks a lot for the DDF!

The unknown in Hue Essentials is unrelated to this issue and will be fixed in the next version, it's just a missing icon.

[Smanar]

@Ltek Nice thx ^^

Have updated the DDF,

• set the mode to "Gold"
• removed all description
• changed the type to TYPE_EXTENDED_COLOR_LIGHT
• increased all small "refresh.interval" as the device support bind/report

{
  "schema": "devcap1.schema.json",
  "manufacturername": "LEDVANCE",
  "modelid": "A19 RGBW",
  "product": "A19 RGBW",
  "sleeper": false,
  "status": "Gold",
  "subdevices": [
    {
      "type": "$TYPE_EXTENDED_COLOR_LIGHT",
      "restapi": "/lights",
      "uuid": [
        "$address.ext",
        "0x01"
      ],
      "items": [
        {
          "name": "attr/id"
        },
        {
          "name": "attr/lastannounced"
        },
        {
          "name": "attr/lastseen"
        },
        {
          "name": "attr/manufacturername"
        },
        {
          "name": "attr/modelid"
        },
        {
          "name": "attr/name"
        },
        {
          "name": "attr/swversion"
        },
        {
          "name": "attr/type"
        },
        {
          "name": "attr/uniqueid"
        },
        {
          "name": "config/colorcapabilities",
          "static": 16
        },
        {
          "name": "config/ctmax",
          "static": 666
        },
        {
          "name": "config/ctmin",
          "static": 142
        },
        {
          "name": "state/alert",
          "default": "none"
        },
        {
          "name": "state/bri",
          "refresh.interval": 600
        },
        {
          "name": "state/colormode",
          "refresh.interval": 84000
        },
        {
          "name": "state/ct",
          "refresh.interval": 600
        },
        {
          "name": "state/effect",
          "default": "none"
        },
        {
          "name": "state/hue",
          "refresh.interval": 600
        },
        {
          "name": "state/on",
          "refresh.interval": 600
        },
        {
          "name": "state/reachable"
        },
        {
          "name": "state/sat",
          "refresh.interval": 600
        },
        {
          "name": "state/x",
          "refresh.interval": 600
        },
        {
          "name": "state/y",
          "refresh.interval": 600
        }
      ]
    }
  ],
  "bindings": [
    {
      "bind": "unicast",
      "src.ep": 1,
      "dst.ep": 1,
      "cl": "0x0006",
      "report": [
        {
          "at": "0x0000",
          "dt": "0x10",
          "min": 1,
          "max": 300
        }
      ]
    },
    {
      "bind": "unicast",
      "src.ep": 1,
      "dst.ep": 1,
      "cl": "0x0008",
      "report": [
        {
          "at": "0x0000",
          "dt": "0x20",
          "min": 1,
          "max": 300,
          "change": "0x00000001"
        }
      ]
    },
    {
      "bind": "unicast",
      "src.ep": 1,
      "dst.ep": 1,
      "cl": "0x0300",
      "report": [
        {
          "at": "0x0007",
          "dt": "0x21",
          "min": 1,
          "max": 300,
          "change": "0x00000001"
        },
        {
          "at": "0x0003",
          "dt": "0x21",
          "min": 1,
          "max": 300,
          "change": "0x0000000A"
        },
        {
          "at": "0x0004",
          "dt": "0x21",
          "min": 1,
          "max": 300,
          "change": "0x0000000A"
        },
        {
          "at": "0x0008",
          "dt": "0x30",
          "min": 1,
          "max": 300
        }
      ]
    }
  ]
}

I haven't freezed state/colormode (so this value can be wrong) but have freeezed other as the legacy code do it

        lightNode->addItem(DataTypeUInt16, RConfigCtMin)->setValue(142);
        lightNode->addItem(DataTypeUInt16, RConfigCtMax)->setValue(666);
        lightNode->addItem(DataTypeUInt16, RConfigColorCapabilities)->setValue(0x0001 | 0x0008 | 0x0010);

If you can test the last DDF pls ? You know how to replace them ?

[manup]

I'd propose to add @Smanar DDF. Currently we don't have any LEDVANCE lights available as DDF but a bunch of C++ workarounds predating DDF. So it would be cool to transition these lights to DDF over time and remove the C++ parts after that.

[ebaauw]

Can we please, please create new DDFs first-time-right?

• config/colorcapabilities of 16 is wrong. If the legacy code is correct, it should be 0x19. Are we sure this light doesn't support enhanced hue/saturation nor color loop? Probably, as these are pre-Zigbee-3 ZHA lights, without the ZLL extensions (hence the wrong type, as Enhanced Color Light is a ZLL extension).
• config/colorcapabilities, config/ctmax and config/ctmin have been deprecated - please use cap/color/capabilities, cap/color/ct/max and cap/color/ct/min instead.
• Please add the cap/color/xy capabilities. I'm not sure if LEDVANCE exposes these, nor whether they use the ColorPoint ZIgbee attribute set (like Hue), or the Primary set (like IKEA). If not, hard-code them (like MLI). According to my notes, LEDVANCE use: { r: [0.6972, 0.3027], g: [0.1737, 0.6991], b: [0.1227, 0.0959] }. Multiply these API values by 0xFFFF to get the Zigbee values.
• Please double-check whether they support Off with Effect, Move to Level (with On/Off), Trigger Effect, and, if so, include the corresponding cap items.
• Please double check whether they update the Current X and Current Y values when setting color temperature, and, if so, include cap/color/ct/computes_xy.
• Please double check whether they support Execute_If_Off and if so, include the corresponding config items.
• I don't suppose they support the Startup attributes, as OSRAM uses a specific command to set the startup state. Not sure if we ever exposed that through the API.

For examples, see:

• Hue: devices/philips/light_zb3_C_gradient.json
• MLI: devices/mli/zbt-extendedcolor.json
• IKEA: devices/ikea/cws_light.json