modbus: Update for pymodbus 3.5.2; Support for pymodbus 2.x dropped #832
Conversation
Update auf Version 2.0.6
- Changes for pymodbus 3.5.2 - Support for pymodbis 2.x dropped
- Changes for pymodbus 3.5.2 - Support for pymodbis 2.x dropped
- Changes for pymodbus 3.5.2 - Support for pymodbis 2.x dropped
- Changes for pymodbus 3.5.2 - Support for pymodbis 2.x dropped
|
Every Plugin which had really breaking changes (as dropping support for version 2 here) had a copy of the previous version in a subfolder like |
msinn
changed the title
- Einbindung des Huawei SUN 2000 Wechslerichters per Modbus TCP - Einbindung des Huawei Smart Power Meter - Einbindung der Huawei LUNA 2000
Update auf Version 0.1.0: - Es können nun auch mehrere , verkettete Wechselrichter gelesen werden. - Strukturell noch mal umgebaut
Das Schreiben zum Wechselrichter war noch nicht korrekt umgesetzt, wurde aktuell allerdings auch noch nicht getestet.
- viele Optimierungen - das Schreiben in Register funktioniert jetzt auch
|
Pull Requests sollten nur Änderungen für ein Plugin enthalten, damit der PR zurück genommen werden kann, falls in dem Plugin Probleme auftreten. In diesem PR sind Änderungen in 6 Plugins enthalten. Bei Problemen in einem Plugin würden bei Rücknahme des PR deshalb die Änderungen in allen 6 Plugins zurück genommen. Außerdem ist es so schwierig, jemanden zu finden, der den PR beurteilt. Dazu muss im Prinzip jemand alle 6 Plugins einsetzen. Bitte als einzelne PRs einstellen und dabei den Kommentar von @bmxp berücksichtigen. |
|
Der Ansatz von bmxp scheint sinnvoll. Der pull request macht allerdings so Sinn, weil die Änderungen sich alle um das Gleiche Problem drehen. Das bedeutet, wenn es bei einem Plugin Probleme gibt, sollte das Problem auch bei den anderen auftreten und somit wären sowieso auch für die anderen die PRs zurück genommen werden. |
|
Vielleicht mache ich das auch falsch. Ich habe einen fork zu den plugins von SmartHomeNG gemacht. Damit macht ja jedes commit einen request auf alle plugins. Sollte ich da irgendwie einen fork zu einem einzelnen plugin machen, sofern das geht? |
|
Du machst pro Plugin auf Basis des aktuellen develop einen eigenen branch. |
|
Das kann ich künftig so machen. Aber jetzt sind die Sachen ja schon alle drin. Ich wüsste jetzt nicht, wie ich das alles wieder rückgängig und dann wieder einen neuen Fork machen soll. Es tut mir leid, ich habe mir Mühe gegeben das zu fixen, aber ich kriege das zumindest jetzt nicht anders hin, dass da einzupflegen. Ich verstehe das System mit fork, pull und dem ganzen Kram nicht wirklich. |
|
Bitte in getrennten PR je plugin neu stellen :) |
Because of many changes in pymodbus and some incompatibilities with different pymodbus-versions, I've dropped the support of pymodbus 2.x. Also Python >= 3.8 is neccessary.
The pluggit plugin is tested and working. All other plugins like ksemmodbus, kostalmodbus, modbus_tcp and sma_mb are untested.