zwave-js / node-zwave-js

Z-Wave driver written entirely in JavaScript/TypeScript
https://zwave-js.github.io/node-zwave-js/
MIT License
741 stars 594 forks source link

Supervision Report not being sent by device: ZL-PD-100 #2538

Closed tseven closed 3 years ago

tseven commented 3 years ago

Checklist:

Describe the bug

Home Assistant's UI does not update to reflect the changes in state of the device when the state is changed in Node Red or via the physical button on the device.

Also when toggling the state from the Device UI the switch will revert back previous position after a few seconds.

It appears to not be sending a Supervision Report back.

Device information

Which device(s) is/are affected (make/model)?
ZL-PD-100 by zwaveproducts.com Firmware: 50.3 https://products.z-wavealliance.org/products/2809?selectedFrequencyId=-1 https://cdn.shopify.com/s/files/1/0067/9814/7669/files/ZL-PD-100_Users_Guide.pdf?953

What are the node IDs? Node ID: 7

Did you change anything?

Did this use to work before?

How are you using node-zwave-js

To Reproduce

Steps to reproduce the behavior:

  1. Go to 'Configruation' --> Devices
  2. Click on 'Dimmer device'
  3. Trigger the device to be on or off from Node Red or from physical button
  4. UI in HA is not updated.
  5. When toggling the state from the Device UI the switch will revert back previous position after a few seconds.

Additional context

Add any other context about the problem here.

Logfile: z-wave-silly.log

blhoward2 commented 3 years ago

From Slack, this device worked under OZW.

From the log, this happens each time. @AlCalzone I'm assuming this is another faulty supervision implementation? If you agree I PRd in the compat flag.

2021-05-04T22:53:26.298Z DRIVER » [Node 007] [REQ] [SendData]
                                  │ transmit options: 0x25
                                  │ callback id:      23
                                  └─[SupervisionCCGet]
                                    │ session id:      22
                                    │ request updates: true
                                    └─[MultilevelSwitchCCSet]
                                        target value: 0
2021-05-04T22:53:26.303Z SERIAL « [ACK]                                                                   (0x06)
2021-05-04T22:53:26.306Z SERIAL « 0x0104011301e8                                                       (6 bytes)
2021-05-04T22:53:26.307Z SERIAL » [ACK]                                                                   (0x06)
2021-05-04T22:53:26.308Z DRIVER « [RES] [SendData]
                                    was sent: true
2021-05-04T22:53:26.321Z SERIAL « 0x010500131700fe                                                     (7 bytes)
2021-05-04T22:53:26.322Z SERIAL » [ACK]                                                                   (0x06)
2021-05-04T22:53:26.324Z DRIVER « [REQ] [SendData]
                                    callback id:     23
                                    transmit status: OK
2021-05-04T22:53:26.336Z SERIAL « 0x010b00040007056c021600008a                                        (13 bytes)
2021-05-04T22:53:26.337Z SERIAL » [ACK]                                                                   (0x06)
2021-05-04T22:53:26.339Z DRIVER « [Node 007] [REQ] [ApplicationCommand]
                                  └─[SupervisionCCReport]
                                      session id:          22
                                      more updates follow: false
                                      status:              NoSupport
                                      duration:            [Duration: 0 seconds]
AlCalzone commented 3 years ago

status: NoSupport

Very likely. It obviously supports Supervision, but does not support supervision-encapsulated Multilevel Switch CC, which is a no-no.