zwave-js / zwave-js-ui

Full featured Z-Wave Control Panel UI and MQTT gateway. Built using Nodejs, and Vue/Vuetify
https://zwave-js.github.io/zwave-js-ui
MIT License
984 stars 201 forks source link

[question] 7.x changes behavior or Aeotec door/window sensors (ZWA187 and ZWA008) #2638

Closed drizuid closed 2 years ago

drizuid commented 2 years ago

Hello, I just finished running through some tests to verify my findings. I tested a Zooz S2 ZST10 500 and Zooz S2 ZST10 700 stick with zwavejs2mqtt 6.16 and 7.x (both 7.0 and 7.1).

In my testing, with 6.x 5-113-0-Access Control-Door state would reflect [23] Window/door is closed or [22] Window/door is open. In Home assistant, this would also expose access_control_door_open which had an on or off state to home assistant (core). Moving to 7.x now gives access_control_door_state which has a 22 or 23 Door/window is closed state.

Oddly enough, 22 under 7.x never shows Window/door is open in the zwavejs2mqtt web ui, while in 6.x it would show the words, not just the numeric state.

My question is, is this the new and intended behavior for door/window sensors (the same issue presents on both my recessed door sensor 7 (ZW187) and my windows sensor 7 pro (ZWA008)). I am happy to provide some logs if anyone has any that will be the most useful. I have no problem flipping back and forth between 6.x and 7.x during testing. (though i'll stay on the zst 10 500 stick for now due to some quirky issues with the 700)

thanks in advance for all you guys do :)

drizuid commented 2 years ago

I believe I may have found my mistake. Under the binary sensorv2 there is now 5-48-0-Door/Window which will show false when closed and true when open but this is only for the recessed door sensor (ZWA187).

For the ZWA008, the only binary sensor that zwavejs2mqtt shows in 7.x is 6-48-0-Tilt which does NOT change from false to true when the window is open, though the 6-113-0-Access Control-Door state does show [22] when open and [23] Window/door is closed when closed. Of note, 6-112-0-2 shows that the binary sensor should also show open/closed events, but this sensor isn't present in 7.x.

I think it is worth noting that with 6.x home assistant would detect that these were doors/windows and icons would display as such. in 7.x we just see

image

this isn't a huge issue as I can simply change the icon, but my point is just how the changes have affected other things.

drizuid commented 2 years ago

to add some missing details, on 6.16.0 the binary sensor for the ZWA0008 is still just the tilt sensor, but the 8-113-0-Access Control-Door state shows [23] Window/Door is closed and [22] Window/Door is open this is really the only difference I'm seeing. (7.x shows [23] Window/Door is closed and [22])

kpine commented 2 years ago

Fixed in https://github.com/zwave-js/node-zwave-js/releases/tag/v10.1.0

Merge notification metadata for variables with multiple states

drizuid commented 2 years ago

Great news and I'm glad I'm not just crazy. I'm building the container from master now to test out! thank you!

bleh, ill have to come back to this later, building from master, which should have grabbed master for node-zwave-js based on https://github.com/zwave-js/zwavejs2mqtt/blob/e284ef9aadcc197d74748653abcd3eea4a4fcc87/docker/Dockerfile#L42 still built against zwavejs 10.0.4 rather than 10.1.0. I'll tinker more when I get home. I overwrote the ARG for now with ARG zwavejs=https://github.com/zwave-js/node-zwave-js/archive/refs/tags/v10.1.0.tar.gz and it's building

I'm clearly messing something up here, so I will reattack this tomorrow after some r&r.

robertsLando commented 2 years ago

Feel free to close this if it's fixed

Verbalinsurection commented 2 years ago

The v10.1.0 is in PR merged https://github.com/zwave-js/zwavejs2mqtt/pull/2643

drizuid commented 2 years ago

I'm not seeing a change in the behavior I've described with the updated node-zwave-js, but I can work around the issues manually or by going back to 6.16, so I am good to go on my question as it seems the changes are indeed by design. thank you all for all you do!

image