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

RangeError after trying to include vision ZD2102EU-5 door/window sensor #4335

Closed
3 of 11 tasks
VerbruggenBart opened this issue Mar 8, 2022 · 3 comments · Fixed by #4337
Closed
3 of 11 tasks

RangeError after trying to include vision ZD2102EU-5 door/window sensor #4335

VerbruggenBart opened this issue Mar 8, 2022 · 3 comments · Fixed by #4337
Assignees
Labels
bug Something isn't working
Projects

Comments

@VerbruggenBart
Copy link

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

YES, BUT a Home Assistant developer has told me to come here

Is your problem within ZWaveJS2MQTT?

YES, BUT a ZWaveJS2MQTT developer has told me to come here

Checklist

Describe the bug

I have trouble including the vision ZD2102EU-5, which led me to look at the log-files (pointed there by the home assistant community)

In the zwave js to mqtt add-on I noticed that the node (now node 4) did not seem to be interviewed properly, even after waking the device up manually for a significant amount of times (it only stays awake for 10secs, so I kept repeating this while eating).
However, it kept saying unkown manufacturer/unknown product and there is no sensor available, except once I got it to show the battery somehow.

Obviously I hoped the device would show up and I would have a binary sensor for the open/close status.

How I got to this point:

  1. I excluded the device from the old 'gateway'
  2. I started inclusion through the zwave js to mqtt interface
  3. I put in the battery in the sensor to get it into inclusion mode (in a second attempt I just opened the lid with the same effect)
  4. I waited
  5. I tried to wake up the device a lot
  6. I tried to re-interview
  7. I tried to get help from the home assistant community
  8. I started silly loggin
  9. I noticed the error below and got pointed here :)

The error that got me sent here:

2022-03-08T05:44:13.655Z DRIVER RangeError: The value of "offset" is out of range. It must be >= 0 and <= 14.
Received 15
at new NodeError (node:internal/errors:371:5)
at boundsError (node:internal/buffer:86:9)
at Buffer.readInt8 (node:internal/buffer:424:5)
at parseRSSI (/opt/node_modules/zwave-js/src/lib/controller/SendDataShared
.ts:91:22)
at new BridgeApplicationCommandRequest (/opt/node_modules/zwave-js/src/lib
/controller/BridgeApplicationCommandRequest.ts:81:24)
at Function.from (/opt/node_modules/zwave-js/src/lib/message/Message.ts:22
8:15)
at Driver.serialport_onData (/opt/node_modules/zwave-js/src/lib/driver/Dri
ver.ts:2135:18)
at SerialAPIParser.emit (node:events:520:28)
at SerialAPIParser.emit (node:domain:475:12)
at addChunk (node:internal/streams/readable:315:12)

Device information

Manufacturer: Vision
Model name: ZD2102EU-5
Node ID in your network: 4 (in first attempt it was node 3)

How are you using node-zwave-js?

  • zwavejs2mqtt Docker image (latest)
  • zwavejs2mqtt Docker image (dev)
  • 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:
Driver Version: 8.11.6
Server Version: 1.15.0

Z-wave to MQTT:
version: 0.35.2

Did you change anything?

no

If yes, what did you change?

No response

Did this work before?

Yes (please describe)

If yes, where did it work?

It worked with the fifthplay gateway, however fifthplay stopped its operation and I wanted to recycle some of the devices.
Obviously I excluded them from the fifthplay gateway first.

Attach Driver Logfile

zwavejs_2022-03-08.log
zwavejs_2022-03-07.log

@zwave-js-bot zwave-js-bot added this to Needs triage in Triage Mar 8, 2022
@AlCalzone
Copy link
Member

Sounds like a bug in the serial parser with a probably easy fix. Will try to get this fixed tonight.

@AlCalzone AlCalzone moved this from Needs triage to High priority in Triage Mar 8, 2022
@AlCalzone AlCalzone self-assigned this Mar 8, 2022
@AlCalzone AlCalzone added the bug Something isn't working label Mar 8, 2022
Triage automation moved this from High priority to Closed Mar 8, 2022
@AlCalzone
Copy link
Member

Fixed on master and 8.11.9 (which is going to be released in a few minutes)

@VerbruggenBart
Copy link
Author

Wow, what a quick response!
Wonderful! 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
No open projects
Triage
Closed
Development

Successfully merging a pull request may close this issue.

2 participants