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 Retryable feature to indicate vehicles that are temporarily offline #12841

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

andig
Copy link
Member

@andig andig commented Mar 10, 2024

Goal of this PR is to make minsoc/limitsoc editable for broken vehicles. To avoid an intermediate step, we're combining this with moving the vehicle settings from arrival tab to config ui.

Fix #12460.
This is an intermediate step. Full solution depends on #11572.

Changes:

  • ☁️ show unreachable icon instead of "$title (unavailable)"-text. Added tooltip and link to restart dialog.
  • ♻️ fixed tooltip and rotation animation for autodetect state (new icon, no wobbling)

fixed autodetect animation

detect.mp4

new unreachable icon

not.reachable.mp4

TODO

@andig andig added the enhancement New feature or request label Mar 10, 2024
@naltatis
Copy link
Member

Ich bin mir noch nicht sicher, ob wir hier nicht zu sehr von der Implementierungsseite denken:

"Offline Fahrzeug, dass Retryable ist und wieder ein Online Fahrzeug werden kann" vs. "Online Fahrzeug, dass vorübergehend nicht erreichbar ist"

Ich weiß, ist erstmal "nur" Wording. Wird sich aber durch alle Stellen/Logiken im Code durchziehen wo wir auf diesen Zustand speziell reagieren müssen. Und die sind jetzt ja schon nicht immer einfach zu verstehen.

Aktueller Zustand: Wir behandeln das Fahrzeug größtenteils wie ein Offline Fahrzeug (kWh-Limits, ...), haben aber auch Online-Aspekte (minSoC).

Für mich steht und fällt das mit der Frage ob dieser Zustand in der Praxis eher was temporäres ist (Server nicht erreichbar, Auto hat keinen Empfang), oder ob das überwiegend ein permanenter Zustand (API hat sich geändert, Warten auf evcc Bugfix) ist. Bei letzterem find ich den hier beschriebenen Weg gut, wenns eher das erste ist, würde ich es auch anders herum modellieren.

@andig
Copy link
Member Author

andig commented Mar 11, 2024

ob dieser Zustand in der Praxis eher was temporäres ist

Der Zustand ist permanent, solange bis es Lifecycle gibt. Wenns den gibt dann brauchts zusätzlich einen Retry Mechanismus der eingreift wenn ein Fahrzeug dieses Feature (oder ein anderes neues API) hat.

@andig andig changed the title Add Retryable feature to indicate vehicles that are temporarily offline Move vehicle settings ui, add Retryable feature to indicate vehicles that are temporarily offline Mar 13, 2024
@naltatis
Copy link
Member

@andig was hältst du davon den jetzigen Stand schon mal zu mergen? Das bringt für sich schon eine Verbesserung (Darstellung des Offline+Reachable Zustands). Dann wir das nicht so ein Riesenchange.

Die verlinkten Issues wären damit noch nicht gelöst, dafür würden wir nen neuen PR aufmachen.

@andig
Copy link
Member Author

andig commented Mar 13, 2024

@naltatis ich traus mich kaum zu sagen: das ist jetzt aber wieder zurück zu meinem Vorschlag, plus kein Icon? Geht gleich rein, muss nur die Referenzen bearbeiten. 👍🏻

@naltatis
Copy link
Member

das ist jetzt aber wieder zurück zu meinem Vorschlag, plus kein Icon?

Weiß nicht genau worauf du dich beziehst. Wir haben so vieles besprochen 😅

@andig andig changed the title Move vehicle settings ui, add Retryable feature to indicate vehicles that are temporarily offline Add Retryable feature to indicate vehicles that are temporarily offline Mar 14, 2024
@andig andig merged commit df57f17 into master Mar 14, 2024
6 checks passed
@andig andig deleted the feat/feature-retryable branch March 14, 2024 09:13
thierolm pushed a commit to thierolm/evcc that referenced this pull request Mar 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Min.Ladung% lässt sich nicht ändern wenn Fahrzeug Offline
2 participants