search

aartse / athom.zipato

2 stars 9 forks source link

PIR motion sensor is added as "Alarmsirene (TEST)" #69

Closed danikrau closed 2 years ago

danikrau commented 3 years ago

Hello, I tried to add the Zipato PIR motion sensor (NE-NAS-PD01Z.EU) on Homey but it's always shown as "Alarmsirene (TEST)". Is it possible to fix this?

Screenshot_20210825-102152 Screenshot_20210825-102117

aartse commented 3 years ago

i'll change this for the next release. but i think the images are also not correct.

just to be sure, is this the device shown on this page: http://manuals-backend.z-wave.info/make.php?lang=en&sku=ne-nas-ab02z.eu&cert=ZC10-18096228&type=mini

danikrau commented 3 years ago

This should be the device I'm trying to add: https://www.zipato.com/wp-content/uploads/2017/06/ne-nas-pd01z-Zipato-PIR-Motion-Sensor-Data-Sheet-1.0.pdf Screenshot_20210825-103940

But for some reason it's always added as a siren (as shown in the images); I do not even have a siren so that's strange. By the way, it doesn't make a difference whether I add it as a PIR motion sensor or as a basic z-wave device; it always is added as a siren.

aartse commented 3 years ago

i think there is a mixup with product ids. some older issues addresses the opposite behaviour (#55). i added product id 4227 to NE-NAS-PD01Z.EU, so i hope that homey now adds the device as a PIR.

test version is 3.5.2.

danikrau commented 3 years ago

Just tested it with version 3.5.2. The PIR motion sensor is still added as a siren (it just says "Alarmsirene" now instead of "Alarmsirene (TEST)"). So is there maybe still a mixup with product ids?

danikrau commented 3 years ago

@aartse, could it be that the PIR motion sensor is still added as a siren because NE-NAS-AB02Z.EU (the siren) has 4227 (the PIR) as product id? Shouldn't there be a different product id in NE-NAS-AB02Z.EU (the one from the siren only)? [Please apologize in case I'm wrong, I'm still rather new to Homey and also the structure of the code.] Screenshot 2021-08-27 151435

danikrau commented 3 years ago

@aartse Is there any chance this can be fixed (see also my previous comments)? Also with the current test version 3.5.2 I'm unable to add my sensors as PIR motion sensors, so that it's not possible to use these with Homey. Any feedback or alternative solution would be very much appreciated. Thanks a lot in advance!

fhutter commented 2 years ago

@aartse I have 1:1 the same issue with the PIR sensor which is added as [Alarmsirene (TEST)"]

same IDs as danikrau. just device ID is 15 not 35 product ID 4227 manufac ID 305 Producttype ID 3

69

aartse commented 2 years ago

Since homey firmware 7.3 the following change has been applied:

[Z-Wave] Prefer selected driver when pairing a Z-Wave device

so I’ll check the beta firmware version to see if this change will fix this problem.

danikrau commented 2 years ago

With the Homey beta firmware 7.3 it's finally possible to add the PIR sensors. Thanks for the hint!

aartse commented 2 years ago

fixed in version 3.5.3 (and homey 7.3)