zwave-js / node-zwave-js

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

Nortek/GoControl PD300EMZ5-1 parameters 7 & 8 stuck similar to zwave-js/node-zwave-js#5271. #6118

Closed windsurferwill closed 1 year ago

windsurferwill commented 1 year ago

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?

NO, my problem is NOT within Home Assistant or the ZWave JS integration

Is your problem within Z-Wave JS UI (formerly ZwaveJS2MQTT)?

NO, my problem is NOT within Z-Wave JS UI

Checklist

Describe the bug

What causes the bug?

What do you observe? Change parameters 7 or 8 in the zwave device config menu. They change but when exiting the UI and coming back in they didn't stick. kpine had me submit diagnostic logs which show the parameter size sent vs expected don't match See https://github.com/zwave-js/node-zwave-js/issues/5271. Using Developer Tools to set the state works.

What did you expect to happen?

Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '...'
  3. Scroll down to '...'
  4. See error

Device information

Manufacturer: nortek / goControl Model name: PD300EMZ5-1 Node ID in your network: 12

How are you using node-zwave-js?

Which branches or versions?

version:Driver Version: 11.6.0 Server Version: 1.30.0 node-zwave-js branch: zwave-js-ui branch:

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

Don't know, this is a new device

If yes, where did it work?

No response

Attach Driver Logfile

zwave_js (1).log

zwave-js-bot commented 1 year ago

👋 Hey @windsurferwill!

It looks like you attached a logfile, but its filename doesn't look like it a driver log that came from Z-Wave JS. Please make sure you upload the correct one.

AlCalzone commented 1 year ago

Looks like I made a change in the wrong parameter in that PR (7 instead of 8). #6124 should fix it.