-
-
Notifications
You must be signed in to change notification settings - Fork 660
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 charge planner #5445
Add charge planner #5445
Conversation
@schenlap Du könntest mal einen ersten Test wagen, sowohl für einfaches Zielladen mit festem Tarif wie auch mit variablem Tarif. |
2f40dca
to
c93971a
Compare
364a3ee
to
9290cbe
Compare
This is very nice! |
The planner can work on any "tariff/rates". It does not care if the |
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
Test läuft. Das ist jetzt mal der einfachste Fall, Auto ist noch recht voll.
-> hat geladen wie geplant. Ladevorgang war im ersten Slot schon fertig. |
This comment was marked as off-topic.
This comment was marked as off-topic.
bin gespannt ob das klappt. Zum einen generell, zum Anderen weil die Ladekurve bei vollem Auto ja schlechter wird. Bei deinem Plan oben wärs noch nett, wenn er die angebrochene Stunde auch nur halb zeigen würde? |
This comment was marked as off-topic.
This comment was marked as off-topic.
Mir ist gerade aufgefallen dass nach dem Laden / Abstecken des Fahrzeugs und wieder anstecken der Timer im UI immer noch angezeigt wurde (in diese, Fall heute, 5AM also ~12h in der Vergangenheit). Ich weiß jetzt nicht ob das immer schon so war. Auch ein Reload der Seite hat nichts geändert. Meine Erwartung wäre dass nach dem Laden (oder zumindest dem abstecken) der Timer gelöscht wurde. Im Log habe ich keinen Hinweis gefunden dass der Timer noch aktiv wäre. |
This comment was marked as resolved.
This comment was marked as resolved.
@schenlap könntest Du das weiter analysieren? Was ist falsch daran auch die Daten von "tomorrow" mit einzubeziehen? |
@schenlap wann genau sollten wir das tun:
|
Ja mache ich. Wenn es funktioniert ist daran nichts falsch. Jetzt scheint es noch wo einen Bug zu geben. |
Ladung hat heute auch funktioniert, es wurde allerdings öfters ein/ausgeschaltet als gedacht. Das liegt wie damals schon mal diskutiert an
virtualCapacity Berechnung funktioniert bei mir einfach nicht. |
This comment was marked as resolved.
This comment was marked as resolved.
Mhhm- warum tut das inplace weh? Die Reihenfolge hinterher stimmt doch? |
Schlimmstenfalls muss dann die restliche Zeit geladen werden. Normalerweise kommen die neuen Preise aber rechtzeitig. Schöne Lösung weil implementierungsseitig sehr einfach. |
aff5bd5
to
645c98b
Compare
Anbei mal ein kurzer Test von meiner Seite. Siehe Logs und Tibber Screenshots.
Ich sehe:
Vielleicht helfen euch die Daten. |
Warum will der bei dir so lange laden?
|
Der zweite Wert kommt aufgrund 90% Effizienz, der erste ist unklar. |
Hilft hier meinte config? Habe im loadpoint auf 1phasig gestellt und 6A-20A min max eingestellt. Beim Tesla auf 3 Phasen, was er auch real nutzt. rechnet er vielleicht mit 1 Phase 20A? |
Glaskugel -> Reparatur 💁♂️ |
Ja, sehr wahrscheinlich. |
Das Problem liegt hier: Line 56 in d1a629e
Lass uns das in einem Folge-PR verbessern, siehe offene Punkte. |
Anbei meine Loadpoint config, falls man da was sehen kann. Also Auto war es ein Tesla mit Phase 3 config. Ich ändere die Config jetzt ab und werde im Loadpoint mal aufräumen und den Test wiederholen. |
@Hofyyy |
Jetzt verstehe ich es noch weniger (und es fehlt der Rest der Config um ein klares Bild zu bekommen). Die Wallbe hat keine Phasenumschaltung. Wie soll da eine Umschaltung auf 3p passieren? Lass uns das bitte eine eigene Discussion auslagern, das ist hier erstmal OT. |
Der zweite Test läuft mit korrigierter Config.
Ich sehe:
[lp-1 ] DEBUG 2022/12/22 13:23:54 charge power: -1W [lp-1 ] DEBUG 2022/12/22 13:36:44 planning 3h1m0s until 2022-12-23 07:00:00 +0000 UTC @andig Siehe: |
This comment was marked as resolved.
This comment was marked as resolved.
@Hofyyy also die Sortierung der Slots stimmt, das hab ich grad überprüft. |
Ich gucke mir das morgen noch mal an, oder probiere mich mal an einem Test. |
Wie oft werden die Kosten (z.B. von Tibber) aus dem Internet geladen? Ich hatte jetzt 2x beim Testen den Fehler, dass keine "Rates" verfügbar sind. Werden die dann probiert alle 5 min neu zu Laden, oder wie geht man mit dem Fehlerfall um, dass es ganz am Anfang nicht geklappt hat Sie zu holen? |
Hier gar nicht. Neues Ticket für Tibber mit Logfile? |
ok, dachte das hängt mit am planner. |
Seems as if all open points are resolved. Will merge asap. |
407cc46
to
1f08b02
Compare
@andig Vielleicht eine doofe Frage. Gibt es eine Art Designdokument (Architecture), die den Planner (inklusive Persistenz und aller Features) beschreibt. Ich bin zwar neu hier, aber das kling eher nach einer komplexen Erweiterung als nach einem einfachen Enhancement. |
Ne, gibts nicht. Die wesentlichen Dinge haben wir auf Slack diskutiert. Wenn dich die Interna interessieren und du da einsteigen willst meld dich gerne bei Slack an. |
Fix #5337. This work builds on #2211 by @schenlap. It provides the following features:
cheap
price setting- it's no longer neededsoc/Timer
is removed and replaced by the (much simpler) plannerVehicleFinishTimer
- we'll always rely on the estimator now (simplifiedRemainingChargeDuration
)The
planner
is essentially a cost optimiser. It's task is to chose a lowest-cost plan of suitable charging slots from a list of slots (typically from a variable tariff), each with its own cost. Constraints are:TODO:
cheap
config optionBonus features:
GetMaxPower()
for complex vehicle config (Add charge planner #5445 (comment))