98_average.pm erweitert

[erwin]

                                                   

Hi Rudolf,
ich nehme mal an, du bist der Author von diesem Modul.

Ich habe versucht das Modul um *min* und *max* Funktionalität zu erweitern.

Nach meinen Tests funktioniert es soweit, ich hab allerdings noch einige
Unsicherheiten bei 2 Punkten:
1) in Zeile 63    *next if($devName !~ m/^$re$/ && "$devName:$s" !~
m/^$re$/ || $s =~ m/_avg_/);*
    muss ich den Filter um * || $s =~ m/_min_/ ** || $s =~ m/_max_/      
erweitern?
2) *Das mit den Notifies ist mir nicht ganz klar, ist das so richtig wie
ich es gemacht hab?
    Stichwort *$dev->{CHANGED}[]

*Bitte um check des Moduls und bei Gefallen um Übernahme!
Als return für deine Mühe werde ich die commandref.html anpassen und ein
Wiki zum Thema schreiben.

Attachments: geändertes Modul, diff.-file

Auszug aus Average-LOG:

2012-07-13_00:00:05 HMS100TF_473d temperature_avg_day: 24.8
2012-07-13_00:00:05 HMS100TF_473d temperature_max_day: 25.8
2012-07-13_00:00:05 HMS100TF_473d temperature_min_day: 23.9
2012-07-13_00:00:05 HMS100TF_473d humidity_avg_day: 46.4
2012-07-13_00:00:05 HMS100TF_473d humidity_max_day: 52.8
2012-07-13_00:00:05 HMS100TF_473d humidity_min_day: 39.1
Auszug aus global-LOG:

2012.07.12 23:59:04 1: mytestavg N-update: HMS100TF_473dhumidity42idx=1
2012.07.13 00:00:04 1: mytestavg day/month-update: HMS100TF_473dtemperature24.4idx=0
2012.07.13 00:00:04 1: mytestavg N-update: HMS100TF_473dtemperature24.4idx=1
2012.07.13 00:00:04 1: mytestavg day/month-update: HMS100TF_473dhumidity42.1idx=0
2012.07.13 00:00:04 1: mytestavg N-update: HMS100TF_473dhumidity42.1idx=1


Was mir noch aufgefallen ist: Dadurch dass der avg/min/max Event jeweils
mit dem ersten notify nach Mitternacht getriggert wird, sind natürlich die
timestamps im Log um einen Tag falsch.
kann man eigentlich notifies mit "falschen" timestamps triggern? - Ich bin
zwar dennoch nicht sicher, ob das die Lösung wäre, muß checken wie z.B.
dbLog die Timestamps zur Datenbank schickt......

Herzlichen Dank
Erwin*

***

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com

[rudolfkoenig]

                                                   

Hallo Erwin,

> *Bitte um check des Moduls und bei Gefallen um Übernahme!

Hab deine Aenderungen eingecheckt (updatefhem ab morgen)


> Als return für deine Mühe werde ich die commandref.html anpassen und ein
> Wiki zum Thema schreiben.

Vielen Dank im Voraus

Gruss,
  Rudi

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com

[erwin]

                                                   

Modul funktioniert!
Danke Rudi!

Teil1 des ROI:

updated commandref, diff file.
DasWiki folgt im laufe der nächsten Woche.


l.g. erwin

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com

[rudolfkoenig]

                                                   

> updated commandref, diff file.

Danke, eingecheckt, und auf fhem.de hochgeladen. Bitte demnaechst einen
patch-faehigen diff (diff -u/diff -c) machen, so habe ich mehr Zeit mit dem
diff-einbauen verbracht, als mit evtl. selbst-schreiben.

Btw. hab auch zweimal [/list] noch eingebaut (andere Abschnitte), wird gerne
vergessen, und der Inhalt wandert deswegen langsam nach rechts.
Noch'n Grund commandref.html zu splitten.

--
To unsubscribe from this group, send email to
fhem-users+unsubscribe@googlegroups.com