Closed prilly-dev closed 3 years ago
modbus documentation modbus source (message by IssueLinks)
Hey there @adamchengtkc, @janiversen, @vzahradnik, mind taking a look at this issue as its been labeled with an integration (modbus
) you are listed as a codeowner for? Thanks!
(message by CodeOwnersMention)
Do you really expect that your language motivates anyone to help you ?
First of all, instead of having a HELL, downgrade ! Others including myself do that all the time.
Secondly if you do not like the official modbus integration, do not use it, copy the 2021.3 modbus which you seem to like, and make a custom component, with the changes you want to have. That is the best way for you not to be affected with what you do not like.
Finally “breaking in all possible ways” is a rather imprecise error description, which are useless.
I prefer not to comment the actual problems I see, since you directly state you do not want us to mess with the modbus code.
I am decoupling myself from the issue and wish you luck finding a solution.
Hey you, now i think you should calm down here, it seems you have issues with alot of guys here.
First of, why the HELL are you messing with the OFFICAL WORKING modbus integration? Why havent you made a new integration for your modbus experiment?
You should have done the same with this integration as how zwave was fixed, parallell integration, whenbits stable we can switch. This way you dont mess up ha for everyone using modbus?
As i have stated before EVERYTHING was working perfect before you started messing with modbus, now you require alot og folks to fix this issues that has progressed for months,
Do you think this is the right way to introduce this kind of heavy changes when you know it will result in problems?
I hope you decouple from ha, not just modbus, i will reintroduce other for this issues. I dont know if you are familiar with the FORK of ha that is discussed recently, i hope the mess in this project can be solved so we can avoid this fragmentations.
@MartinHjelmare Are you looking at what is going on with modbus? And how this contributor is attacking ticket submitters? We cant have it this way, i know that there are alot of frustrations regarding how modbus has become. Why dont move this new contributions into a new integration?
Anyway, i hope that someone with controll of ha can put the breaks on here. This is NOT what HA needs right now.
Just for the record, I have been maintaining modbus since march 2020, so the modbus in 2021.3 contains quite large parts from me. We are a small team (of 2) that actively maintain modbus, and we have made it a custom to review each others work on top of the review from the maintainers. We have successfully solved many problems and I believe the modbus is in a better shape now than a year ago.
It is correct that modbus are haunted by a stability issue, related to device disconnecting, which frustrates me because I cannot reproduce it (I am testing with a modbus server, where I can simulate disconnect etc). I believe it is a timing issue so logs with many integrations does not help a lot, because the HA calls are not predictable.
I did not and do not want to attack you or anybody else. I made some serious suggestions in order to help you.
Just for the record, I have been maintaining modbus since march 2020, so the modbus in 2021.3 contains quite large parts from me. We are a small team (of 2) that actively maintain modbus, and we have made it a custom to review each others work on top of the review from the maintainers. We have successfully solved many problems and I believe the modbus is in a better shape now than a year ago.
It is correct that modbus are haunted by a stability issue, related to device disconnecting, which frustrates me because I cannot reproduce it (I am testing with a modbus server, where I can simulate disconnect etc). I believe it is a timing issue so logs with many integrations does not help a lot, because the HA calls are not predictable.
I did not and do not want to attack you or anybody else. I made some serious suggestions in order to help you.
Now we are moving in the right direction, first of i have to thank you and the other guy for your contribution to ha and modbus, good work.
It might be a timing issue, but to me it seems like multiple issues of some sort. I have noticed async is always a part of the problem.
I have a docker container running in a xen VM on my dell t630 home server connected to a systemair modbus iam module with tcp. I suspect i have the issue unfolding here, are there anything i can provide you with or be at assistant?
Yes I see multiple problems in your log, including a division by zero, that securely influences the overall performance.
The message
2021-05-04 18:04:48 ERROR (SyncWorker_26) [homeassistant.components.modbus.modbus] Pymodbus: Exception Response(131, 3, SlaveBusy)
Is a modbus message (look in the modbus standard for “exception response”) where your device tells it is busy. A typical cause for this problem is scan being too fast (scan_interval needs to have a larger value).
The old modbus did not use SCAN_INTERVAL correctly that was changed earlier first in climate/cover then later in all platforms (actually that part is not my code/idea). With SCAN_INTERVAL working correctly, the integration is a lot faster in responding than it used to be and that of course puts a higher demand on the devices.
I am looking for a debug log including pymodbus as described in the ha modbus documentation, from a system with a device disconnect problem, where the configuration only contains modbus (and lovelace to confirm if it becomes unresponsive), no templates no “real” integration. Such a configuration allows me to determine where pymodbus “hangs” and thus isolate the problem.
@prilly-dev Attacking an integration maintainer, or anyone else for that matter, like you did here is a violation of our Code of Conduct. It will not be accepted in our community.
If you want to continue interacting with our community, please stay civil and follow our Code of Conduct. Otherwise we're not interested in interacting with you, and will issue a permanent ban.
The problem
Modbus is causing HA to crash/halt/slowdown eventualy brings ha to a effective stop. this started as soon as modbus was modified. EVERYTHING WAS WORKING PERFECTLY FINE UNTIL 2021.03.
After the modbus havoc started i have seen absolutly all possible error messages HA can produce, after soon two months with this hell and it currently seems this is going to continue, it has to be stopped before the rest of HA is destroyed.
What is version of Home Assistant Core has the issue?
ALL VERSION AFTER 2021.03 including DEV
What was the last working version of Home Assistant Core?
2021.03
What type of installation are you running?
Home Assistant Container
Integration causing the issue
Modbus
Link to integration documentation on our website
https://www.home-assistant.io/integrations/modbus
Example YAML snippet
Anything in the logs that might be useful for us?
Additional information
currently running core-2021.5.0b7
Lovelace gui stops respondig, seems like webserver is crashed sometimes, disable modbus, everything fine!
All modbus related switches, sensors and so on is blinking unavailable/available sensor variables, but this is probably related to loglines: slavebusy, this is a new one, never seen that before, AND WHAT EVER YOU DO, DONT CLAIM THAT THE THE SLAVE HAD RESPONSPROBLEMS BEFORE MESSING WITH MODBUS CODE, that is 100% ruled out.
Regarding the new modbus config style, same problem exists on both formats, it even seems worse with the old style.