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 42 commits into
base: master
Choose a base branch
from
Open

Add loadpoint config api (BC) #12958

wants to merge 42 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

@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
Copy link
Member Author

andig commented Mar 29, 2024

@naltatis soc ist auch drin. Du müsstest rebasen ;)

@github-actions github-actions bot added the stale Outdated and ready to close label Apr 26, 2024
@andig andig removed the stale Outdated and ready to close label Apr 27, 2024
@github-actions github-actions bot added the stale Outdated and ready to close label May 18, 2024
@andig andig added backlog Things to do later and removed enhancement New feature or request stale Outdated and ready to close labels May 19, 2024
@naltatis naltatis mentioned this pull request May 31, 2024
@naltatis
Copy link
Member

naltatis commented Jun 8, 2024

@andig master ist drin, ui baut wieder. fehlt nur noch go :D

@andig
Copy link
Member Author

andig commented Jun 8, 2024

Wir haben irgendwas komisches mit der ThresholdsConfig gemacht, nicht sicher warum.

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
None yet
Development

Successfully merging this pull request may close these issues.

UI: Make loadpoints configurable
2 participants