-
-
Notifications
You must be signed in to change notification settings - Fork 917
Planner: add precondition phase before departure #20431
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
Conversation
Hier ein erster Entwurf für die Precondition Anzeige. Ich bin damit noch nicht 100% happy (Doppel Switch Elemente sehen komisch aus). Aber der Entwurf erfüllt schonmal folgende Anforderungen.
Daher hier die Kombination aus einer einfachen Aktivierung, die dann Erklärung und Zusatzoptionen erscheinen lässt. Sinnvolle Iconsprache habe ich nicht gefunden. Daher lieber unmissverständlicher Text. Der Titel "Pflege" sehr kompakt (weil 1.) und alleine nicht verständlich. In Kombination mit "Akkupflege" sollte das aber klar genug sein. Englische Übersetzung "Care" / "Battery care". Inhaltlich passt das Wording auf die Anwendungsfälle "Auf 100% Laden aber den Wagen nicht lange voll stehen lassen wollen" und "Nicht mit kaltem Akku starten". Mich nervt auch noch, dass der Dialog springt weil eine Zeile dazukommt. Würde das aber hier erst einmal in Kauf nehmen und später noch mal drüber nachdenken ob nicht das Plan-Diagramm eigentlich nach ganz oben gehört. precondition.webm@andig in der HTTP-API wurde |
@naltatis ich finde "Pflege" gar nicht so prägnant. Wie wäre es alternativ mit "Spät" oder "Warm"? Die Pflegedauer wollten wir ja egtl. gar nicht raus reichen, spricht aber nix dagegen- auch die 2te Zeile sieht für mich gut aus.
Charge-and-go :O |
Ja "Spät" / "Spät laden:" finde ich auch eine gute Idee. Da wir die Zeit ja kommunizieren müssen damit dass Feature verständlich würd hab ich's gleich bearbeitbar gemacht. Nennen wir die Zeit nicht entstehen beim Nutzer vmtl Fragezeichen im Kopf > Doku erforderlich. |
Danke für den Ping. Ohne deinen ganzen Entwurf kaputt machen zu wollen: |
@Maschga ja, sehe ich auch so. Habe auch Entwürfe gehabt wo das noch etwas abstrakter war: „Ladestrategie“. Wenn das noch weiter wachsen soll, werden wir da mehr Struktur brauchen. Aber Schritt für Schritt 😄 |
@naltatis rein damit und Wording später anpassen? |
Die UI ist aktuell wirklich nur ein "first shot". Das funktioniert nur auf großem Breakpoint wenns keine Repeating Plans gibt. Hier muss ich noch etwas Arbeit reinstecken. |
Mhhm. Gäbe es eine low-effort Variante, die neuen Einstellungen bei kleinen Breakpoints z.B. hinter [...] Button zuverstecken und nur bei Bedarf ein zweizeiligesLayout aufzumachen? Ich fürchte ganz ohne gibts zuviele Rückfragen. |
Lass mich doch erstmal den kleinen Breakpoint bauen 😂 |
Achso. Ich dacht wir diskutieren releasen or not. Ich wollte nur sagen: sollten wir nicht releasen ;) |
Top! |
@andig all done from my side. Rein damit? |
Klappt das auch wenn es für viele Stunden einen negativen Strompreise gibt? |
Ups, nein... aber wir können schnell 0 durch -Inf ersetzen ;) |
Fix #19690, replace #20409
This PR modifies the planner for "late start". If late start is enabled, the plan will always include the latest possible slot. Planner will ensure this simply by marking the latest slot as having zero price.
Note: Planner does not take into account how long this latest slot lasts.
TODO
preCondition
in seconds @andigrate.At
changes @andig