-
Notifications
You must be signed in to change notification settings - Fork 68
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
Range error on shim-array #481
Comments
Something weird is happening here. The package collections and shim-array are not a dependency of homebridge-alexa. Did you make any recent changes or are you running something non-standard ? Have you tried removing the plugin and re-installing ? |
The issue seems to arise whenever I set routines to true in my config. |
Turned out to be an issue with another plugin I was using. Once I got rid of that, I was able to enable routines and everything runs smoothly. |
Which plugin ?
… On Dec 25, 2021, at 2:38 PM, aaronkbutler ***@***.***> wrote:
Turned out to be an issue with another plugin I was using. Once I got rid of that, I was able to enable routines and everything runs smoothly.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.
|
homebridge-myq |
Hello @NorthernMan54, it seems the package collections is used by the packages mqtt you have dependencies on. I haven't got the problem myself, but I've encountered it on users using other plugins than the one mentioned by @aaronkbutler. I ask the plugin list to the users who had the problem and let you know. |
@NebzHB I went and looked at the collections package, and they have this issue mentioned need to read it in detail to see what they are saying |
It seems they fixed it in march (with a quick overview of the issue)... Edit : my 2 users are on rpi 3b with nodejs 14. I can't reproduce on a debian vm |
Hello again @NorthernMan54, I see you are using a version 2.x of mqtt. And they are version 4.x, could it be that ? i have the plugin list of one of my users : |
I have a major renovation planned for this winter, and that is one of many items on the backlog. |
have a third user with the problem : Rpi3+/buster, plugin list : |
i can reproduce on my debian vm ! |
@NebzHB Well I was able to recreate the issue by installing myq into my development environment. So far I have tried updating the mqtt package version to the latest version, nope, and then tried the same version as myq, same. I then went and implemented the collections fix to homebridge-alexa ie added Am thinking the only true fix is to have collections package publish the release. |
strange.. none of my users uses myq... on my own system, i fixed it with adding if(!Array.from) {} wrap around the function re-defining Array.from in collections (as proposed in one of their PR (#montagejs/collections#212)) see https://github.com/NebzHB/collections/blob/master/shim-array.js#L33 . I see in original Issue of Collections that camera-ffmpeg does have the issue too (#homebridge-plugins/homebridge-camera-ffmpeg#1240) and ring throw the issue away too #dgreif/ring#844 ... it seems the error message sticks to the first plugin using mqtt... There will be hot-potato game in that case ;) |
@aaronkbutler if you can try updating to the latest version, I implemented a workaround for the issue. |
your fix works for me :) |
Tks closing this |
Thanks for working on this! Works great now. |
I’m getting an error in the Homebridge console every few minutes which causes Homebridge to restart.
The text was updated successfully, but these errors were encountered: