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 loadpoint config api (BC) #12958

Open
wants to merge 152 commits into
base: master
Choose a base branch
from
Open

Add loadpoint config api (BC) #12958

wants to merge 152 commits into from

Conversation

andig
Copy link
Member

@andig andig commented Mar 15, 2024

Fix #12903

This PR deprecates the following yaml loadpoint settings:

  • mode
  • title
  • phases
  • min/max current
  • priority

TODO

  • UI for loadpoint properties @naltatis
  • UI for adding/changing charger @naltatis
  • UI for adding/changing lp meter @naltatis
  • e2e tests @naltatis
  • add translations @naltatis
  • UI default vehicle select @naltatis
  • UI circuit assignment @naltatis
  • API for assignable circuits @andig
  • API set meter, charger, defaultVehicle @andig
  • UI for adding/removing loadpoints @naltatis
  • API for adding/remove loadpoints @andig
  • implement config as settings
  • unable to save priority=0 @andig
  • remove orphan meters/chargers (no site/loadpoint ref) on startup
  • fix unit tests @andig
  • fix initial lp saving issue (logger, settings) @andig
  • store loadpoint settings in config (based on chore: persist device details as json #15224) @andig
  • handle pointer values like smartcostlimit
  • decide/adapt migration strategy (use evcc migrate?) @andig @naltatis
    • evcc migrate --reset removes all ui configured entities (vehicles, meters, chargers, loadpoints)
    • migrating these entities from yaml to ui will be done in a separate issue later
  • demo mode solution @naltatis
  • loadpoint settings adapter (config & runtime values) @andig

@andig andig added the enhancement New feature or request label Mar 15, 2024
@andig andig requested a review from naltatis March 15, 2024 09:17
@andig andig changed the title Add loadpoint config api Add loadpoint config api (BC) Mar 15, 2024
@naltatis
Copy link
Member

@andig die APIs sehen gut aus. Funktioniert alles wie es soll. Wir hatten ja gesagt, wir wollten den Scope klein halten und Vehicle/Charger erstmal ausklammern. Ich glaub es ist doch ne gute Idee, wenn wir die Referenz-Felder hier gleich mit aufnehmen. Sonst haben wir einen komischen Zwischenstand den man schwer releasen kann. Magst du die beiden Felder ergänzen?

Bildschirmfoto 2024-03-25 um 09 04 49

@andig
Copy link
Member Author

andig commented Mar 25, 2024

Mache ich. Am Fahrzeug könnten wir dann noch maxCurrent1p und minCurrent3p ergänzen 🙃.

@naltatis
Copy link
Member

↔ wider layout (general)
🏷️ loadpoint status

Bildschirmfoto 2024-03-25 um 20 20 24

@andig
Copy link
Member Author

andig commented Mar 25, 2024

Ich glaub es ist doch ne gute Idee, wenn wir die Referenz-Felder hier gleich mit aufnehmen.

So, jetzt nochmal langsam. Was meinst Du damit konkret? Phases z.B. ist doch drin?

@naltatis
Copy link
Member

Mit Referenz Felder meine ich:

  • lp.vehicle
  • lp.charger
  • lp.meter

Da stehen dann die 'name's der am loadpoint verknüpften devices drin und sind darüber auch änderbar.

@andig
Copy link
Member Author

andig commented Mar 26, 2024

Ah, ok. Aber erstmal ohne Updatefähigkeit?

@naltatis
Copy link
Member

Wie es passt. Update wäre cool (bspw default Fahrzeug). Lesen wäre aber auch ein Fortschritt.

@naltatis
Copy link
Member

@andig wollen wir, wo wir gerade dabei sind, nicht auch gleich soc und enable/disable mit reinnehmen? Dann sind wir hier durch und wir haben keinen Zwischenstand, den wir wieder erklären müssen. Könnten wir einfach flach abbilden. So viele Werte sind das ja glücklicherweise nicht.

  • socPollMode
  • socPollInterval
  • socEstimate
  • enableThreshold
  • enableDelay
  • disableThreshold
  • disableDelay

@andig andig force-pushed the feat/loadpoint-config branch from fd7e1e3 to 32550fe Compare December 6, 2024 17:27
@naltatis
Copy link
Member

Setup von evcc mit einer leeren evcc.yaml 🎉

setup.webm

@naltatis
Copy link
Member

@andig ich hab gerade noch mal ausprobiert wie sich die Einstellungen von UI-konfigurierten Fahrzeugen an UI-konfigurierten Ladepunkten verhalten. Das sieht alles korrekt, bzw. wie erwartet aus. Die fahrzeugspezifischen Werte (bspw. vehicle.db:329.planSoc) landen weiterhin gepräfixt in der settings Tabelle. Heißt das Fahrzeug kann zwischen den Ladepunkte weiterhin hin und verwechseln ohne dass wir Daten verlieren.

Dementsprechend steht hier nur noch eine gute Lösung für den Demo-Modus aus. Hier tendiere ich gerade dazu das aktuelle Verhalten erst einmal zu belassen:

  • keine evcc.yaml > Demo Modus
  • existierende evcc.yaml > Normaler Modus (Welcome Dialog, wenn keine Ladepunkte)

Ein explizites Anwählen des Demo Modus (bspw. durch CLI Param) würd ich erst dann machen, wenn wir Config UI nicht mehr hinter experimental Flag haben.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backlog Things to do later prio Priority
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

UI: Make loadpoints configurable
4 participants