-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Group with one device, two endpoints not working with latest dev
branch
#24792
Comments
Trying to remove one of them fails, possibly because the command seems to target endpoint one instead of the actual group member endpoint, which might be another multi-endpoint frontend issue:
|
I've tested it with 2.0.0 branch (including frontend), with a TS0002, I can't reproduce. Removing works as well.
#24432 changes the logic quite a bit though, so, I guess it could have removed the problem? |
@Nerivec It's possible the problem is connected to the fact that the Hue Twilight has actually three endpoints ( |
@mundschenk-at could you provide the debug log when controlling works with 1.40.2 and not with 1.41.0? |
@Koenkk The difference is between 1.41.0
dev
|
@Koenkk, @Nerivec I've looked over the changes between I can set up more detailed debugging if you tell me where to look - I am completely stumped at this point (it does not help that the frontend is buggy with multi-endpoint devices, excluding/including worked around those issues for me). |
@mundschenk-at could you check with the current |
@Koenkk: Unfortunately, it looks pretty much the same:
|
For groups the list of toZigbee converter is determined here: zigbee2mqtt/lib/extension/publish.ts Line 307 in acba533
console.log() here to debug this?
|
With additional
@Koenkk What looks weird to me is
as |
I have linked so e additional reports. |
Hi, I have the same problem maybe i can add some more info to the matter :) this is my version 1.42.0 commit: 861cba6 and it is running on a container basically i have many groups with different devices in them. but the problem is manifested only on one this is the group definition in the conf:
The problem manifest with the group: if i send the command:
to the topic: i got this error interesting if i send the same command to the group |
@mundschenk-at could you try if the following change fixes your issue? https://github.com/Koenkk/zigbee2mqtt/pull/25156/files |
@Koenkk Unfortunately, no change in behavior. |
Added extra debug logging to https://github.com/Koenkk/zigbee2mqtt/pull/25156/files, can you provide the debug log with this? See this on how to enable debug logging. |
@Koenkk Sorry for the delay, here's the log:
|
Found & fixed the issue! Changes will be available in the dev branch in a few hours from now. Note that the dev branch currently contains breaking changes, see #24198 |
What happened?
The Hue Twilight has to light endpoints (
11
and12
). Since adding support for it it in zhc, I've had a group for it to switch both lights on and off simultaneously. Withdev
from immediately after the release of1.41.0
, this worked fine. However, two days ago I updated to the latestdev
(after the merge of myevent
PRs). Now that group does not work anymore. Groups containing several different devices seem to be fine.With groups only containing endpoints from a single device, the result when trying to toggle them on looks like this:
What did you expect to happen?
The group should work normally.
How to reproduce it (minimal and precise)
No response
Zigbee2MQTT version
1.41.0-dev
Adapter firmware version
20240710
Adapter
zStack3x0
Setup
Plain
Debug log
No response
The text was updated successfully, but these errors were encountered: