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 target charging #370

Merged
merged 3 commits into from
Jan 2, 2021
Merged

Add target charging #370

merged 3 commits into from
Jan 2, 2021

Conversation

andig
Copy link
Member

@andig andig commented Oct 1, 2020

Fix #225, experimental implementation, no UI

@andig andig marked this pull request as draft October 1, 2020 16:36
@andig andig added the enhancement New feature or request label Oct 1, 2020
core/targetcharge.go Outdated Show resolved Hide resolved
@premultiply
Copy link
Member

Fixes #225

@andig andig marked this pull request as ready for review October 2, 2020 16:07
@andig andig force-pushed the feature/target-charging branch from 59bf161 to 3845b90 Compare October 4, 2020 11:33
@andig andig force-pushed the feature/target-charging branch from f7db939 to 2edc0cf Compare October 6, 2020 07:17
@andig
Copy link
Member Author

andig commented Oct 6, 2020

Was wäre bei Target SoC das erwartete Verhalten wenn wir im PV oder PV+Min Modus sind? Soll hier bei Erreichung des SoC der Ladevorgang wirklich abgebrochen werden? Für Batterieschonung mag das sinnvoll sein, PV Energie würde allerdings schlimmstenfalls vergeudet.

@Caibuk
Copy link

Caibuk commented Oct 6, 2020

Ich hätte für beide Varianten je einen Anwendungsfall (wenn in PV oder PV+Min).

  • Ladevorgang abbrechen: ich möchte den Autoakku zumindest auf xx % laden, danach soll der Hausakku geladen werden
  • Ladevorgang fortsetzen: wenn der Hausakku voll ist (oder einen Min-SoC erreicht hat) und sonst eingespeist würde, soll der Autoakku weitergeladen werden

Ich denke, man könnte das durch eine wechselnde Priorisierung abbilden. Zunächst Auto auf Target SoC laden, falls Hausakku Min-SoC enthält, kann das Autoladen fortgesetzt werden. Ansonsten zunächst den Hausakku aufladen und danach ggf. das Auto weiterladen. (Ich benötige immer einen Min-SoC im Hausakku, da ich Hochtarifzeiten überbrücken will/muss.)

Wenn man in "now" lädt sollte abgebrochen werden (hier könnte man noch überlegen, ob bei weiter vorhandenem Überschuss entsprechend der o.g. Regeln weitergeladen wird).

@andig
Copy link
Member Author

andig commented Oct 6, 2020

@Caibuk die Priorität für den Heimakku bekommst Du (im PV Mode!) mit #369. Hier geht es eher darum was passiert wenn das Auto seinen Zielwert erreicht hat (ggf. nachdem der Heimakku vorher mit Prio gefahren wurde).

Ein Feature den Hausakku in mehren Stufen zu priorisieren ist hier ebenfalls nicht Gegenstand, sondern dieser PR konzentriert sich erstmal auf die Fahrzeugseite.

@Caibuk
Copy link

Caibuk commented Oct 6, 2020

Stimmt, den anderen PR hatte ich schon wieder vergessen.
Dann würde ich sagen, dass bei Target-SoC gestoppt wird - egal, welcher Modus eingestellt wird. Wenn ich mehr laden wollen würde, müsste ich eben den Zielwert höher setzen.
Ein use case fürs Stoppen bei Zielerreichung auch bei PV only wäre bei mir: am Wochenende stoppe ich am Samstag nach dem Mittagspeak das Laden bei ca. 70 Prozent. Am Sonntag starte ich das Laden dann kurz vorm Mittagspeak um ein Abregeln zu verhindern und lasse für die Woche vollladen.
Soweit mein Senf dazu.

@andig andig mentioned this pull request Oct 6, 2020
@mark-sch
Copy link
Contributor

mark-sch commented Oct 7, 2020

Die Gedanken den Hausakku beim laden zu schonen bzw. in eine Prio zu bringen sind natürlich sinnvoll. Auf der anderen Seite sind hier aber auch Grenzen gesetzt, z.B. im "Sofort" Modus, hier wird zunächst so lange Battery2Battery geladen bis der Hausakku leer ist. Gut wäre, das gewünschte Hausbatterie-Verhalten per MQTT zu publishen, damit eine Wechselrichter Subscriber-Implementierung dies im Lade/Entladeverhalten berücksichtigen kann. Im Sunny5 WR kann die Entladerate z.B. festegelegt werden, während des Sofort-Ladens könnte der WR benachrichtigt werden, nur noch für die Grundlast 10%/500W zu entladen. Am Ende des Auto Ladens kann der Akku so weiter für dies Haus-Aufgaben genutzt werden - ohne unnötige Deep-Cycles auf den Hausakku.

@andig
Copy link
Member Author

andig commented Oct 12, 2020

Hier die jetzt implementierte Logik wie sie sein sollte:

-- Grafik gelöscht --

@mark-sch
Copy link
Contributor

TipTop, der Entscheidungsbaum ist gut verständlich und deckt eine Menge Scenarien ab. Was ich sehr hilfreich fände ist die Einbeziehung des Hausspeichers in das Ladeverhalten von EVCC. Min PV ist ein sehr praktischer Modus, im Zweifel entläd dieser Modus auch den Hausspeicher (langsam), was ja auch o.k. Ein Parameter wie HomeSoCLimit wäre sehr praktisch, der bei Unterschreitung (und wenn definiert) ein Umschalten von MinPV in den PV Modus auslöst. Dieser hätte ein ähnliches Verhalten wie MinSoC, aber es bleibt sicher gestellt, dass die Fahrzeugladung nur aus selbst erzeugtem Solarstrom geschieht. Das Battery2Battery Laden könnten man so in sinnvollen Grenzen halten.

@andig
Copy link
Member Author

andig commented Oct 14, 2020

Warum nixht einfach gleich PV Modus nehmen? Mir erschliesst sich der Nutzen der zusätzlichen Komplexität nicht :O

@andig andig added the ux User experience/ interface label Oct 29, 2020
@andig andig force-pushed the master branch 2 times, most recently from a5c07fe to 865acdc Compare October 30, 2020 12:14
@andig andig force-pushed the feature/target-charging branch 2 times, most recently from 71d9958 to dbbe0fe Compare November 22, 2020 13:49
@andig andig force-pushed the master branch 3 times, most recently from b5298a7 to 505f9de Compare November 23, 2020 17:48
@andig andig force-pushed the feature/target-charging branch from dbbe0fe to a366b5e Compare January 2, 2021 15:18
@andig andig merged commit ccdb1cb into master Jan 2, 2021
@delete-merged-branch delete-merged-branch bot deleted the feature/target-charging branch January 2, 2021 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request ux User experience/ interface
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Zielladen
4 participants