@anbie in Deinem PR gehts etwas durcheinander ;)

@andig, ich hatte diese PR auf Basis von #21972 erstellt.
Da in beiden PRs zwar dieselbe Datei geändert wird, aber in unterschiedlichen Zeilen, war das eher unnötig und sorgt sicher für das Durcheinander.
Was schlägst du vor? PR schließen und Neuen öffnen?

2 branches, 2 prs. PRs NIEMALS von master machen.

2 branches:

#21972: feat/lambda_neg_excess_energy (https://github.com/anbie/evcc/tree/feat/lambda_neg_excess_energy)
#22196: feature/config_ports4modbustcp (https://github.com/anbie/evcc/tree/feature/config_ports4modbustcp)

…dennoch sind hier dinge drin die nicht rein gehören. Git revert der falschen commits?

@premultiply das betrifft ALLE Geräte die Modbus unterstützen, aber direkt über host/port konfiguriert sind. Wollen wir das konsequent überall ändern? Falls ja: port konfigurierbar machen oder die Modbus Mimik einsetzen?

@premultiply das betrifft ALLE Geräte die Modbus unterstützen, aber direkt über host/port konfiguriert sind. Wollen wir das konsequent überall ändern? Falls ja: port konfigurierbar machen oder die Modbus Mimik einsetzen?

Und nicht mal nur Modbus, sondern auch Geräte die nur den host erfordern. Ich musste in den letzten Wochen auch negative Erfahrungen mit der GUI machen. Ändern sich IP oder Port werden die Geräte nicht mehr geladen, auch in der ConfigGui nicht mehr angezeigt. Ohne Zugriff auf die db ist dadurch keine Anpassung von host/port möglich.

Die "Modbus-Mimik" (das Template) funktioniert immer dort und sollte auch wo immer möglich zum Einsatz kommen wo ein Endgerät (nur) eine ID hat. Ob diese am Gerät tatsächlich veränderbar ist ist egal. Im Template wird der feste Wert als Default eingetragen.

Ist der PR soweit wieder auf "ready for review" zu gehen ?

Nein. Wenn wir das machen dann bitte konsequent bei allen Geräten und nicht nur bei einem. Und es muss geprüft werden, ob das ein breaking change ist.

@andig
Wie machen wir hier weiter? Ohne Port-Einstellungen kann ich die Lambda nicht nutzen. Ich versteh ja "konsequent bei allen Geräten". Das muss dann wohl in allen templates geändert werden. Aber muss es denn in einem Rutsch erfolgen? Oder gibt es eine zentrale Stelle?

(Ich bin grad in dem Bergen mit wenig Netzwerk-Verbindung. Ich helfe gerne weiter mit wenn ich zurück bin.)

Gerne ein PR für alle, mindestens alle Modbusgeräte. Es hilft uns leider nicht, EInzellösungen zu bauen :(

Ich selbst habe keine Möglichkeit, Modbus-Geräte in größerem Umfang zu testen. Wenn ich es richtig gesehen habe, lässt sich der Port aber bei den meisten Charger ohnehin schon konfigurieren – nur bei wenigen ist er noch hart auf 502 gesetzt. Aber ich lass hier mal gut sein und konfiguriere meinen modbus proxy anders.