[deconz] Color item state not correct after sendCommand

[Owlbertz]

Expected Behavior

Issuing a command to a Deconz extendedcolorlight color channel does not update the item state.
As far as I can tell this worked in the 4.1.X version, but the behavior changed with 4.3.5.

Current Behavior

The item state is not the same as the issued command.

Setting the value to 13,97,10 makes the item state go to 15.641,87.13100,100.00 and after a short delay it becomes 21.749,87.37800,52.781. The light itself is dimmed to (whats seems to be) 10% and stays that way. It never has a brightness of 100% (which would be the first state after issuing the command) nor does it change its brightness in anyway after the initial dimming.

Issuing the same command again will not change the state to 13,97,10 either.

Steps to Reproduce (for Bugs)

Thing:

extendedcolorlight         Light_LED_Office              "LED-Streifen Büro"          @ "Office"          [ id="16" ]

Item:

Color                 Light_LED_Office_Color                          "LED-Streifen Büro Farbe"                                                                          {channel = "deconz:extendedcolorlight:homeserver:Light_LED_Office:color"}

1. Send command 13,97,10
2. Check state -> it is 15.641,87.13100,100.00
3. Check state after 1 second again -> it is 21.749,87.37800,52.781

Context

I noticed this because I have rules that set the light value and then change it back after a while if it has not changed. This check now fails, because the expected value (the command issued) will always differ from the current state.

Your Environment

openHABian with 4.3.5 on a Raspberry Pi

[andrewfg]

In the last few months there has been some work on the Deconz lighting stuff as follows. It looks like only the first one or two of them could have impacted your case. But can you please study the respective threads and let me know which one do you think may be the culprit?

• [deconz] Fix color conversion #16769 included in OH 4.2 Milestone
• [deconz] Improve color temperature channel #17777 included in OH 4.3 Milestone
• [deconz] Support QuantityType Color Temperature command #17942 included in OH 5.0 Milestone
• [deconz] Cross update color & color temperature channels #18482 included in OH 5.0 Milestone

Also -- in any case -- can you please log:set TRACE org.openhab.,binding.deconz and post the (relevant parts of) the log file here.

[Owlbertz]

Hi @andrewfg, thank you for your response.

This is the trace log for my command Color -> 100,100,80. Afterwards the state became 100,100,100. So any logic checking if the brightness is 80 (I already added some tolerance before because it was always like 0.1 off) fails.

2025-06-19 20:43:09.153 [TRACE] [internal.handler.DeconzBridgeHandler] Sending {"on":true,"bri":203,"xy":[0.2210631504407662,0.7055816687044134]} via PUT to <my-deconz-instance>/api/<my-api-key>/lights/3/state
2025-06-19 20:43:09.176 [TRACE] [ernal.handler.DeconzBaseThingHandler] Result code=200, body=[{"success":{"/lights/3/state/on":true}},{"success":{"/lights/3/state/bri":203}},{"success":{"/lights/3/state/xy":[0.2210631504407662,0.7055816687044134]}}]
2025-06-19 20:43:09.182 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":203,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2211,0.7056]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.183 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=203, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2211, 0.7056], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:09.184 [TRACE] [z.internal.handler.LightThingHandler] Ignoring differing update after last command until 1750358589426
2025-06-19 20:43:09.329 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":87,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2211,0.7056]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.330 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=87, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2211, 0.7056], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:09.331 [TRACE] [z.internal.handler.LightThingHandler] Ignoring differing update after last command until 1750358589426
2025-06-19 20:43:09.431 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":87,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2375,0.7056]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.434 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=87, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2375, 0.7056], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:09.438 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":87,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2375,0.6711]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.439 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=87, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2375, 0.6711], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:09.555 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":87,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2302,0.6711]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.556 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=87, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2302, 0.6711], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:09.581 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":158,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2302,0.6711]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:09.582 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=158, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2302, 0.6711], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}
2025-06-19 20:43:11.551 [TRACE] [nternal.netutils.WebSocketConnection] Websocket$1750261981393-2 received raw data: {"e":"changed","id":"3","r":"lights","state":{"alert":"none","bri":158,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2302,0.6822]},"t":"event","uniqueid":"<id of my light>"}
2025-06-19 20:43:11.553 [TRACE] [z.internal.handler.LightThingHandler] deconz:extendedcolorlight:homeserver:Light_LED_LivingRoom received LightMessage{hascolor=null, ctmax=null, ctmin=null, type=null, state=LightState{reachable=true, on=true, bri=158, alert='none', colormode='xy', effect='none', effectSpeed=null, ontime=null, hue=21504, sat=254, ct=500, xy=[0.2302, 0.6822], transitiontime=null}, e='changed', r='LIGHTS', t='event', id='3', manufacturername='', modelid='', name='', swversion='', ep='', uniqueid='<id of my light>'}

