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

Older Z-Wave SDKs don't include the route speed in the GetPriorityRoute response #6616

Closed
4 of 7 tasks
AlCalzone opened this issue Jan 19, 2024 Discussed in #6608 · 0 comments · Fixed by #6746
Closed
4 of 7 tasks

Older Z-Wave SDKs don't include the route speed in the GetPriorityRoute response #6616

AlCalzone opened this issue Jan 19, 2024 Discussed in #6608 · 0 comments · Fixed by #6746
Assignees
Labels
bug Something isn't working

Comments

@AlCalzone
Copy link
Member

Discussed in #6608

Originally posted by raman325 January 15, 2024

Checklist

  • I have read and followed the above instructions
  • I have checked the troubleshooting section and my problem is not described there.
  • I have read the changelog and my problem was not mentioned there or the fix did not work.

Describe the issue

What is happening?

As reported in home-assistant/core#108021 the HA integration expects protocolDataRate to be provided in the lwr and nlwr statistics objects based on the typing: https://github.com/zwave-js/node-zwave-js/blob/master/packages/zwave-js/src/lib/node/NodeStatistics.ts#L56

So to me it sounds like either the typing is wrong or there is some state that statistics get put in that is incorrect.

What did you expect to happen instead?

I think that either the typing needs to be updated to make protocolDataRate optional or the logic that is leaving the object in a bad state is updated.

Steps to reproduce the behavior:

Hard to say since it is happening on initialization. It's also not a widespread error as this is the first report we've seen.

Anything else we should know?

The user mentioned that it was not an issue in HA 2023.9.X which was released on September 6th. Looking at the zwave-js-server release logs, that suggests that this was introduced in zwave-js 12.x. I did try to run 2023.9.3 using the latest node-zwave-js just to make sure it wasn't some sort of backwards compatibility issue, but I was able to bring up my network fine

cc @maheus

Software versions

Driver (node-zwave-js): 12.4.1
Z-Wave JS UI: 9.6.2.6e369a1

Home Assistant Z-Wave Integration: Home Assistant 2023.11.3

Home Assistant Z-Wave JS Addon: ...

ioBroker.zwave2 Adapter: ...

If you are using something non-standard, tell us here: ...

Device information

Unknown at this time

Manufacturer: ...
Model name: ...
Node ID: ...

Checklist

  • I made sure to provide a driver log on level debug.
  • The log includes a re-interview of the problematic device (if applicable).
  • The log includes the problematic interaction with the device (if applicable).
  • I provided the node ID of the problematic device (if applicable).

Upload Logfile

Waiting for the user to provide debug logs as they have stated they are worried about leaking personal information, but logging the issue anyway as a placeholder

@AlCalzone AlCalzone added the bug Something isn't working label Jan 19, 2024
@AlCalzone AlCalzone self-assigned this Jan 19, 2024
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
None yet
Development

Successfully merging a pull request may close this issue.

1 participant