Koenkk / zigbee2mqtt

Zigbee 🐝 to MQTT bridge 🌉, get rid of your proprietary Zigbee bridges 🔨
https://www.zigbee2mqtt.io
GNU General Public License v3.0
11.98k stars 1.67k forks source link

[New device support]: TS0601 manufacturer _TZE284_nlrfgpny solar siren #24149

Open peroanjo opened 2 weeks ago

peroanjo commented 2 weeks ago

Link

https://es.aliexpress.com/item/1005007022197993.html

Database entry

{"id":11,"type":"EndDevice","ieeeAddr":"0xa4c138e5cdd4c8e4","nwkAddr":62092,"manufId":4417,"manufName":"_TZE284_nlrfgpny","powerSource":"Battery","modelId":"TS0601","epList":[1],"endpoints":{"1":{"profId":260,"epId":1,"devId":81,"inClusterList":[4,5,61184,0,60672],"outClusterList":[25,10],"clusters":{"genBasic":{"attributes":{"65487":14400,"65503":"�S�.f�S�.\u0012","65506":56,"65508":0,"appVersion":77,"stackVersion":0,"dateCode":"","modelId":"TS0601","manufacturerName":"_TZE284_nlrfgpny","powerSource":3,"zclVersion":3,"hwVersion":1}}},"binds":[],"configuredReportings":[],"meta":{}}},"appVersion":77,"stackVersion":0,"hwVersion":1,"dateCode":"","zclVersion":3,"interviewCompleted":true,"meta":{},"lastSeen":1727637030467}

Zigbee2MQTT version

1.40.1

Comments

Hello everyone,

It seems like this device is already supported as NAS-AB06B2 in the official z2m release. However _TZE284_nlrfgpny manufacturer is not registered.

Also, in the vendor page, there are comments of people saying it works with z2m. So it would seem like the seller recently modified the manufacturer id and is not recognized by z2m anymore.

Any help will be kindly appreciated!

External definition

const definition = {
    zigbeeModel: ['TS0601'],
    model: 'TS0601',
    vendor: '_TZE284_nlrfgpny',
    description: 'Automatically generated definition',
    extend: [],
    meta: {},
};

module.exports = definition;

What does/doesn't work with the external definition?

I tried using an external definition I found in #19827. However I couldn't get it to work...

averter commented 2 weeks ago

Hello, indeed I am experiencing the same issue with a siren from the same vendor. Thanks for mentioning issue #19077 as it might be relevant to some extent.

averter commented 2 weeks ago

Recently updated to Zigbee2MQTT version 1.40.2 commit: e06848db

Coordinator type zStack3x0

Coordinator revision 20230507

Coordinator IEEE Address 0x00124b002c3a6118

Frontend version 0.7.4

zigbee-herdsman-converters version 20.21.0

zigbee-herdsman version 2.1.3

and when I try to interview the siren there is a new log message:

2024-10-05 07:33:07zh:controller:device: Device '0xa4c1382f18ffcfc2' is only compliant to revision '21' of the ZigBee specification (current revision: 23).

Any thoughts?

peroanjo commented 1 week ago

For me I can interview the siren just fine and I won’t see any errors. Are you getting this error consistently? I saw the issue with the new manufacturer has also been mentioned in https://github.com/Koenkk/zigbee2mqtt/discussions/17325#discussioncomment-10803048 #17325

averter commented 1 week ago

For me I can interview the siren just fine and I won’t see any errors. Are you getting this error consistently? I saw the issue with the new manufacturer has also been mentioned in #17325 (comment) #17325

Based on the comment in #17325 could it be as simple as adding _TZE284_nlrfgpny to this line? As in

fingerprint: tuya.fingerprint('TS0601', ['_TZE200_t1blo2bj','_TZE284_nlrfgpny', '_TZE204_t1blo2bj', '_TZE204_q76rtoa9']),

peroanjo commented 1 week ago

I think so, yes. But I haven’t been able to test it yet, as I am using the HassOS addon and don’t have access to the z2m internal files. Are you able to modify it?

averter commented 1 week ago

Bingo! image

I didn't even had to re-interview; simply had to restart zigbee2mqtt. Specifically, I'm using docker on a raspberry pi and had to change the above file, which is located in /opt/zigbee2mqtt/node_modules/zigbee-herdsman-converters/devices/neo.js I'll create a PR as soon as possible. Thanks for directing me to the comment which provided the solution :-)

peroanjo commented 1 week ago

Nice! Thanks for testing it, do you also see issues when trying to modify the alarm melody or alarm mode of the siren?

averter commented 1 week ago

Can't test those atm as I am not at home. I'd also note that the battery hasn't shown up, which is known as a long-standing issue.

gregoriusus commented 1 week ago

Just for info. For me everything is working. Battery is shown, changing alarm mode is working. Melody haven't tested yet. Maybe it also depends on manufacturer. image

averter commented 1 week ago

@gregoriusus Interesting...just to confirm, is your manufacturer _TZE284_nlrfgpny?

gregoriusus commented 1 week ago

Hmm, no, I have: image

averter commented 1 week ago

@gregoriusus ok then it's not applicable to our case since it's not the same manufacturer. @peroanjo I've now fully tested the siren and the alarm melody and alarm mode work perfectly. The only thing which does not work is the Battery SoC.

averter commented 1 week ago

There seems to be an active development on the battery issue though. See https://github.com/Koenkk/zigbee2mqtt/issues/19827

peroanjo commented 1 week ago

@averter that’s good news! I have switched to the dev branch and the siren got recognized perfectly, however I haven’t been able to test it to be working correctly as my siren ran out of battery and I don’t have physical access to it for now. Thank you for your insight, I think we can then close this issue as the battery problem seems to be a general issue with this model.