[gelöst] Frostalarm Meldung mit notify

newby · 12 Januar 2016, 16:04:09

Hallo,

ich hoffe mir kann jemand helfen das ganze ans laufen zu bekommen und am besten noch so das ich verstehe was ich da mache.

Idee:
Das Reading "alarm"der DS18B20 zu nutzen um eine Meldung per Mail zu generieren wenn die Temperatur einen bestimmten Wert unterschreitet und "alarm" auf "1" gesetzt wird.
Die DHT22 Sensoren haben ein Userreading "alarm".
Nach der Meldung wird der dummy Frostalarm auf "Alarm" gesetzt und soll weitere Mails unterbinden solange die Alarm nicht zurückgesetzt wurde.

aktueller Code mit diversen Problemen:

Code Auswählen

.*DHT22:.*|.*_Temp:.* {
my $alarm=ReadingsVal($_,"alarm","0");
my $temp=ReadingsVal($_,"temperature","122");
my $templow=ReadingsVal($_,"templow","122");
my $Status=InternalVal("Frostalarm","STATE","tilt");
{ if (($alarm !~ m/0/) && ($Status eq "Aus" { 
{DebianMail('mail@mail.de','FHEM Frostwarnung','Die Temperatur von ' .$NAME. ' liegt bei ' .$temp)}
 fhem ("set Frostalarm on"); 
   Log 3, "$NAME : Frostwarnung  Temp: $temp"; 
  } 
else {
  fhem ("set Frostalarm off");
  Log 3,"$NAME : Frostwarnung Temp: $temp";
  }
 }
}

"$_" ist wohl mein erster Fehler, da die Werte nicht korrekt bzw. gar nicht übernommen werden.
Mir sind da leider aufgrund fehlender Kenntnis die Ideen ausgegangen.

Im Log werden 2 Einträge über else generiert, und zwar für jedes geänderte Reading eines Devices.
Hier temperature und alarm

Code Auswählen

2016.01.12 15:23:59 3: EMZ_Temp : Frostwarnung  Temp: 122 
2016.01.12 15:23:59 3: EMZ_Temp : Frostwarnung  Temp: 122

das ("set Frostalarm off") ist leider auch noch nicht von großer Dauer, da so bei jedem 3.ten Lauf wieder der Stauts wieder Aus.

Ich hoffe das ist einigermaßen verständlich

Gruß
Sven

Puschel74 · 12 Januar 2016, 20:03:18

Fangen wir mal an Anfang an:
Als sich die Biene auf die Blume setzte ...

- ne Blödsinn (sorry).
Ich hab nur grad etwas Zeit und würde mich breit schlagen lassen das mit dir durchzugehen.

Erstmal - brav das du das über das DEF machst.
Das erleichtert die Hilfe ungemein.

Hier

Code Auswählen

.*DHT22:.*|.*_Temp:.* {
triggert dein notify auf
-) alles was auf DHT22 UND
-) alles was auf _Temp endet
--) mit allen Readings die die Geräte haben.
Willst du das so?
Was DHT22 ist verstehe ich noch aber was sind die Geräte mit _Temp am Schluss?

Wie gesagt - wir gehen es mal langsam an.

Edith: Ich liefer dir deswegen keine fertige Lösung (obwohl es für mich schneller erledigt wäre) weil ich möchte das du verstehst was der Code macht.

Ellert · 12 Januar 2016, 20:10:19

Alternativ: Mit einem DOIF ist auch lösbar, etwa so

Code Auswählen

define di DOIF ([Sensor:temperature] < [Sensor:templow] ) (({DebianMail(...)}))
DOELSEIF ([Sensor:temperature] > ([Sensor:templow] + 3))

Sendet 1 Mail und dann erst wieder, wenn die Temperatur 3°C über templow liegt und danach templow wieder unterschritten wird.

Puschel74 · 12 Januar 2016, 20:12:32

@Ellert
Du hattest 5 Minuten Zeit meinen Beitrag zu lesen.
Wenn du dir die Zeit nimmst und dem TE erklärst was der/DEIN Code macht darfst du gerne weiter machen und ich bin raus.

newby · 13 Januar 2016, 08:47:52

Moin Puschel74,

vielen Dank für Dein Angebot mir das ganze zu erklären.
"_Temp" ist im Namen aller DS18B20 Devices vorhanden. Bsp. (EMZ_Temp)

Die Abfrage soll alle Devices abfangen mit alarm Reading. (momentan 5 Stk.)
Aber ich benötige für die Mail ja auch das Reading "temperature".

@Ellert
Danke für den Vorschlag,
aber wie gesagt ich möchte das ganze was ich da zusammen stricke auch verstehen und nicht ewig hier den DAU spielen.
und da es sich um mehrere Sensoren handel die auch gerne noch zuwachs bekommen, ist eine variable Lösung, die ich nicht jedes mal erweitern muss besser.

Gruß
Sven

Ellert · 13 Januar 2016, 09:53:17

Mit dem Hinweis auf DOIF und dem schlichten Beispiel wollte ich zeigen, dass es auf der FHEM-Ebene möglich ist fast ohne Perl-Code auszukommen.
Das DOIF war für mich als Einsteiger vor etwa einem Jahr eine gute Hilfe, da ich mit Perl und Regulären Ausdrücken noch keine Erfahrung hatte.
Bei Interesse, hätte ich natürlich geholfen eine individuelle Lösung zu entwickeln.

Das DOIF bietet in einer funktionierenden Testversion DOIF neue Features: Generalisierung, $DEVICE, $EVENT, Attribut notexist die Möglichkeit Gerätenamen über Reguläre Ausdrücke anzusprechen.

Eine Lösung mit einem notify und Perlcode zu suchen ist ein guter Weg Erfahrungen zu sammeln, genauso wie das Verwenden von Alternativen.

newby · 13 Januar 2016, 11:00:20

Bin natürlich auch für alle Ideen offen.
Hatte nur irgendwie im Kopf das DOIF nichts für mehrere Devices ist.
Aber lasse mich auch gerne hier eines besseren beleeren.
Hat sich wohl auch einiges geändert zum Jahreswechsel.

Puschel74 · 13 Januar 2016, 11:51:58

DOIF geht genauso, da ich es aber nicht benutze kann ich nur Hilfe zu notify anbieten.

Zitat"_Temp" ist im Namen aller DS18B20 Devices vorhanden. Bsp. (EMZ_Temp)

Ok, soll dann bei Änderung des Reading temperature das notify triggern oder wenn sich alarm ändert?

Wenn es nur bei alarm auslösen soll und du KEINE anderen Sensoren mit diesem reading hast genügt ein einfaches:

Code Auswählen

.*:alarm
als regexp.
Hier triggert von JEDEM Device das Reading alarm wenn es ein Event generiert.
Events kannst du dir im EventMonitor anschauen.

Wenn es mehrere Geräte mit dem Reading gibt und es sollen NUR deine Geräte mit _Temp am Ende des Namens triggern dann sollte das regexp so

Code Auswählen

.*_Temp:alarm
aussehen.

An das Reading temperatur kommen wir im notify immer noch dran

Soll aber von JEDEM Geräte das Reading alarm UND temperatur triggern dann passt dieses regexp:

Code Auswählen

.*:(alarm|temperature).*
Aber ACHTUNG! Hier triggert JEDES Event des Reading temperature.
Das kann durchaus jede Menge werden und ist evtl. nicht nötig.

Wenn du also nur auf alarm triggern (hören) willst sind die beiden obigen passender.

Edith:

Code Auswählen

.*:alarm {
Log(1,"$NAME - $EVENT");
}

Das mal ins DEF eingeben und im Logfile schauen wenn alarm ein Event auslöst - und das Ergebniss bitte mal posten.
Ja sorry ich werfe gerne mit Klammern um mich die evtl. unnötig sind - aber so funktionieren meine Codes bei mir zumindest wunderbar

Edith1: @Ellert
Wenn du die Muße hast darfst du natürlich auch zu DOIF helfen - sorry für meinen "harschen" Einwurf.
Der TE soll ja was lernen

newby · 13 Januar 2016, 12:05:21

ZitatOk, soll dann bei Änderung des Reading temperature das notify triggern oder wenn sich alarm ändert?

Nein nur bei Änderung des Readings alarm soll das ganze ausgelöst werden.
Hatte auch das regexp

Code Auswählen

.*DHT22:alarm:.*|.*_Temp:alarm:.*
definiert aber wieder geändert weil ich nicht wusste wie ich sonst an die Werte temperature etc.komme.

Nur die Geräte .*DHT22 und .*_Temp haben bis jetzt das Reading, theoretisch würde dann ja .*:alarm reichen
Mir ist kein weiteres Gerät bekannt

in der Config das dieses Reading hat und nicht dazugehört, wäre für spätere Erweiterungen ja dann auch die bessere Variante.

Puschel74 · 13 Januar 2016, 12:18:24

Schau oben mein Edith und probier das mal aus.

Ellert · 13 Januar 2016, 12:27:23

@Puschel74: Alles ist gut, ich werde deine Lerneinheit nicht unterbrechen. Wofür steht eigentlich TE? Teilnehmende Einheit?

Puschel74 · 13 Januar 2016, 12:29:11

Zitat von: Ellert am 13 Januar 2016, 12:27:23
Wofür steht eigentlich TE? Teilnehmende Einheit?

Nein, ThreadErsteller

ZitatMir ist kein weiteres Gerät bekannt

Dem kommen wir mit meinem Edith auf die Schliche

newby · 13 Januar 2016, 13:47:09

Ich habe das DEF ein wenig abgeändert auf

Code Auswählen

.*:alarm:.* {
Log(1,"$NAME - $EVENT");
}

Logeinträge seit dem.

Code Auswählen

2016.01.13 13:06:36 1: EMZ_Temp - alarm: 1
2016.01.13 13:06:54 1: Pumpe_Temp - alarm: 0
2016.01.13 13:07:24 1: Speicher_Temp - alarm: 0
2016.01.13 13:30:06 1: Keller.DHT22 - alarm: 0
2016.01.13 13:30:08 1: Keller.DHT22[b] - [/b]alarm: 0
2016.01.13 13:30:57 3: Kueche_TH : measured-temp: 19.1

Wieso werden die alarm Einträge mit - angezeigt und die anderen mit: ??
Warum der Keller 2 mal gelistet ist ist mir ein Rätsel aber ist jetzt auch nicht relevant denke ich.

Puschel74 · 13 Januar 2016, 13:57:27

ZitatWieso werden die alarm Einträge mit - angezeigt und die anderen mit: ??

Weil wir in unserem code als Trenner - drinne haben.
Gut zu erkennen sind auch die unterschiedlichen Loglevel - 1 bei uns und 3 aus FHEM.
Keller hat 2 Events erzeugt mit 2 Sekunden unterschied daher wird er auch zweimal geloggt.

Sorry für das löschen aber erst kommt noch $EVENT dran.

$EVENT besteht aus 2 Teilen:

Zitatalarm: 1

d.h. in $EVTPART0 steht alarm:
und in $EVTPART1 steht 1 (oder eben 0).

Code Auswählen

.*:alarm:.* {
Log(1,"$NAME - $EVTPART0 -- $EVTPART1");
}

sollte das beweisen.

newby · 13 Januar 2016, 14:34:29

Und ich dachte ich habe schon Hallos.
Dann hebe ich mir das Log Temperatur mal für gleich auf

EVTPART habe ich schon gelernt

Habs aber nochmal der Vollständigkeit halber gemacht

Code Auswählen

2016.01.13 14:30:06 1: Keller.DHT22 - alarm: -- 0
2016.01.13 14:30:07 1: Keller.DHT22 - alarm: 0
2016.01.13 14:30:07 1: Keller.DHT22 - alarm: -- 0