-
-
Notifications
You must be signed in to change notification settings - Fork 698
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
UI Planner falsche Anzeige wenn Ziel nicht erreicht. #17704
Comments
Das wäre ja erstmal richtig- macht der Planner sonst auch. Er sollte in diesem Fall aber weiter „heute“ als Ziel zeigen? |
Ja, genau. Ansonsten verwirrt das - meiner Meinung nach - den Benutzer. |
@TobiasHuber1980 gibts ein Logfile? Ich vermute hier wird bei repeating plans der alte Plan bereits beendet was bei laufendem Plan nicht passieren dürfte. |
@andig ist technisch ein etwas größeres Thema. Wir haben die Logik, dass laufende Pläne nicht abgebrochen werden sofern das Ziel noch nicht erreicht ist. Das ist das was hier auch zum richtigen Ladeverhalten führt. Für die Anzeige, aber auch um den Plan SoC auf den geladen wird, zu ermitteln betrachten wir momentan immer stumpf die aktuelle Uhrzeit und suchen dann den nächsten Plan raus. In dem Beispiel von @TobiasHuber1980 wird die Ladung nicht abgebrochen weil seine nächste Planzeit (morgen 15:30) auch 80% als Ziel hat. Wäre der nächste Plan unter dem aktuellen Ladestand gewesen hätte er abgebrochen. Heißt wir müssen an die Anzeige und ans Verhalten ran. Ich bin mir noch nicht ganz sicher, was hier eine gute und nicht zu komplizierte technische Lösung ist. Vmtl. müssen wir die Infos zum aktiven Ladevorgang (aktiv wie "in progress"), also das Ladeziel und den Zeitpunkt noch mal als Kopie halten und erst beim "finish" wieder auf die Suche nach dem nächsten Ladeziel anhand der aktuellen Zeit switchen. |
Ja, solange der Ladevorgang des bereits "abgelaufenen" Planners noch läuft, sollte wie beim bisherigen Planner "die Restzeit/Ziel wann" in orange angezeit werden. Wäre mich mich logischer (so wie im Screenshot.) Log:
|
Wir müssen wohl das aktuelle Ziel mit publishen um die Logik aus dem UI rauszuhalten? |
@andig Das aktuelle Ziel wird mWn. schon separat für die UI gepublisht. "Nur" die Logik muss angepasst werden. |
@Maschga
Planner heute auf 15:30 gestellt, Ladeziel konnte auf Grund zu wenig Zeit bis Zieluhrzeit nicht erreicht werden. UI spring dann auf "morgen" lädt aber trotzdem mit "schnell" weiter um das Ziel heute noch zu erreichen.
Slack Message
The text was updated successfully, but these errors were encountered: