-
Notifications
You must be signed in to change notification settings - Fork 21
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
Lost connection to inverter after changing settings multiple times in quick succession, needs HA restart #192
Comments
I think I might have seen this as well using UDP... I put it down to the W610 not liking connection from multiple sources at the same time, but it might actually be this (with a slightly different message of course, because UDP). |
I've hit this a few times today, and I've had Wireshark open the entire time. Each time, the problem has been with the adapter -- we send a correct packet, but the adapter never responds. |
@canton7 yeah happened to me again last night, I only made a single change this time and it lost it, sounds like something in the underlying python libraries is getting messed up. For the average user it won’t happen very often and a restart fixes it so perhaps keep this on the back burner. |
I'm not sure if it can be pymodbus -- the packets it's sending out look fine. It's just that (in my case) the adapter wasn't responding |
Ahh ok, maybe the socket isn't being reset properly after the change and the adaptor is left hung - possibly not much you can do about this one. Next time it happens rather than restart i'll go and pull the ethernet cable out of the adaptor and see if that helps it tidy up. |
Well, I'm looking at UDP and the W610 -- there's no socket state there. |
It's entirely possible we're seeing different issues, of course! |
I noticed that the TCP connection to the inverter/adapter wasn't being properly closed when the integration was reloaded, which might be related: I can imagine lots of not-yet-timed-out TCP sockets causing the adapter/inverter some issues |
Ok, fingers crossed that's what it is - ironically last week I spent over 10 minutes going in / out of the config changing the number of reads, poll time etc.. and it didn't break once 😣 😏 |
From #40 (reply in thread):
The text was updated successfully, but these errors were encountered: