Skip to content

Conversation

@ndrsnhs
Copy link
Contributor

@ndrsnhs ndrsnhs commented Oct 15, 2025

[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

device_config: FroniusConfiguration


class FroniusProductionCount(AbstractInverter):
Copy link
Contributor

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?

Copy link
Contributor Author

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?

Copy link
Contributor

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".

Copy link
Contributor

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.

@seaspotter
Copy link
Contributor

seaspotter commented Oct 17, 2025

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,
Copy link
Contributor

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?

@LKuemmel LKuemmel merged commit def184b into openWB:master Oct 27, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants