Fronius inverter production counter#2821
Conversation
| device_config: FroniusConfiguration | ||
|
|
||
|
|
||
| class FroniusProductionCount(AbstractInverter): |
There was a problem hiding this comment.
Warum bist Du von der üblichen Benennung mit Counter abgewichen?
There was a problem hiding this comment.
Die farbliche Darstellung in der Nutzeroberfläche unter "Geräte und Komponenten" hängt von der Benennung ab. Die Funktion getComponentTypeIcon(type) in der HardwareInstallation.vue führt eine regex Filterung durch.
In der if-else Verzweigung wird zurerst nach counter gefiltert. Dadurch wird die Komponente als Zähler (rot) statt als WR (grün) dargestellt, was hier nicht korrekt ist.
Will man die Logik nicht aufbohren bleibt nur die Umbenennung. Ich habe alles angepasst damit es in UI und Backend konsistent ist.
Besser nur den type anpassen?
There was a problem hiding this comment.
Wie wäre es mit "Meter" anstatt "Counter"? Wäre sogar vom Begriff her richtig. Counter ist eher "Denglisch".
There was a problem hiding this comment.
Es gibt auch noch diese Funktion
Damit wird aus dem Komponententyp, der weitestgehend indiviudell benannt werden kann, zugeordnet, ob es ein Zähler, WR, Speicher, ... ist. Die Regelung muss ja auch wissen, ist es als Zähler oder WR zu behandeln. Deshalb hatten wir damals festgelegt, dass immer counter, bat oder inverter im Komponententyp vorkommen muss.
…s/core into fronius-external-inverter
|
Hier im PR / Ticket gehts doch um externe Fronius WR/Erzeuger, ist die Änderung der bat_control hier versehentlich mit reingerutscht? Sollte das nicht besser ein eigener PR werden? Das hat doch mit dem eigentlichen Ticket/PR nichts zu tun? |
| counter_s0=create_counter_s0_component, | ||
| inverter=create_inverter_component, | ||
| inverter_secondary=create_inverter_secondary_component, | ||
| inverter_counter_production=create_inverter_production_meter_component, |
There was a problem hiding this comment.
Ist die Benennung so korrekt?
[Ticket #92122709]
Externe Erzeuger lassen sich über zusätzlichen Smartmeter erfassen und im Fronius als Erzeugerzähler definieren.
Aktuell lassen sich solche Erzeuger nur als Zwischenzähler und nicht als Wechselrichter einbinden.
Die Komponente "externer Wechselrichter" bildet das Szenario nicht ab
openWB/openwb-ui-settings#809