CounterOffset für 15_CUL_EM.pm

[abc2006]

Hi,
ich hatte mich ein bisschen mit der 15_CUL_EM.pm herumgeschlagen, um meinen EM1000GZ mit meinem Gaszähler zu synchronisieren.
Da ich irgendwie zu doof bin, mit corr1 und corr2 den Zählerstand vorzugeben, habe ich mit wenigen Zeilen ein zusätzliches Attribut eingefügt, mit welchem man die Differenz zwischen dem Zähler und dem Stand des EMGZ einstellen kann.
Anwendung wäre ist ganz einfach, attr CounterOffset löschen, Zählerstand notieren, dann Taschenrechner bemühen, notierten Zählerstand - total, und den dann errechneten Wert als CounterOffset eintragen.

Patch habe ich angehängt, wenn ich Zustimmung finde, ergänze ich gerne auch noch die Doku.

Grüße
Stephan

[rudolfkoenig]

Habs angeschaut, aber in der aktuellen Form kann ich es nicht einchecken:
- viele zusaetzliche Kommentare (Deutsch/UTF-8): ich kenne meinen Code, und wenn Kommentar, dann english
- sinnlose neue Logs (man uebt Datumsformatierung)
- Format der raw Reading geaendert (geht gar nicht)
- direkter Zugriff auf Attribut statt AttrVal.
- patch ist verkehrt herum.

[abc2006]

Hi,

danke für deine Hinweise. Habe versucht alle zu berücksichtigen. Hoffe ich habe nichts übersehen oder gar neue Fehler reingemacht

Diff habe ich jetzt mit

Code Auswählen Erweitern

svn diff 15_CUL_EM.pm > /root/15_CUL_EM.pm.patch
erstellt. Ist das so okay?


- viele zusaetzliche Kommentare (Deutsch/UTF-8): ich kenne meinen Code, und wenn Kommentar, dann english
+ Kommentare entfernt
- sinnlose neue Logs (man uebt Datumsformatierung)
+ War zum Debug, entfernt
- Format der raw Reading geaendert (geht gar nicht)
+ Ist für die Funktion nicht wichtig, zurückgeändert. Würdest du mir sagen, warum du damit ein anscheinend großes Problem hast, damit es mir für zukünftige Vorschläge klar ist? Einfach dein gutes Recht als Entwickler/Maintainer, oder hat es technische Gründe?

- direkter Zugriff auf Attribut statt AttrVal.
Meinst du hier die Zeile

Code Auswählen Erweitern

my $counter_offset = $attr{$n}{"CounterOffset"};
Hatte den Code von Zeile 159 übernommen. Ist jetzt geändert in AttrVal, nach meinen Tests funktionierts auch noch/wieder


- patch ist verkehrt herum.
+ klar, seh ich jetzt auch 

Danke für die Mühe, mir zu helfen!

Stephan

[rudolfkoenig]

Diff ist OK, wenn ich jetzt noch Doku kriege, wirds eingecheckt.

Würdest du mir sagen, warum du damit ein anscheinend großes Problem hast

Dieses Reading/Event gibt es seit laengerem, viele (ich auch) loggen und plotten diese Daten.
Nach deiner Aenderung muessten die FileLog und SVG-Plot Regexps umgebaut werden, ganz ohne Grund.

[abc2006]

Okay, dann mal wieder dünnes Eis:

hab ein <li> angefügt, und ein <code> für ein Beispiel aus einer anderen Datei kopiert.
Hoffe, das ist so okay?
join.pl läuft ohne Fehler (von dieser Datei) durch...


edit: und ich hab doch die falsche Datei erwischt ...  jetzt passts

[rudolfkoenig]

Habs eingecheckt.

[abc2006]

juhu 
Danke!