I am running 4.3.5 Release Build, so it should not be anything that has been added in 5.0. Also it is seemingly not related to color temperature. I assume it would be the first linked PR then.

[andrewfg]

Just a quick response: it looks like the "Ignoring differing update" indicates the cause of the error. But its not (yet) obvious why..

I will look deeper tomorrow..

[Owlbertz]

Thanks. Just looked into my bridge config, but besides host and API key the only setting present is timeout=5000.

[andrewfg]

Yes. I think there is definitely a bug in the binding.

1. your initial command sends the following state {"on":true,"bri":203,"xy":[0.2210631504407662,0.7055816687044134]}
2. and the thing responds with {"alert":"none","bri":203,"colormode":"xy","ct":500,"effect":"none","hue":21504,"on":true,"reachable":true,"sat":254,"xy":[0.2211,0.7056]}
3. however the binding's call to equalsIgnoreNull() returns 'false' and therefore it wrongly ignores the updated value.

The equalsIgnoreNull() SHOULD be comparing the on state (which is indeed the same on), the brightness (which is also the same 203), and the xy values (which are actually the same to within four places of decimals, but the binding is wrongly concluding them to not be the same), plus it also WRONGLY comparing the hue and the saturation (which it must NOT check since the lamp is operating in color XY mode).

=> pinging @J-N-K can you please advise?

    public boolean equalsIgnoreNull(LightState o) {
        return equalsIgnoreNull(on, o.on) && equalsIgnoreNull(bri, o.bri) && equalsIgnoreNull(hue, o.hue)
                && equalsIgnoreNull(sat, o.sat) && ((xy != null && o.xy != null) ? Arrays.equals(xy, o.xy) : true);
    }

[andrewfg]

Suggested change as follows. @J-N-K I will create a PR if you think this is the correct approach.

// LightState
    public boolean equalsIgnoreNull(LightState other) {
        boolean colorsEqual = false;
        if ("ct".equals(this.colormode) || "ct".equals(other.colormode)) {
            colorsEqual = equalsIgnoreNull(this.ct, other.ct);
        } else if ("hs".equals(this.colormode) || "hs".equals(other.colormode)) {
            colorsEqual = equalsIgnoreNull(this.hue, other.hue) && equalsIgnoreNull(this.sat, other.sat);
        } else if ("xy".equals(this.colormode) || "xy".equals(other.colormode)) {
            colorsEqual = this.xy == null || other.xy == null
                    || this.xy instanceof double[] thisXY && thisXY.length > 1 && other.xy instanceof double[] otherXY
                            && otherXY.length > 1 && Math.abs(thisXY[0] - otherXY[0]) < 0.01 && Math.abs(thisXY[1] - otherXY[1]) < 0.01;
        }
        return colorsEqual && equalsIgnoreNull(this.on, other.on) && equalsIgnoreNull(this.bri, other.bri);
    }

[Owlbertz]

Thanks for the fast resolution. Will this be also included in some 4.3.X release or do I need to switch to the latest 5.X.X milestone?

[andrewfg]

Probably you need to move to 5.0 .. there should be a release in mid July.