EventAggregator: Patch für konfigurierbare Dezimalstellen und Laufzeit-Parameter

[wurmthomas80]

Hallo zusammen,

ich habe den EventAggregator für meinen Anwendungsfall erweitert und möchte die Änderung hier zur Diskussion stellen bzw. der Allgemeinheit zur Verfügung stellen.

Motivation

Standardmäßig ist beim EventAggregator die Anzahl der ausgegebenen Dezimalstellen nicht konfigurierbar, und Änderungen bestimmter Parameter erfordern ein Neuinitialisieren bzw. Neustarten.
Für meine Anwendung (kontinuierliche Messwerte / dynamische Konfiguration) war es notwendig:

• die Anzahl der Dezimalstellen frei festlegen zu können
• Parameteränderungen zur Laufzeit zu ermöglichen, ohne Neustart oder Recreate
• bestehende Funktionalität unverändert zu lassen (abwärtskompatibel)

Umsetzung

Ich habe dazu Anpassungen in folgenden Dateien vorgenommen:

• fhem.pl
• TimeSeries.pm

Die Änderungen ermöglichen:

• Angabe der gewünschten Dezimalstellen
• Übernahme geänderter Parameter während des Betriebs
• keine Auswirkungen auf bestehende Konfigurationen ohne Nutzung der neuen Optionen

Patch

Bei Interesse kann ich gerne den Patch zur verfügung stellen.

Fragen / Feedback erwünscht

• Ist Interesse vorhanden, diese Funktion offiziell zu unterstützen?
• Gibt es Bedenken hinsichtlich Nebenwirkungen oder Design?
• Sollte das anders implementiert werden (z. B. als Modul-Erweiterung statt Core-Änderung)?

Falls gewünscht, kann ich auch eine sauber aufbereitete Version oder Dokumentation nachreichen.

Viele Grüße
Thomas

[rudolfkoenig]

Gibt es Bedenken hinsichtlich Nebenwirkungen oder Design?

Um das zu beurteilen muesste man die Aenderungen sehen koennen.

[betateilchen]

Patch

Bei Interesse kann ich gerne den Patch zur verfügung stellen.

Hey, Du hast doch Interesse daran und Du möchtest Feedback haben.

Where's the beef?

[wurmthomas80]

Hallo,

hier sind wie angekündigt die Patches.

Mir ging es in erster Linie darum, die Änderungen der Allgemeinheit zur Verfügung zu stellen und ggf. zur Aufnahme in den offiziellen Code anzubieten, damit sie bei zukünftigen Updates nicht verloren gehen.

Die Erweiterungen laufen bei mir seit einiger Zeit stabil im produktiven Einsatz. Feedback ist natürlich willkommen, zwingend benötige ich es jedoch nicht – falls kein Interesse besteht, nutze ich die Anpassungen einfach weiterhin lokal.

Inhalt der Patches

1) Unterstützung für konfigurierbare Dezimalstellen beim event-aggregator 
2) Übernahme von Parameteränderungen zur Laufzeit (Reinitialisierung der TimeSeries-Instanz bei Änderungen)


Falls weitere Informationen, Tests oder eine andere Aufbereitung gewünscht sind, liefere ich das gerne nach.

Viele Grüße
Thomas