-
Notifications
You must be signed in to change notification settings - Fork 107
Fronius inverter production counter #2821
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
Conversation
| device_config: FroniusConfiguration | ||
|
|
||
|
|
||
| class FroniusProductionCount(AbstractInverter): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Warum bist Du von der üblichen Benennung mit Counter abgewichen?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Es gibt auch noch diese Funktion
| def type_to_topic_mapping(component_type: str) -> str: |
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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