Marstek Venus C, E battery

[andlem74]

Add support for Marstek Venus C, E battery
GUI PR is openWB/openwb-ui-settings#794

[seaspotter]

Already exists as approved PR: #2683

[benderl]

Im Gegensatz zu dem anderen PR finde ich es hier gut, dass die unterstützten Modelle (Venus C, E) genannt werden. Einfach nur "Marstek", wie der andere PR es macht, würde auch den Venus A sowie alle kommenden Produkte einschließen.

[seaspotter]

@benderl dann solltet ihr vllt mal für die Zukunft abstimmen wie etwas reviewt und approved wird oder gleich alles selbst machen wenn nicht mehr gewünscht ist das man PRs beisteuert weil man überall das Haar in der Suppe findet...

[benderl]

Ja, es gibt für die Reviews sicher noch Potential. Ansonsten finde ich Deine Reaktion etwas übertrieben. Ich schiebe es mal auf den frühen Montag Morgen.

Wenn es schon zwei PRs zu dem selben Thema gibt, ist es doch selbstverständilich, dass die miteinander verglichen werden. Das hat nichts mit unerwünschten Beiträgen oder dem sprichwörtlichen Haar in der Suppe zu tun.

[andlem74]

Ich möchte hier noch darauf hinweisen, dass dieser PR anders als #2683 die aktive Speichersteuerung implementiert.

[seaspotter]

@andlem74 Richtig, weil das aktuell noch getestet wird, siehe: https://forum.openwb.de/viewtopic.php?t=9872&start=20 Zudem gibt bisher kein positives power_limit für Speicherladen, das ist in der Implementierung auch nicht korrekt.

[andlem74]

Ich kann dazu nur sagen, dass ich es mit 2.1.8-RC.1 getestet habe und es funktioniert gut.
Falls der Review-Prozess Änderungsbedarf ergibt, passe ich den Code gerne an.

[benderl]

Zudem gibt bisher kein positives power_limit für Speicherladen, das ist in der Implementierung auch nicht korrekt.

Ich würde es nicht "nicht korrekt" nennen. Der Code wird aktuell niemals genutzt, da der Parameter entweder Null oder eine negative Zahl ist. Das spricht aber nicht dagegen, die Logik für positive Werte für die aktive Ladung zu ergänzen. Im Modul für die SonnenBatterie ist das ebenfalls bereits vorbereitet.

Wir werden nach dem aktuellen 2.1.8er Release mit den Reviews fortfahren und dann sehen, wie wir das Beste aus den beiden PRs machen können. Danke an euch beide für eure Arbeit.

[seaspotter]

Das spricht aber nicht dagegen, die Logik für positive Werte für die aktive Ladung zu ergänzen. Im Modul für die SonnenBatterie ist das ebenfalls bereits vorbereitet.

Da sind wir dann wieder mal erneut bei dem Thema keinerlei Feedback zu Verbesserungsvorschlägen zur Speichersteuerung. Es wird einfach ignoriert und totgeschwiegen und erzeugt so viel massiv Frust (bei mir). Mit nem Konzept könnte man so viel besser machen was jetzt alles so viel Probleme erzeugt. Aber man schweigt sich lieber aus.

[benderl]

Aktuell sind unsere Kapazitäten in anderen Teilprojekten gebunden. Bitte noch etws Geduld.

[LKuemmel]

Wie in der in PR #2683 beschriebenen Doku und Umsetzung braucht der SoC noch einen Faktor.

Suggested change

	soc = self._read_reg(32104, ModbusDataType.UINT_16)
	soc = self._read_reg(32104, ModbusDataType.UINT_16) * 0.1

[andlem74]

Die Doku ist hier inkorrekt bzw. veraltet. Zumindest bei meinem Venus-C mit aktueller Firmware darf nicht mit 0.1 multipliziert werden, sonst stimmt der angezeigte SoC in openWB nicht. Ich gehe davon aus, dass der Venus-E sich genauso verhält.
Falls @seaspotter wirklich mit einem Venus-E getestet hat und sicher ist, dass dieser den Faktor 0.1 tatsächlich benötigt, so wie es der Doku entspricht, sollten wir hier eine Variante einführen.

[LKuemmel]

Ok