Feature local consumption counter by AlexanderHa98 · Pull Request #3660 · openWB/core

AlexanderHa98 · 2026-07-14T07:58:33Z

openWB/openwb-ui-settings#1013

Damit die Github check-action durchläuft, muss ein Test angepasst werden:
#3662

Aufbau

Neues Gerät

Fügt das neue Gerät: Verbrauchszähler mit lokaler Auslegung hinzu.

Neue Komponente

Fügt die Komponente openWB Lokaler Verbrauchszähler hinzu.

GUI

Gerät

Keine Konfigurationsmöglichkeiten.
Der verwendete USB-Port wird fest im Code gesetzt.

Komponente

Das Zählermodell ist fest vorgegeben, da derzeit nur der SDM630_72 unterstützt wird.
Die Modbus-ID muss konfiguriert werden.

Funktionsweise

Die Implementierung orientiert sich am bestehenden ConsumptionCounter aus openwb_flex.

Unterschiede:

Die Modbus-Kommunikation erfolgt seriell statt über TCP.
Das silent_interval ist konfigurierbar.

Silent_Intervall

ConsumptionCounter (openwb_flex): Standardwert 0.1
Lokaler ConsumptionCounter: Standardwert 0.06

Testergebnisse

Getestet mit einem realen Aufbau aus 10 Zählern.

`silent_interval = 0.06` (60 ms)

Auslesen von 10 Zählern: funktioniert nicht
- device2 konnte nicht innerhalb des konfigurierten Timeouts abgearbeitet werden.
Auslesen von 9 Zählern: funktioniert sporadisch
Auslesen von 8 Zählern: funktioniert zuverlässig

`silent_interval = 0.03` (30 ms)

Auslesen von 10 Zählern: funktioniert zuverlässig

Copilot

Pull request overview

Diese PR führt ein neues openWB-Gerät „openwb_flex_local“ ein, um einen lokalen Verbrauchszähler per Modbus RTU (seriell) auszulesen (angelehnt an den bestehenden openwb_flex ConsumptionCounter), und erweitert den SDM630_72-Treiber um ein einstellbares Lese-Intervall.

Changes:

Neues Gerät/Modul openwb_flex_local inkl. Device-, Config-, Versions- und ConsumptionCounter-Komponente (serieller Modbus-Client).
sdm.Sdm630_72: silent_interval als Parameter eingeführt und in den Lese-Pfaden statt fixer time.sleep(0.1) verwendet.
elgris.Elgris: silent_interval-Attribut ergänzt, um Kompatibilität mit dem neuen sdm.Sdm630_72-Verhalten sicherzustellen.

Reviewed changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
packages/modules/devices/openwb/openwb_flex_local/versions.py	Factory-Funktionen für unterstützte Zählertypen (aus openwb_flex gespiegelt).
packages/modules/devices/openwb/openwb_flex_local/device.py	Neues ConfigurableDevice mit Modbus-RTU Initializer und Component-Updater.
packages/modules/devices/openwb/openwb_flex_local/consumption_counter.py	Neue LocalConsumptionCounter-Komponente (serielle Modbus-Auslesung).
packages/modules/devices/openwb/openwb_flex_local/config.py	Setup-/Configuration-Klassen für Device und Komponente.
packages/modules/devices/openwb/openwb_flex_local/init.py	Package-Datei für das neue Modul.
packages/modules/devices/elgris/elgris/elgris.py	Ergänzt `silent_interval`, damit geerbte SDM630_72-Reads funktionieren.
packages/modules/common/sdm.py	SDM630_72: `silent_interval` parametrierbar gemacht und genutzt.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

+        factory = consumption_counter_factory(self.component_config.configuration.type)
+        self.fault_state = FaultState(ComponentInfo.from_component_config(self.component_config))
+        self.__client = factory(self.component_config.configuration.id, self.__serial_client,
+                                self.fault_state, silent_interval=0.06)
+        self.sim_counter = SimCounter(self.__device_id, self.component_config.id, prefix="bezug")


