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

Easee: never stop session, only pause charge #10077

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

GrimmiMeloni
Copy link
Collaborator

@GrimmiMeloni GrimmiMeloni commented Sep 26, 2023

User feedback on #9271 was given, that the change from pause to stop command is actually causing some annoyance. Background to this is, that stopping a charge, puts the charger back into the "unauthenticated" state. This has the side effect, that the LED strip on the charger starts blinking. Additionally when the charge is restarted later, the re-authentication is confirmed with an audible sound. When clouded weather is changing the charge status several times, this apparently becomes annoying to some.

This change simply removes the code path for the authenticated charger, that it uses stop command instead of pause. Using pause keeps the existing authentication, but this is OK, as it is sufficient to authenticate once on connection of a new vehicle. On disconnect the Easee de-authenticates anyways. For the charge (re)start this is also OK.

This change was tested already by me locally and is looking good to go.

@premultiply
Copy link
Member

Ich glaube ich hatte zu Beginn der Easee-Änderungen genau davor gewarnt. 🙂
Ferngesteuerte Sessions sind meist problematisch und bringen nicht zuletzt u. a. auch das interne Ladelog einer Box durcheinander und machen die Nutzung von echten RFID-Tags meist unmöglich.

@GrimmiMeloni
Copy link
Collaborator Author

Ich kann mich nicht daran erinnern, daß irgendjemand vor Störungen durch visuelle oder akustische Signale gewarnt hat. Probleme mit den Sessions hat auch bisher niemand erwähnt.
Was möchtest Du mit diesem Kommentar erreichen?

@premultiply
Copy link
Member

Nichts.
Wollte nur ausdrücken dass diese Änderung sehr sinnvoll ist und wir ähnliche Erfahrungen auch bei anderen Boxen schon hatten. Denn grundsätzlich ist die Sessionerzeugung und -beendigung Sache der Box. Externe Manipulationen führen hier in der Regel irgendwo zu unerwarteten Effekten und Problemen - auch wenn es dazu manchmal irgendwelche APIs oder Register gibt.

@andig andig added the enhancement New feature or request label Sep 27, 2023
@andig
Copy link
Member

andig commented Sep 27, 2023

OT: wollen wir das bei Keba egtl. Raus schmeissen?

@andig andig merged commit fc0bcf9 into evcc-io:master Sep 27, 2023
@premultiply
Copy link
Member

Wäre nochmal getrennt zu diskutieren was da genau das Szenario war.
Tendenziell eher ja.

@allcoolusernamesaregone
Copy link
Contributor

Nichts. Wollte nur ausdrücken dass diese Änderung sehr sinnvoll ist und wir ähnliche Erfahrungen auch bei anderen Boxen schon hatten. Denn grundsätzlich ist die Sessionerzeugung und -beendigung Sache der Box.

Die Session endet aber überhaupt nicht bei stop_charging. Die Session in der Easee endet mit Abstecken des Fahrzeugs.
Der Change hier ist zwar schon ok, kein Problem damit. Aber bis auf das Gepiepe und Geblinke ändert sich nichts positiv..
Wobei wohl auch nur das Gepiepe wirklich ein Problem sein könnte. Das Blinken zeigt im Gegenteil eher nützlich an, dass eben nicht am Laden. Bei pause_charging ist der LED Streifen unverändert dauerhaft an. Und: mit dem CHARGER_OP_MODE 7 ist definitiv klar: kein Laden. Bei pause_charging ist man immer auf den REASON_FOR_NO_CURRENT angewiesen (#9302).
Anders gesagt: ohne Piepen wäre stop_charging dem pause_charging weitaus überlegen.. Schade, dass man das nicht deaktivieren kann.

@GrimmiMeloni GrimmiMeloni deleted the fix/easeeOnlyPauseNotStop branch October 20, 2023 14:36
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.

4 participants