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

Loadpoint: add welcomecharge feature #13534

Merged
merged 4 commits into from
Apr 25, 2024
Merged

Loadpoint: add welcomecharge feature #13534

merged 4 commits into from
Apr 25, 2024

Conversation

andig
Copy link
Member

@andig andig commented Apr 21, 2024

Fix #13423

This PR adds a kickcharge feature (are there better names)? When a vehicle connection is detected, and any of the vehicles that are currently not connected to any charger has this feature, then charging will start immediately if in PV mode.

This works such that vehicle defaults (like resetting mode) are only applied after starting the charging session.

@andig andig added enhancement New feature or request needs documentation Triggers issue creation in evcc-io/docs labels Apr 21, 2024
@VolkerK62
Copy link
Contributor

wenn kein Überschuss vorhanden, schaltet disable.delay das wieder aus?

@andig
Copy link
Member Author

andig commented Apr 21, 2024

Das ist die Idee- deshalb wäre nur einschalten neu.

@VolkerK62
Copy link
Contributor

wäre ein kurzer fester Wert ein Problem?
Ich kann mir vorstellen, dass es "Beschwerden" gibt .... "warum 3 Minuten, wenn doch 30 Sekunden reichen"

@andig
Copy link
Member Author

andig commented Apr 21, 2024

Viel mehr Logik -> erstmal nein.

@VolkerK62
Copy link
Contributor

recognitionCharge?
preCharge?

@naltatis
Copy link
Member

naltatis commented Apr 24, 2024

recognitionCharge

Den Vorschlag find ich besser. Der Bergriff erklärt "warum" das gerade gemacht wird. Allerdings ist recognition so ein kompliziertes wort 😆

Weitere Vorschläge, die klarmachen, dass das nur am Anfang einmal passiert:

  • startcharge
  • initcharge
  • welcomecharge

@andig
Copy link
Member Author

andig commented Apr 24, 2024

Welcome klingt doch sehr einladend?

@naltatis
Copy link
Member

Ja, ist etwas mehr Umarmung als Tritt in den Hintern 😄

@andig andig changed the title Loadpoint: add kickcharge feature Loadpoint: add welcomecharge feature Apr 24, 2024
@premultiply
Copy link
Member

preCharge finde ich neutral und gut.

Könnte man auch ggf. später für eine optionale automatische Charakterisierung der Fahrzeugladeeigenschaften von Gastfahrzeugen im Rahmen des Lastmanagements o.ä. nutzen.

@andig
Copy link
Member Author

andig commented Apr 25, 2024

Hatte ich auch erst überlegt, aber pre was? Ich fände weiter welcome/hello passend.

@RTTTC
Copy link
Contributor

RTTTC commented Apr 25, 2024

How about IDCharge ? This, similar to recognitionCharge explains why it is happening, while weclome charge would just confuse me personally ;)

Would this charge also happen if the loadpoint mode is off ??

@andig
Copy link
Member Author

andig commented Apr 25, 2024

Why? This is not about identifying anything but about making sure that it will accept a later energy offer.

@andig andig merged commit 1d4e695 into master Apr 25, 2024
7 checks passed
@andig andig deleted the feat/kickcharge branch April 25, 2024 11:36
@RTTTC
Copy link
Contributor

RTTTC commented Apr 25, 2024

HandShakeCharge then ?

@andig
Copy link
Member Author

andig commented Apr 25, 2024

A second too late :O. Happy to change it before we release!

@RTTTC
Copy link
Contributor

RTTTC commented Apr 26, 2024

Only this needs to be changed api/feature.go ?? I am not the optimal person to change something in the code :DD

@Andy4OS
Copy link
Contributor

Andy4OS commented Apr 28, 2024

Is this feature, if compiled in, active by default, or do I have to activate in the config? I would like to test this feature.

@andig
Copy link
Member Author

andig commented Apr 28, 2024

You‘ll need to add the welcomecharge vehicle feature to the vehicle‘s config.

@naltatis
Copy link
Member

@andig mir ist hier bisher nicht klar, wie du dir die Konfiguration vorstellst. Ich hätte gedacht, wir setzen diese Feature-Eigenschaft direkt in die Templates von dene wir wissen, dass diese Fahrzeuge sie benötigen (bspw. Dacia #11710). Also analog features: ["coarsecurrent"] im Tesla Template.

Ein allgemeines Hinzufügen zu beliebigen Template haben wir aktuell nicht. Da würde nur gehen, wenn der Nutzer mit den "echten Implamentierungstypen" hantiert und nicht mit type: template. Für Plugins ist diese Möglichkeit natürlich sinnvoll.

Also konkret sollten wir mMn folgende Dinge tun:

@premultiply
Copy link
Member

Beim Dacia ist das, wie bei den meisten Fahrzeugen, auch nur ein zwischenzeitlich gefixter Softwarebug.

Das Feature also brauchen nur vereinzelte Anwender deren Fahrzeug noch mit veralteter Software herumfährt.

@naltatis
Copy link
Member

Was ist denn dein Vorschlag? Es temporär fest ins Dacia Template einzubauen (transparent für Nutzer)? Oder im Dacia Template als opt-in mit Erklärung wann genau man das braucht?

Ich würd das ungern bei allen Fahrzeugen zu einer auswählbaren Option machen. Ist ja wenig sinnvoll wenn wir das bspw. beim Tesla anbieten, diese Funktion dort aber ehr verwirrt als wirklich hilft.

@premultiply
Copy link
Member

Wenn nur im Template als auswählbare Option.

andig added a commit that referenced this pull request Apr 29, 2024
@andig
Copy link
Member Author

andig commented Apr 29, 2024

Aktuell wäre es notwendig, das Fahrzeug direkt, also nicht über Template, zu konfigurieren. Am Beispiel i3:

features:
  - coarsecurrent
  - welcomecharge

@VolkerK62
Copy link
Contributor

Vehicle per type ist "doof" ;)
Idee:
ist zwar eine Fahrzeugeigenschaft, aber die Aktion betrifft doch den Loadpoint.
Wäre es am Loadpoint in der Form welcomecharge: true möglich?

@naltatis
Copy link
Member

Ja, sehe ich auch so. Wir wollen doch Richtung Template. Die nativen fahrzeugtypen sind ja auch (zu recht) nirgendwo öffentlich dokumentiert. Für für Plugins finde ich diese Ebene gut, aber für "ich will mein Fahrzeug auswählen und einstellen" brauchen wir Führung.

Daher wäre mein Vorschlag diese Option bei den Herstellern wo wir wissen, dass das ein Thema sein kann, einzubauen.

Den alternativen Vorschlag mit dem loadpoint find ich auch gut. Dann funktioniert das auch in Situationen (firmenparkplatz, ...) wo man das Fahrzeug nicht kennt.

@andig
Copy link
Member Author

andig commented Apr 30, 2024

Ich finds ein bisschen unglücklich, wenn die Featurediskussion jetzt nach dem Merge wieder anfängt. Loadpoint war hier kein Thema und der PR ist durch.

@naltatis
Copy link
Member

Wir diskutieren hier doch nicht ums Feature, sondern darum, wie der Nutzer das benutzen kann. Mir geht es darum, was wir noch ergänzen müssen, damit das ein rundes Bild gibt und das auch verständlich ist. Gerade ists das halt noch nicht.

Loadpoint: sehe ich auch gerade nicht als kritisch. Wenn es da Nachfrage gibt, sollten wir die Option aber beleuchten.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request needs documentation Triggers issue creation in evcc-io/docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Anladen" im PV Modus
6 participants