+class FlexLocalConfiguration:
+    def __init__(self, port: str = '/dev/ttyACM0') -> None:
+        self.port = port


LKuemmel · 2026-07-14T10:39:48Z

+    def error_handler():
+        run_command([f"{Path(__file__).resolve().parents[4]}/modules/common/restart_protoss_admin"])


Suggested change

def error_handler():

run_command([f"{Path(__file__).resolve().parents[4]}/modules/common/restart_protoss_admin"])

Einen Protoss gibt es in dieser Installation nicht.

LKuemmel · 2026-07-14T10:40:01Z

+
+class LocalConsumptionCounterSetup(ComponentSetup[LocalConsumptionCounterConfiguration]):
+    def __init__(self,
+                 name: str = "openWB Lokaler-Verbrauchszähler",


Suggested change

name: str = "openWB Lokaler-Verbrauchszähler",

name: str = "openWB Lokaler Verbrauchszähler",

LKuemmel · 2026-07-14T10:44:16Z

+
+class FlexLocalConfiguration:
+    def __init__(self) -> None:
+        self.port = '/dev/ttyACM0'


Am besten wäre es, wenn wir die client detection von den Internen Ladepunkten so umstrukturieren, dass wir sie hierfür auch nutzen können, dann braucht man einen neuen USB-Adapter /dev/ttyXXX nur an einer Stelle einfügen und kann ihn für alles verwenden.

LKuemmel · 2026-07-14T10:47:26Z

+def kit_counter_version_factory(
+        version: int) -> Type[Union[mpm3pm.Mpm3pm, lovato.Lovato, sdm.Sdm630_72, b23.B23]]:
+    if version == 0:
+        return mpm3pm.Mpm3pm
+    elif version == 1:
+        return lovato.Lovato
+    elif version == 2:
+        return sdm.Sdm630_72
+    elif version == 3:
+        return b23.B23
+    else:
+        raise ValueError("Version "+str(version) + " unbekannt.")
+
+
+def kit_inverter_version_factory(
+        version: int) -> Type[Union[mpm3pm.Mpm3pm, lovato.Lovato, sdm.Sdm630_72, sdm.Sdm120]]:
+    if version == 0:
+        return mpm3pm.Mpm3pm
+    elif version == 1:
+        return lovato.Lovato
+    elif version == 2:
+        return sdm.Sdm630_72
+    elif version == 3:
+        return sdm.Sdm120
+    else:
+        raise ValueError("Version "+str(version) + " unbekannt.")
+
+
+def kit_bat_version_factory(version: int) -> Type[Union[mpm3pm.Mpm3pm, sdm.Sdm630_72, sdm.Sdm120]]:
+    if version == 0:
+        return mpm3pm.Mpm3pm
+    elif version == 1:
+        return sdm.Sdm120
+    elif version == 2:
+        return sdm.Sdm630_72
+    else:
+        raise ValueError("Version "+str(version) + " unbekannt.")


Suggested change

def kit_counter_version_factory(

version: int) -> Type[Union[mpm3pm.Mpm3pm, lovato.Lovato, sdm.Sdm630_72, b23.B23]]:

if version == 0:

return mpm3pm.Mpm3pm

elif version == 1:

return lovato.Lovato

elif version == 2:

return sdm.Sdm630_72

elif version == 3:

return b23.B23

else:

raise ValueError("Version "+str(version) + " unbekannt.")

def kit_inverter_version_factory(

version: int) -> Type[Union[mpm3pm.Mpm3pm, lovato.Lovato, sdm.Sdm630_72, sdm.Sdm120]]:

if version == 0:

return mpm3pm.Mpm3pm

elif version == 1:

return lovato.Lovato

elif version == 2:

return sdm.Sdm630_72

elif version == 3:

return sdm.Sdm120

else:

raise ValueError("Version "+str(version) + " unbekannt.")

def kit_bat_version_factory(version: int) -> Type[Union[mpm3pm.Mpm3pm, sdm.Sdm630_72, sdm.Sdm120]]:

if version == 0:

return mpm3pm.Mpm3pm

elif version == 1:

return sdm.Sdm120

elif version == 2:

return sdm.Sdm630_72

else:

raise ValueError("Version "+str(version) + " unbekannt.")

Die brauchen wir hier nicht.

LKuemmel · 2026-07-15T09:55:15Z

+        if self.kwargs['client'] is None:
+            raise Exception("Verbrauchszähler mit lokaler Auslesung konnte Port nicht ermitteln, ")


Suggested change

if self.kwargs['client'] is None:

raise Exception("Verbrauchszähler mit lokaler Auslesung konnte Port nicht ermitteln, ")

raise Exception("USB-Adapter für die lokale Auslesung unbekannt oder nicht eingesteckt.")

So ist es für den Support und Endkunden besser verständlich, was konkret das Problem ist.

AlexanderHa98 requested a review from Copilot July 14, 2026 08:27

Copilot started reviewing on behalf of AlexanderHa98 July 14, 2026 08:27 View session

Copilot AI reviewed Jul 14, 2026

View reviewed changes

LKuemmel requested changes Jul 14, 2026

View reviewed changes

AlexanderHa98 force-pushed the feature_local_ConsumptionCounter branch from 342238f to 874a277 Compare July 14, 2026 12:59

LKuemmel requested changes Jul 15, 2026

View reviewed changes

AlexanderHa98 added 4 commits July 15, 2026 12:01

Add Local_ConsumptionCounter

c94a12e

Add suggestions und get_serial_modbus_devices

e885b62

Update device config

5c4e4f5

Add Exception für unkown port

7189bc9

AlexanderHa98 force-pushed the feature_local_ConsumptionCounter branch from 4205035 to 7189bc9 Compare July 15, 2026 10:09

Update Exception msg

881b2f5

LKuemmel approved these changes Jul 15, 2026

View reviewed changes

AlexanderHa98 added 2 commits July 16, 2026 08:58

Add Topic to hardware-installation-access

e0ee698

Fix Error

586636d

AlexanderHa98 requested a review from ndrsnhs July 17, 2026 07:58

AlexanderHa98 added 2 commits July 21, 2026 07:57

Add FaultSate für Komponenten im Timeout

689a389

Fix line length

e7ba182

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Feature local consumption counter - #3660

Feature local consumption counter#3660
AlexanderHa98 wants to merge 9 commits into
openWB:masterfrom
AlexanderHa98:feature_local_ConsumptionCounter

AlexanderHa98 commented Jul 14, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

LKuemmel Jul 14, 2026

Uh oh!

LKuemmel Jul 14, 2026

Uh oh!

LKuemmel Jul 14, 2026

Uh oh!

LKuemmel Jul 14, 2026

Uh oh!

LKuemmel Jul 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

		def error_handler():
		run_command([f"{Path(__file__).resolve().parents[4]}/modules/common/restart_protoss_admin"])

	name: str = "openWB Lokaler-Verbrauchszähler",
	name: str = "openWB Lokaler Verbrauchszähler",

		if self.kwargs['client'] is None:
		raise Exception("Verbrauchszähler mit lokaler Auslesung konnte Port nicht ermitteln, ")

	if self.kwargs['client'] is None:
	raise Exception("Verbrauchszähler mit lokaler Auslesung konnte Port nicht ermitteln, ")
	raise Exception("USB-Adapter für die lokale Auslesung unbekannt oder nicht eingesteckt.")

Uh oh!

Conversation

AlexanderHa98 commented Jul 14, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Aufbau

Neues Gerät

Neue Komponente

GUI

Gerät

Komponente

Funktionsweise

Unterschiede:

Silent_Intervall

Testergebnisse

silent_interval = 0.06 (60 ms)

silent_interval = 0.03 (30 ms)

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

LKuemmel Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

LKuemmel Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

LKuemmel Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

LKuemmel Jul 14, 2026

Choose a reason for hiding this comment

Uh oh!

LKuemmel Jul 15, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

AlexanderHa98 commented Jul 14, 2026 •

edited

Loading

`silent_interval = 0.06` (60 ms)

`silent_interval = 0.03` (30 ms)