-
Notifications
You must be signed in to change notification settings - Fork 11
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
Connection reset RW432MP panel #61
Comments
This version of the library doesn’t support Lightsys+. Try @markxroberts library (sorry - self promotion). |
And maybe, if you are not using the proxy mode, consider using the official
home assistant integration with local mode :
https://www.home-assistant.io/integrations/risco/
Johann
Le jeu. 25 janv. 2024 à 14:09, Mark Roberts ***@***.***> a
écrit :
… This version of the library doesn’t support Lightsys+. Try @markxroberts
<https://github.com/markxroberts> library (sorry - self promotion).
—
Reply to this email directly, view it on GitHub
<#61 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAECPHMG2ZAWJCDB7EYJM33YQJKPFAVCNFSM6AAAAABCKMG6IKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGE4TAOBTHE>
.
You are receiving this because you are subscribed to this thread.Message
ID: ***@***.***>
|
I am getting the same error in your
I have been using that recently but it stopped working and has been quite flaky for me. Wanted to try something different :) |
In fact the library should support for RW432 Panel (I don't know if the
"MP" suffix is relevant)
Check the code here :
https://github.com/vanackej/risco-lan-bridge/blob/6a36e1430ba98e95ad44896e3ef27f4934f65e11/src/RiscoComm.ts#L251
Your logs stop even before the panel model detection. So I think your
issue is not related to the panel model.
If your panel is not supported, you should have an exception with the panel
model as reported by the panel it-self after connection :
https://github.com/vanackej/risco-lan-bridge/blob/6a36e1430ba98e95ad44896e3ef27f4934f65e11/src/RiscoComm.ts#L343
Le jeu. 25 janv. 2024 à 14:57, Maarten Weyns ***@***.***> a
écrit :
… This version of the library doesn’t support Lightsys+. Try @markxroberts
<https://github.com/markxroberts> library (sorry - self promotion).
I am getting the same error in your risco-mqtt-local fork. Should I be
changing my configuration?
consider using the official home assistant integration with local mode
I have been using that recently but it stopped working and has been quite
flaky for me. Wanted to try something different :)
—
Reply to this email directly, view it on GitHub
<#61 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAECPHMPSS4ZR5DN53YBZJ3YQJQFPAVCNFSM6AAAAABCKMG6IKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGI3TANRYGA>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
In that case I am unsure what is causing the error. Have you ever seen this before? Is it possible the pin changed out of itself? 🤷🏼♂️ |
Really no idea, sorry.
The best would be to checkout and run the library locally with a debugger
attached, set breakpoints and see what happens, but it requires advanced
developer skills (I don't know if you are a developer or not...).
Le jeu. 25 janv. 2024 à 15:47, Maarten Weyns ***@***.***> a
écrit :
… In that case I am unsure what is causing the error. Have you ever seen
this before? Is it possible the pin changed out of itself? 🤷🏼♂️
—
Reply to this email directly, view it on GitHub
<#61 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAECPHMDMJ4C4GDUOYKDE6TYQJWBJAVCNFSM6AAAAABCKMG6IKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJQGM2TQNRVGE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Hello @vanackej , @maartenweyns : For the 432mp lightsys+ , I add some code on the risco-lan-bridge : https://github.com/needles77/risco-lan-bridge . I will continue to work on it next week. Bye |
If you have working code to fix this issue I’m happy to try to include in my fork of the library if you open a pull request. |
sorry @markxroberts I hadn't read your comment. When I worked and try to debug de risco-lan-bridge last years with home bridge, I had some error with CRC and the delay of reconnection. I fixed this with :
Now, I will try your fork for the next days, and I'll keep you informed. Bye |
I've published a new version with both of these configurable. |
Good to see some developments in this issue! I have tried the new version @markxroberts released, with the adapted configuration from @needles77. It however did not resolve the issue I am having. I took a gander at the traffic between my host and the alarm panel in Wireshark and I do indeed see that the alarm panel replies with the reset bit set after the RMT command. No idea why that is happening I attached the new logs below (they look slightly different this time):
|
What happens if you set |
I am seeing the same behavior with that configuration |
Describe the bug
The HomeAssistant addon is unable to connect to my RW432MP panel (connection is being reset)
Configuration
Logs
Debug logs:
where
INT_ARRAY
is an arbitrary-looking array of integers (removed because unsure if it's sensitive data).Command log:
Additional context
No additional context
The text was updated successfully, but these errors were encountered: