Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inovelli VZW31-SN Firmware 1.2 New Parameters #6273

Open
3 of 11 tasks
DAVe3283 opened this issue Sep 11, 2023 · 5 comments
Open
3 of 11 tasks

Inovelli VZW31-SN Firmware 1.2 New Parameters #6273

DAVe3283 opened this issue Sep 11, 2023 · 5 comments
Labels
config ⚙ Configuration issues or updates

Comments

@DAVe3283
Copy link

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?

Firmware 1.2 (https://community.inovelli.com/t/red-series-2-1-firmware-changelog-vzw31-sn/14579) was recently promoted to stable (zwave-js/firmware-updates#83). It adds 2 new parameters (temperature and overheat) and an option from 1.1 (leading or trailing edge dimming). These options don't have config values in this repo.

What do you observe?

The temperature parameter shows up, but it is not being reported as a measurement, and the temperature units are missing. This causes Home Assistant to display it is a string, instead of a number and recording the history in a graph.
image

Switching between leading and trailing edge is also somewhat ambiguous.
image
On other parameters, the values would be labeled in a drop-down, e.g. [0] Leading Edge, [1] Trailing Edge.

What did you expect to happen?

The new parameters should show up with the correct units as the other parameters do. This would let the temperature show up as a history graph like other Z-Wave devices reporting a temperature have.

Steps to reproduce the behavior:

  1. Update a VZW-31SN to firmware 1.2
  2. Enable one of the new parameters (e.g. Internal Temperature) in Home Assistant
  3. Note how the parameter has no units and doesn't get a history graph.

Device information

Manufacturer: Inovelli
Model name: Red Series 2-1 Dimmer (VZW31-SN)
Node ID in your network: 9

How are you using node-zwave-js?

  • zwave-js-ui (formerly zwavejs2mqtt) Docker image (latest)
  • zwave-js-ui (formerly zwavejs2mqtt) Docker image (dev)
  • zwave-js-ui (formerly zwavejs2mqtt) Docker manually built (please specify branches)
  • ioBroker.zwave2 adapter (please specify version)
  • HomeAssistant zwave_js integration (please specify version)
  • pkg
  • node-red-contrib-zwave-js (please specify version, double click node to find out)
  • Manually built from GitHub (please specify branch)
  • Other (please describe)

Which branches or versions?

Z-Wave JS UI: 8.24.2
Z-Wave JS: 11.14.0
Home Assistant Add-On Version: 1.15.9

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

No, it never worked anywhere

If yes, where did it work?

No response

Attach Driver Logfile

Do you need a log of a re-interview to see the new parameters? Let me know what you need and I will provide it.

@zwave-js-bot
Copy link
Collaborator

👋 Hey @DAVe3283!

Thanks for opening an issue! It doesn't look like you provided a logfile though. While not strictly necessary for every issue, having a driver log is required to diagnose most issues.

Please consider uploading a logfile that captures your problem.

@InovelliUSA
Copy link

For P26 can we NOT have it as a selectable option in the GUI or have it read only? Changing this setting remotely can damage the device if an improper load is connected. It is ok to change it via combo button or to send it as a custom configuration parameter from the control panel:

image

Just want to be cautious that people don't willy nilly change it. The combo button is:

Hold down + press config 13x = trailing edge
Hold up + press config 13x = leading edge

@AlCalzone
Copy link
Member

Absolutely!

@AlCalzone
Copy link
Member

Parameter 26 added in #6391

@InovelliUSA
Copy link

@AlCalzone Thanks! I see the note you placed in the file.

@AlCalzone AlCalzone added the config ⚙ Configuration issues or updates label Oct 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
config ⚙ Configuration issues or updates
Projects
None yet
Development

No branches or pull requests

4 participants