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

Add battery grid charging #14855

Merged
merged 35 commits into from
Aug 3, 2024
Merged

Add battery grid charging #14855

merged 35 commits into from
Aug 3, 2024

Conversation

andig
Copy link
Member

@andig andig commented Jul 12, 2024

This PR adds charging from grid. Replaces #10814.

Battery mode is updated as follows:

SmartCost or FastCharge GridCharge Mode
- - Unverändert
x - Hold (falls Änderung)
(-) x Charge (falls Änderung)

Out of scope

  • Netzdienliche Steuerung berücksichtigen

@andig andig added the enhancement New feature or request label Jul 12, 2024
@andig andig requested a review from naltatis July 12, 2024 18:54
server/http.go Outdated Show resolved Hide resolved
@@ -43,6 +43,10 @@ const (
BufferStartSoc = "bufferStartSoc"
MaxGridSupplyWhileBatteryCharging = "maxGridSupplyWhileBatteryCharging"

// battery charging
GridChargeLimit = "gridChargeLimit"
GridChargeActive = "gridChargeActive"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich bin noch nicht überzeugt vom Namen. Hier gehts ja "nur" ums Batterie-Laden. Das Smart Cost Limit am LP ist ja auch ein Grid-Charging, aber halt fürs Fahrzeug. Für Symmetrie würd ich batteryCostLimit vorschlagen.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich find den eigentlich gut. Energie von außen ist günstig und das Feature heisst landläufig Netzladen.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ich hätt gerne die Batterie da mit drin. Gerade weil es ja "nur" die Batterie betrifft. Sonst wird nicht klar, dass es hier nicht um unser "normales laden" geht.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wie wär's mit batteryGridChargeLimit?

Co-authored-by: Michael Geers <[email protected]>
core/loadpoint_api.go Outdated Show resolved Hide resolved
core/loadpoint_api.go Outdated Show resolved Hide resolved
core/loadpoint_api.go Outdated Show resolved Hide resolved
api/rates.go Show resolved Hide resolved
core/site.go Outdated Show resolved Hide resolved
@naltatis

This comment was marked as resolved.

core/site_api.go Outdated Show resolved Hide resolved
@naltatis
Copy link
Member

Dass die Batterie nicht lädt liegt vmtl. an der batterycontrol Implementierung bei Sungrow. Ich hab gerade den Issue/Discussion nicht spontan gefunden. Aber ich meine wir hatten da noch ein offenes Thema, dass neben dem Modus auch noch die Max-Ladeleistung (die sich resettet) immer gesetzt werden muss. Bin mir nicht sicher, wie das damals in der Diskussion ausgegangen ist. \cc @premultiply

andig and others added 2 commits July 13, 2024 23:26
@andig
Copy link
Member Author

andig commented Jul 14, 2024

Dass die Batterie nicht lädt liegt vmtl. an der batterycontrol Implementierung bei Sungrow.

Die ganze BatteryMode Logik passt noch nicht.

@andig
Copy link
Member Author

andig commented Jul 14, 2024

@naltatis komplett neuer Test notwendig. Bisher wurde der Modus gar nicht auf die Batterie geschrieben. Die etwas verwobene Logik ist jetzt komplett entzerrt. smartCostNextActive- da für Site unerheblich- wandert komplett in den Loadpoint.

@naltatis
Copy link
Member

Hier eine Variante, die die Eigenschaften von davor zusammenzieht. Die Beschreibungen für priority und buffer sind überarbeitet und klarer. Die Tab-Höhen auf großem Bildschirm sind identisch (kann je nach Sprache aber abweichen).

🖥️ Large

desktop usage

desktop grid

💻 Medium

tablet  usage

tablet grid

📱 Small

mobile usage

mobile grid

🇩🇪 Deutsche Texte

texte grid deutsch

texte usage deutsch

status deutsch

@naltatis
Copy link
Member

Wenn Netzladen nicht möglich ist (kein dynamischer Tarif, steuerbarer Speicher oder experimental) ⤵︎

Bildschirmfoto 2024-07-31 um 18 14 22

@xerion3800

This comment was marked as off-topic.

@naltatis

This comment was marked as off-topic.

@UDicke

This comment was marked as off-topic.

@xerion3800

This comment was marked as off-topic.

@UDicke

This comment was marked as off-topic.

@xerion3800

This comment was marked as off-topic.

@UDicke

This comment was marked as off-topic.

@naltatis

This comment was marked as off-topic.

@UDicke

This comment was marked as off-topic.

@naltatis

This comment was marked as off-topic.

@UDicke

This comment was marked as off-topic.

@naltatis
Copy link
Member

naltatis commented Aug 2, 2024

@UDicke Das geht hier im PR zu weit ins Detail bzw. übers Thema hinaus. Mach gerne sonst nen separaten Discussion-Thread auf.

@andig andig merged commit b0ea058 into master Aug 3, 2024
6 checks passed
@andig andig deleted the feat/grid-charge branch August 3, 2024 13:23
@MatthiasSchneck
Copy link

Hallo zusammen, ich freue mich schon auf die Funktion "Batterie Laden", da ich auch einen dynamischen Stromtarif nutze. Das wird vor allem im Winter interessant. Wird es auch die Möglichkeit geben, die Batterie nicht komplett voll zu laden, sondern nur bis zu einem einstellbaren SOC? Ich würde nämlich gerne noch etwas "Platz" in der Batterie lassen, falls dann im Winter die Sonne doch mal scheint.

@FantOEM
Copy link

FantOEM commented Aug 6, 2024

So geht es mir auch. Wie wird dann die Priorisierung des Hausbatterieladens gehandabt, wenn die Sonne scheint und der Batterie mit PV geladen wird? Wird dann auf das nachziehen aus dem Stromnetz verzichtet oder lädt er auf jeden Fall, wenn der Strompreis unter den eingestellten Schwellwert sinkt?

Grüße,
FantOEM

@andig
Copy link
Member Author

andig commented Aug 6, 2024

Es gibt keine Magie. Hier wird einfach die Ladung des Akkus unterhalb einer Preisschwelle gesteuert. Sonst ändert sich nichts. Alle weiteren Themen bitte separat diskutieren.

@Wisley7

This comment was marked as off-topic.

@andig
Copy link
Member Author

andig commented Aug 18, 2024

Hier OT, bitte neue Diskussion.

@evcc-io evcc-io locked as resolved and limited conversation to collaborators Aug 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request needs documentation Triggers issue creation in evcc-io/docs prio Priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants