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

Restore persistence of limit soc and energy #10979

Merged
merged 3 commits into from
Dec 8, 2023

Conversation

andig
Copy link
Member

@andig andig commented Dec 3, 2023

Leftover from #10335

@andig andig added the enhancement New feature or request label Dec 3, 2023
@andig andig requested a review from naltatis December 3, 2023 11:17
@naltatis
Copy link
Member

naltatis commented Dec 3, 2023

Ganz so einfach ist das leider nicht. Ich hatte mir die Stelle ja auch schon angeschaut.

Szenario:

  • Nutzer setzt limitEnergy bei Offline Fahrzeug
  • Fahrzeugwechsel: auf SoC-Fahrzeug

Bislang wurde das Limit gelöscht. Jetzt würde limitEnergy bestehen bleiben. Das wäre aus meiner Sicht auch ok, wenn wir garantieren, dass limitEnergy bei SoC-Fahrzeugen ignoriert wird. Sonst wirds nämlich komisch für den Nutzer. Weißt du ob das so ist? Wenn nicht müssen wir beim Wechsel zwischen Fahrzeug-Klassen (mit SoC ohne SoC, bzw. LP-SoC 🙈) wieder gezielt nullen.

@naltatis
Copy link
Member

naltatis commented Dec 3, 2023

Den fehlschlagenden Test hab ich gefixt. Das Verhalten, dass beim Wechsel zwischen Non-SoC Fahrzeugen das Limit erhalten bleibt finde ich erstmal ok und auch erwartungskonform.

tests/limits.spec.js Outdated Show resolved Hide resolved
@andig
Copy link
Member Author

andig commented Dec 5, 2023

Den fehlschlagenden Test hab ich gefixt

Also nochmal beratschlagen oder rein damit?

@naltatis
Copy link
Member

naltatis commented Dec 6, 2023

Für mich ist diese Frage noch offen:

Weißt du ob das so ist?

Wenn ein gesetzte energyLimit bei Fahrzeugen mit SoC keinen Effekt hat (und umgekehrt), dann kann das rein, wie es ist. Wenn nicht, dann müssen wir beim Fahrzeugwechsel das jeweils andere Limit wieder zurücksetzen.

@andig
Copy link
Member Author

andig commented Dec 6, 2023

Wenn ein gesetzte energyLimit bei Fahrzeugen mit SoC keinen Effekt hat (und umgekehrt), dann kann das rein, wie es ist.

Das sollte so sein- hatten wir m.E. mal zusammen glatt gezogen. Falls wir das durch einen Test absichern könnten wäre das natürlich grossartig. Ich habe aber keine gute Idee :/

@andig andig merged commit ff38abf into master Dec 8, 2023
6 checks passed
@andig andig deleted the fix/restore-limit-soc-energy-persistence branch December 8, 2023 16:28
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.

2 participants