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: add battery control #12833

Merged
merged 6 commits into from
Mar 15, 2024

Conversation

andiwist
Copy link
Contributor

@andiwist andiwist commented Mar 9, 2024

Wie diskutiert unter #12752 .
Loesung implementiert mittels Anpassung des GoodWe Templates.
Lokales Kompilat hat funktioniert und bei Fast Charge Batterie auf Standby geschaltet.
Modbus Werte im Wechselrichter mittels NodeRed Modbus Readout verifiziert.

@andiwist
Copy link
Contributor Author

andiwist commented Mar 9, 2024

Noch ein kleiner Remark: Den Charge Modus konnte ich nicht testen mangels Funktion im evcc, aber theoretisch mittels NodeRed getestet.

@premultiply
Copy link
Member

Den Charge Modus konnte ich nicht testen mangels Funktion im evcc

evcc meter -b charge

@premultiply premultiply added the devices Specific device support label Mar 10, 2024
Datentyp von UINT32 auf UINT16 fuer EMSPowerSet geaendert
@andiwist
Copy link
Contributor Author

Den Charge Modus konnte ich nicht testen mangels Funktion im evcc

evcc meter -b charge

Danke. Damit hab ich rausgefunden, dass der Datentyp von EMSpowerset nicht gepasst hat. Ist jetzt gefixt.

@andiwist andiwist changed the title Batteriesperre fuer GoodWe Wechselrichter add battery control GoodWe Mar 13, 2024
@andiwist andiwist closed this Mar 14, 2024
@andiwist
Copy link
Contributor Author

Sorry, hab zu frueh geschlossen. War noch nicht gemerged. Bin noch Github Newbie.

@andiwist andiwist reopened this Mar 14, 2024
@andiwist andiwist requested a review from premultiply March 15, 2024 00:15
@andiwist
Copy link
Contributor Author

@premultiply
Da ich zum ersten Mal GitHub und zum ersten einen PR mache, bitte um Nachsicht wenn ich Fehler mache.
Sollte es von meiner Seite noch was zu tun sein, bitte melden. Ansonsten waere das Durchwinken (sprich Merge in Master) imho kein Problem mehr oder?

@andig andig changed the title add battery control GoodWe GoodWe: add battery control Mar 15, 2024
@andig
Copy link
Member

andig commented Mar 15, 2024

Den Charge Modus konnte ich nicht testen mangels Funktion im evcc, aber theoretisch mittels NodeRed getestet.

@andiwist kannst Du mittels

evcc meter -b charge

@premultiply LGTM

@andiwist
Copy link
Contributor Author

Den Charge Modus konnte ich nicht testen mangels Funktion im evcc, aber theoretisch mittels NodeRed getestet.

@andiwist kannst Du mittels

evcc meter -b charge

@premultiply LGTM

Danke. Hatte ich dann getestet dank dem Hinweis von premultiply. Hat funktioniert. Siehe oben.

@andig andig merged commit bf11cfc into evcc-io:master Mar 15, 2024
6 checks passed
@andig
Copy link
Member

andig commented Mar 15, 2024

Achso- der fehlte auch noch: f8685fd

@andiwist andiwist deleted the feature/goodwe-battery-control branch March 15, 2024 20:40
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 2024
@andiwist
Copy link
Contributor Author

Heute Nacht hab ich das nightly von gestern benutzt um testweise geplantes Laden und mit der Batteriesperre an auszuprobieren . Es war zwar das Auto voll aber auch die Batterie wurde vom Strom mitgeladen. Hab leider keine Zeit es anzuschauen heute.

@andig
Copy link
Member

andig commented Mar 23, 2024

Danke für die Info- dann müsste jetzt anaylsiert werden, was im Template anders ist, als bei Deinem erfolgreichen Test.

@andiwist
Copy link
Contributor Author

andiwist commented Mar 23, 2024

Hab's gefunden. Die 10000 Watt fuer den Charge Mode muessen bei den anderen Modi auf 0 zurueckgesetzt werden. Beim Standby hat es die Folge, dass es mit den 10000 from Grid die Batterie laedt.
Neuer PR #13125

naltatis pushed a commit that referenced this pull request Mar 26, 2024
@maatinh
Copy link
Contributor

maatinh commented Aug 23, 2024

Wäre nicht der EMSPowerMode = 4 der Richtige Modus für Netzladen?

grafik

@andig
Copy link
Member

andig commented Aug 23, 2024

Das Feature ist umgesetzt. Falls es ein Problem gibt bitte Issue. Danke!

@thebvbdom
Copy link

thebvbdom commented Sep 26, 2024

Hab's gefunden. Die 10000 Watt fuer den Charge Mode muessen bei den anderen Modi auf 0 zurueckgesetzt werden. Beim Standby hat es die Folge, dass es mit den 10000 from Grid die Batterie laedt. Neuer PR #13125

Kann man darüber dann nicht auch das Netzladen bei GoodWe umsetzen?

Bzw. gab es hierauf irgendwo schon eine Antwort?

Wäre nicht der EMSPowerMode = 4 der Richtige Modus für Netzladen?

@andig
Copy link
Member

andig commented Sep 26, 2024

Kann man darüber dann nicht auch das Netzladen bei GoodWe umsetzen?

Es ist umgesetzt und dieser PR lange geschlossen.

@evcc-io evcc-io locked as resolved and limited conversation to collaborators Sep 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
devices Specific device support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants