-
-
Notifications
You must be signed in to change notification settings - Fork 518
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
base: master
Are you sure you want to change the base?
Conversation
@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? |
Mache ich. Am Fahrzeug könnten wir dann noch |
So, jetzt nochmal langsam. Was meinst Du damit konkret? Phases z.B. ist doch drin? |
Mit Referenz Felder meine ich:
Da stehen dann die 'name's der am loadpoint verknüpften devices drin und sind darüber auch änderbar. |
Ah, ok. Aber erstmal ohne Updatefähigkeit? |
Wie es passt. Update wäre cool (bspw default Fahrzeug). Lesen wäre aber auch ein Fortschritt. |
@andig wollen wir, wo wir gerade dabei sind, nicht auch gleich
|
Die Refs sind drin (read-only). |
Mein eigentlicher Gedanke war, dass wir, wenn wir soc/enable/disable mit dazunehmen eine einfach verständliche Story für den Anwender haben: "Alle Loadpoint Einstellungen sind jetzt im UI." Deutlich einfacher zu verstehen als ein Misch-Setup wo sich von Release zu Release die Verhältnisse verändern.
Wäre für mich bspw. ein Deal Breaker. Ich nutze hier für das Zusammenspiel von EV und Heizstab bewusst nicht die Defaults. Soc-Settings ans Fahrzeug migrieren find ich auch gut, aber würde ich in einem separaten Schritt machen. Hier wäre mein Fokus erstmal "nur" yaml-Einstellungen zu UI-Einstellungen konvertieren. |
@andig Wenn Das eigentliche Problem ist, dass Speichern von Priority != 0 funktioniert. Schaust du da noch mal rein? |
Verstehe ich nicht. Dir LPs brauchts weiter in der yaml, also sind auch die Settings noch da? |
Ja, mir ging es ja um diesen Punkt:
|
Yoah, aber breaking ist da nix. Schöner wärs. |
@naltatis ist behoben- auch 0 wird jetzt ausgegeben. Spannend ist allerdings auch |
Das Problem hat smartCostLimit ja in der aktuellen api auch schon. 0 bedeutet "aus". Negative Werte sind erlaubt und auch gewünscht. |
@naltatis soc ist auch drin. Du müsstest rebasen ;) |
Fix #12903
This PR deprecates the following yaml loadpoint settings:
TODO