Koenkk / zigbee2mqtt

Zigbee 🐝 to MQTT bridge πŸŒ‰, get rid of your proprietary Zigbee bridges πŸ”¨
https://www.zigbee2mqtt.io
GNU General Public License v3.0
12.22k stars 1.69k forks source link

State change does not work when switch bound to a light group #19592

Open Diaoul opened 1 year ago

Diaoul commented 1 year ago

What happened?

I'm using a group binding to control a Zigbee light (Paul Neuhaus NLG-TW) from a zigbee switch (Legrand 067773) instead of direct binding as described in the binding documentation.

From the documentation:

If your devices do not support reporting put the device in a group and bind the remote to the group instead of directly to the device. This will make Zigbee2MQTT poll the device for updates when the bound remote controls the device. To minimize traffic this has not been enabled for all devices. If this does not work please create an issue for it here

In my case, I don't think the polling works: the state is not updated

What did you expect to happen?

I expected that the state would be updated

How to reproduce it (minimal and precise)

  1. Put the light in a dedicated group
  2. Bind the switch to the group on OnOff and LevelCtrl
  3. Press the button of the switch
  4. Observe that state is not update in Zigbee2MQTT

Zigbee2MQTT version

1.33.2-dev commit: be48d54

Adapter firmware version

20230507

Adapter

tube_zb_gw_cc2652p2_poe

Debug log

When I press the button on the switch:

debug 2023-11-05 22:59:54Received Zigbee message from 'Living Room Entrance Light Switch', type 'commandCheckIn', cluster 'genPollCtrl', data '{}' from endpoint 1 with groupID 0
debug 2023-11-05 22:59:54Received Zigbee message from 'Living Room Entrance Light Switch', type 'commandToggle', cluster 'genOnOff', data '{}' from endpoint 1 with groupID 8
info 2023-11-05 22:59:54MQTT publish: topic 'zigbee2mqtt/Living Room Entrance Light Switch', payload '{"action":"toggle","action_group":8,"battery":80,"brightness":0,"linkquality":54,"rate":255,"update":{"installed_version":2769411,"latest_version":4473347,"state":"available"},"update_available":null,"voltage":2900}'
info 2023-11-05 22:59:54MQTT publish: topic 'zigbee2mqtt/Living Room Entrance Light Switch/action', payload 'toggle'
debug 2023-11-05 22:59:54Received Zigbee message from 'Living Room Entrance Light Switch', type 'commandToggle', cluster 'genOnOff', data '{}' from endpoint 1 with groupID 0
debug 2023-11-05 22:59:54Device 'Living Room Entrance Light Switch' announced itself
info 2023-11-05 22:59:54MQTT publish: topic 'zigbee2mqtt/bridge/event', payload '{"data":{"friendly_name":"Living Room Entrance Light Switch","ieee_address":"0x0004740000c67791"},"type":"device_announce"}'
info 2023-11-05 22:59:54MQTT publish: topic 'zigbee2mqtt/Living Room Entrance Light Switch', payload '{"action":"toggle","battery":80,"brightness":0,"linkquality":105,"rate":255,"update":{"installed_version":2769411,"latest_version":4473347,"state":"available"},"update_available":null,"voltage":2900}'
info 2023-11-05 22:59:54MQTT publish: topic 'zigbee2mqtt/Living Room Entrance Light Switch/action', payload 'toggle'
debug 2023-11-05 22:59:55Received Zigbee message from 'Living Room Entrance Light Switch', type 'readResponse', cluster 'genPowerCfg', data '{"batteryVoltage":30}' from endpoint 1 with groupID 0
info 2023-11-05 22:59:55MQTT publish: topic 'zigbee2mqtt/Living Room Entrance Light Switch', payload '{"battery":100,"brightness":0,"linkquality":102,"rate":255,"update":{"installed_version":2769411,"latest_version":4473347,"state":"available"},"update_available":null,"voltage":3000}'
debug 2023-11-05 22:59:56Retrieving state of 'Living Room Entrance Light Switch' after reconnect
miezie commented 8 months ago

Can confirm this behavior with a Philips Hue Wall Switch module and a Paul Neuhaus NLG-TW light

Diaoul commented 8 months ago

It seems to not affect Paul Neuhaus lights with a more recent firmware. However, I would love to have this fixed as there is currently no way to update the firmware on those lights...

github-actions[bot] commented 2 months ago

This issue is stale because it has been open 180 days with no activity. Remove stale label or comment or this will be closed in 30 days

Diaoul commented 2 months ago

This is still a problem. While there may be some hardware related or spec compliance issue on some lights, I think it would be great to be able to force trigger a refresh on commands impacting affected lights since this seem to work fine.