Skip to content

Add retries for Marstek Venus C/E modbus communication#3390

Open
andlem74 wants to merge 5 commits into
openWB:masterfrom
andlem74:feature-venus_c_e-modbus-retries
Open

Add retries for Marstek Venus C/E modbus communication#3390
andlem74 wants to merge 5 commits into
openWB:masterfrom
andlem74:feature-venus_c_e-modbus-retries

Conversation

@andlem74

Copy link
Copy Markdown
Contributor

Wegen häufiger (mehr als einer pro Stunde) Fehler bei der TCP-Modbus-Kommunikation wurde Retry ergänzt. Zusätzlich muss das Pymodbus-Modul von 2.5.2 auf 2.5.3 upgegradet werden. Die 2.5.3 ist eine Bugfix-Version der 2.5.2. Ohne dieses Upgrade schlägt der Retry mit einer internen Fehlermeldung im Pymodbus-Modul fehl.

@LKuemmel LKuemmel left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich finde es sinnvoll, wenn das allgemein für alle Aufrufe in

super().__init__(ModbusTcpClient(host, port, framer, **kwargs), address, port, sleep_after_connect)
ergänzt wird.
Funktioniert das auch für ModbusSerial?

@LKuemmel LKuemmel added this to the 2.3.0 milestone Jun 12, 2026
@andlem74

Copy link
Copy Markdown
Contributor Author

Ich habe die Argumente als Defaults nach common/modbus.py verschoben.
Meines Wissens funktioniert das auch für ModbusSerial.

@andlem74 andlem74 requested a review from LKuemmel June 12, 2026 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants