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

GoodWe: fix battery control #13125

Conversation

andiwist
Copy link
Contributor

Bugfix: Fixed the charging from Grid when Battery is locked and "charge" mode was previously activated

Bugfix: Fixed the charging from Grid when Battery is locked and "charge" mode was previously activated
@andig andig changed the title Fix for GoodWe BatteryControl GoodWe: fix battery control Mar 24, 2024
@andig andig merged commit 4329599 into evcc-io:master Mar 24, 2024
6 checks passed
@andiwist
Copy link
Contributor Author

Funktioniert mit der nightly von der Nacht zum 25.Mar .
Hab die Uebergaenge von den Modi nochmal damit getestet, schaut alles gut.
Eine Kleinigkeit mit der GUI fällt mir aber auf. Bei mir ist die Anzeige "Batterie laden (gesperrt)" im Falle von Hold nicht ganz richtig. Ich kann es mangels leistungsfähiger PV nicht testen, aber ich vermute wenn ich im Hold-Modus mit 11kw das Auto Schnelllade, und z.b. PV und Wechselrichter 15kw liefern wuerden, wuerde der GoodWe vermutlich die übrige PV Leistung (4kw) in die Hausbatterie einspeisen. Sprich, es steht zwar dann gesperrt da, aber man wuerde dann in der Anzeige trotzdem Leistung sehen. Ich vermute mal das ist kein Problem und es gibt vermutlich kaum jemanden, der das als Problem sehen wird bzw. in die Verlegenheit kommen das zu sehen. Aber ja, es ist momentan nur meine Theorie, kann es nicht testen.

@premultiply
Copy link
Member

Das wäre ja auch richtig.
Die gewünschte Sperrung bezieht sich nur auf die Entladung.

@andiwist
Copy link
Contributor Author

image

Ging ja doch mit dem Testen. Musste nur das Schnellladen auf 1-phasig und 6A begrenzen, dann schafft das meine Anlage auch.
Aber meine Theorie war falsch, er lädt die Batterie nicht mit PV.
Gut, damit ist die GUI doch richtig mit der Anzeige.

@andig
Copy link
Member

andig commented Mar 25, 2024

Aber meine Theorie war falsch, er lädt die Batterie nicht mit PV.

Das sind in der Tat schlechte Nachrichten. Denn hold sollte Entladung verhindern, aber nicht Ladung :(

@andiwist
Copy link
Contributor Author

Alles klar. Leider ist die Doku nicht wirklich hilfreich, und die Erklärung dazu im OpenEMS Projekt ist auch nicht stimmig wie es scheint.
Dann werde ich weiter Tests durchfuehren und z.b.
verschiedene Modi und Einstellungen ausprobieren.
Hab da schon paar Ideen.
Wenigstens habe ich jetzt verstanden was wirklich gebraucht wird.
Kleine Randnotiz, wenn wirklich laden gewünscht ist in Hold, wuerde ich das "gesperrt" in der GUI bei "Out" wegmachen lassen. Stimmt dann nicht.

@premultiply
Copy link
Member

Nur entladen gesperrt ist die Wunschvorstellung. Leider ist die Realität oft eine komplette Sperre als Kompromiss weil das System einfach nix anderes kann.
Die Alternative wäre die Funktion komplett zu entfernen.

@andiwist
Copy link
Contributor Author

Als Doku hab ich momentan nur die OpenEMS Implementierung der Modi : OpenEMS
Davon hab ich fast alles ausprobiert.
Der beste Kandidat fuer das gewuenschte Verhalten waere Modus "Conserve" gewesen.
"In on-grid mode, the battery is continuously charged, and only PV power (AC Couple model takes 10% of the rated power of the power grid) is used. The battery can only discharge in off-grid mode."
An sich macht er auch was wir wollen, aber.... in dem Modus schaut er gar nicht auf den SmartMeter und nimmt einfach die PV Leistung ohne auf den Hausverbrauch zu achten.
Also ich befuerchte fast , was besseres als momentan implementiert ist, bekomme ich nicht hin.

@andiwist
Copy link
Contributor Author

Anderer Ansatz, ich werde noch etwas mit anderen Parametern spielen.
z.b. BMS_DISCHARGE_MAX_CURRENT, new UnsignedWordElement(45355),SCALE_FACTOR_MINUS_1), // [0,1000]
Damit kann ich Discharge max current auf 0 setzen. Bei ersten Tests hat das System sich erstmal paar Sekunden nicht mehr gemeldet. Zum Schluss musste ich GoodWe Inverter und Batterie komplett neu starten.
Ich melde mich falls ich was habe bzw. mein System zerstoert habe.

naltatis pushed a commit that referenced this pull request Mar 26, 2024
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.

3 participants