Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Kostal Plenticore device.py für Hybrid-WR geändert #1231

Merged
merged 2 commits into from
Dec 1, 2023

Conversation

DerHerrW
Copy link
Contributor

@DerHerrW DerHerrW commented Nov 2, 2023

Siehe auch Issue #1201.
Bisher musste die Batterie für einen Kostal Plenticore in der Struktur parallel zum Inverter angeordnet werden, damit die Leistungen passen. Dann stimmen allerdings die Zähler nicht und in der Auswertung wird Batterieentladung nachts als PV-Leistung angezeigt.
Durch einfaches Durchreichen von inverter_status nach pv_status stimmen beide Sachverhalte, sofern der Speicher als Untergerät vom Wechselrichter angeordnet wird wie im Wiki beschrieben.

Möglicherweise müssen die Leistungen noch korrigiert werden, ich nehme an, die weggefallenen Korrekturen hatten einen Grund. Ich konnte allerdings bisher keine Probleme oder Unstimmigkeiten feststellen.

--mein erster Pull-Request--

@LKuemmel
Copy link
Contributor

Beim Portieren des Moduls für software2 kam auch das Thema auf, ob diese Berechnungen benötigt werden. Damals hatte sich der Autor des Moduls gemeldet: snaptec/openWB#2440 (comment) Offenbar passt Kostal die Schnittstelle immer mal an. Hast Du die aktuelle Software auf deinem WR? Hast Du dir die verschiedenen Situationen angesehen, die in dem Post beschrieben werden?

@DerHerrW
Copy link
Contributor Author

Beim Portieren des Moduls für software2 kam auch das Thema auf, ob diese Berechnungen benötigt werden. Damals hatte sich der Autor des Moduls gemeldet: snaptec/openWB#2440 (comment) Offenbar passt Kostal die Schnittstelle immer mal an. Hast Du die aktuelle Software auf deinem WR? Hast Du dir die verschiedenen Situationen angesehen, die in dem Post beschrieben werden?

Ich habe die aktuelle SW auf Wechselrichter und KSEM. Mein pull request hat zur Folge, daß als Inverterleistung die AC-Ausgangsleistung ausgegeben wird. Dies ist für die neue Struktur (Batterie als Untergerät des WR) grundsätzlich schonmal ziemlich richtig. Ich sehe es als Feinheit, ggf. die Verlustleistungen aufzuteilen und einzurechnen - dies würde ich gerne später durchführen sofern nötig.

Die unterschiedlichen Fälle (Erzeugung, Batterie-Laden, -entladen) funktionieren. Momentan bestehen noch Abweichungen in der von OpenWB gezeigten PV-Leistung im Vergleich mit der vom WR angezeigten Leistung, da der WR die DC-Leistung anzeigt (die für Wallbox-Zwecke sinnlos ist) und die Wallbox die für sie relevante Ausgangsleistung anzeigt. Im Erzeugungsbetrieb ist die DC-PV-Leistung 25-70W größer als die abgegebene WR-Leistung. Die Batterieleistung ist noch so wie vorher berechnet - die Umschaltung zwischen Laden und Entladen ergibt auch Sinn, da die Batterieleistung mit der AC-Leistung verrechnet wird.

Dazu die Frage, ob es dokumentierte Anforderungen gibt, was genau die PV-Leistung zu sein hat und was die Batterieleistung? Gibt es da Fallunterscheidungen? Für die Zwecke der Ladestromberechnung scheint mir am sinnvollsten, die AC-Ausgangsleistung und die Batterieleistung zu verwenden.

@LKuemmel
Copy link
Contributor

In den alten Kommentaren steht ja noch, dass Kostal wohl früher keine AC-Leistung ausgegeben hat. Das wurde dann offenbar geändert. Verluste müssen dann nicht mehr berücksichtigt werden, die entstehen bei der Wandlung von DC in AC.

Es gibt keine festen Anforderungen, dass die PV-Leistung AC sein muss, weil manche WR diese eben gar nicht ausgeben. Schöner ist es natürlich.

@LKuemmel LKuemmel added this to the 2.1.3 milestone Nov 21, 2023
@DerHerrW
Copy link
Contributor Author

Eine Möglichkeit wäre noch, die Verluste des WR der AC-Ausgangsleistung zuzuschlagen, dann wäre der Hausverbrauch entsprechend höher. Aber das wäre ziemlich indirekt, und ich bin ein Freund von keep it simple.

@LKuemmel
Copy link
Contributor

Das halte ich auch nicht für sinnvoll, da die größten Verluste bei der Wandlung von DC nach AC entstehen und damit durch die Ausgabe der AC-Leistung abgedeckt sind.

@LKuemmel LKuemmel merged commit 42658da into openWB:master Dec 1, 2023
1 check passed
@DerHerrW DerHerrW deleted the KostalLeistung branch December 6, 2023 18:56
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.

None yet

2 participants