-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Ubisys D1/S2 errors with Cluster with name 'manuSpecificUbisysDeviceSetup' does not exist
#22610
Ubisys D1/S2 errors with Cluster with name 'manuSpecificUbisysDeviceSetup' does not exist
#22610
Comments
@Koenkk could this be the race again were the customClusters are not yet added ? The D1 should have it, as it has the correct extend: https://github.com/Koenkk/zigbee-herdsman-converters/blob/f5ed0bfb5508a28efe507fff8d2578f39c0db4f1/src/devices/ubisys.ts#L832 |
Cluster with name 'manuSpecificUbisysDeviceSetup' does not exist
Cluster with name 'manuSpecificUbisysDeviceSetup' does not exist
@sjorge I don't think so, it's happening even 20 secs after z2m startup. |
So that would mean they don't get added at all then, somehow 🤔 |
Can I help with more debugging info? I have ~30 Ubisys devices lying around :) When running const {deviceEndpoints, identify, onOff, electricityMeter, commandsOnOff, commandsLevelCtrl, commandsColorCtrl} = require('zigbee-herdsman-converters/lib/modernExtend');
const definition = {
zigbeeModel: ['D1 (5503)'],
model: 'D1 (5503)',
vendor: 'ubisys',
description: 'Automatically generated definition',
extend: [deviceEndpoints({"endpoints":{"1":1,"2":2,"3":3,"4":4,"232":232}}), identify(), onOff({"powerOnBehavior":false}), electricityMeter(), commandsOnOff({"endpointNames":["2","3"]}), commandsLevelCtrl({"endpointNames":["2","3"]}), commandsColorCtrl({"endpointNames":["2","3"]})],
meta: {"multiEndpoint":true},
};
module.exports = definition; Both |
It matches on this line https://github.com/Koenkk/zigbee-herdsman-converters/blob/010319fc669fbdc2d7ef9ad3aa12ee4f3c540a0e/src/devices/ubisys.ts#L763 The D1 and R0 are the only devices I don't have and I can't replicate with my S1/S2/J1 which get the clusters extended the same way. I'm not sure what is wrong, my guess would be the modernExtend somehow not adding the cluster. @Koenkk is there and easy way to check on a running instance? |
I'm able to replicate on latest dev, it was fine right after i merged the change though 🤷 I won't have time this weekend to dig further, but should have some time on monday. I don't immediately see what would break it. I wonder if it somehow has to do with not setting a |
@Koenkk I have a theory: We pass So the 3rd argument would be |
@adriaandotcom Seems my theory from yesterday was correct, Koenkk/zigbee-herdsman-converters#7532 should fix this. |
Amazing. Can I test with the dev branch? |
It got merged into zhc, and there has been a zhc release, so I think it should be in dev already. |
What happened?
When trying to set config with (
zigbee2mqtt/0x001fee0000008009/set
) this payload:The same happens with this config (which I copied from an already set up Ubisys D1)
I get:
It didn't previously happen for 2 other D1 devices I have (the other devices did error sometimes - don't have the logs anymore - so I upgraded z2m to the latest version to prevent submitting issues of an older version). I tried with a brand-new device, same issue. Tested with an S2 as well, same issue.
What did you expect to happen?
It would allow me to use
configure_device_setup
without erorrs.How to reproduce it (minimal and precise)
configure_device_setup
commandZigbee2MQTT versions
Adapter firmware version
0x26580700
Adapter
ConBee II
Setup
Add-on on Home Assistant OS on Intel NUC
Debug log
Found some more debug info:
Maybe this commit in Koenkk/zigbee-herdsman-converters
src/devices/ubisys.ts
is related?The text was updated successfully, but these errors were encountered: