-
-
Notifications
You must be signed in to change notification settings - Fork 712
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 aux meter templates #11887
Comments
Mir ist der Sinn noch nicht ganz klar. Aktuell kann ich doch in AUX ganz normale Meter verwenden. Was gewinnen wir durch den neuen Typen? Müssten wir den dann nicht bei fast allen besthenden Metern auch nachtragen? |
Aux sind intelligente Geräte die ihre Leistung selbst regeln. Diese ist daher als "verfügbar" anzusehen. Für die Geräte brauchts Templates wie hier. Siehe #6106.
Das kannst Du auch weiter machen, ändert an der Verwendung aber nichts. |
Ok, ich glaub das war der Part der mir fehlte. Aktuell wird In der Web UI (und configure) ist es aktuell so, dass man nur die Templates angeboten bekommt, die für den Verwendungszweck auch passen. Bei Aux wäre das mit diesem PR dann erstmal nur das eine Gerät. Heißt wir müssten dann Stück für Stück die bestehenden Meter "nachlabeln" die für diesen Anwendungsfall heute schon benutzt werden weil bspw. Multipurpose (bspw Shelly 1pm an einem intelligenten Heizstab). Finde ich gut und auch deutlich besser als alle Meter die es gibt anzuzeigen. Ich hab im Epic Config UI #6029 mal Aux mit aufgenommen. Das fehlte da noch, bekommt mit diesem PR dann aber ne saubere Struktur. Außerdem hab ich dort noch mal mit aufgenommen, dass wir auch die Verwendung von Meter außerhalb ihrer vorgesehenen Usage erlauben sollten. Ob das dann über yaml-Snippet oder UI-geführt (weitere Geräte ...) passiert müssen wir dann mal sehen. |
Ok, aber dann haben wir zumindest ein sauberes Modell und eine Single Source of Truth, die wir vtml noch etwas nachpflegen müssen. |
Es wäre übrigens schick, für den Thor ein Template anzulegen. PR welcome.
Originally posted by @andig in #11885 (comment)
The text was updated successfully, but these errors were encountered: