-
-
Notifications
You must be signed in to change notification settings - Fork 633
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
[bug] Meter report errors #1316
Comments
What's the difference to the zwavejs2mqtt-dev image? You could try and run zwavejs2mqtt without Docker, to make sure the build process of that isn't the problem. |
I can try that tonight. |
Hmm, I'm getting this on my bridge controller branch too. After a half dozen flips between on/off, it got an unhandled exception (with the same GATEWAY error) and the whole server shut down |
So what’s breaking the image then? It’s in every image I’ve built off fix#23/master. |
My guess is this zwave-js/zwave-js-ui#181 but I'm not sure. |
@AlCalzone I do not get this error in a container built with both zwave-js/zwave-js-ui#181 and zwave-js/zwave-js-ui#84. Since it exists when built with only fix#23, zwave-js/zwave-js-ui#181 must fix it. |
I’ve also confirmed that this is fixed when zwave-js/zwave-js-ui#181 is built with the config directory properly linked (per my comment on that PR). As such, this seems to be an artifact. |
Describe the bug
All of my devices that check-in with energy reports throw an error under the node-zwave-js master branch with the zwavejs2mqtt fix#23 branch.
Master Branch
I get the same error message for all three values.
Under the zwavejs2mqtt-dev image
To Reproduce
Steps to reproduce the behavior:
Device information
I'm see this on multiple devices including Aeotec Switch 6s, Aeon Nano Dimmers, and an Aeon Microdimmer.
Logfile:
I can't generate better logfiles because logging does not work as described in the fix#23 PR.
The text was updated successfully, but these errors were encountered: