DOIF Batterie Überwachung

[wuast94]

Ich versuche gerade ein DOIF zu erstellen was darauf reagiert wenn ein gert mit dem reading battery unter 40 fällt eine Push nachricht aufs handy bringt. allerdings scheine ich ieinen fehler zu haben weil das ganze nicht funktioniert.
Das DOIF:
([.*:[Bb]attery] < 40) ("set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln") DOELSE ("set dummy off")

wenn ich das dann versuche mit nem trigger auszuprobieren tut sich genau nix

trigger Schalter_Wohnzimmer battery 20

[mi.ke]

([.*:[Bb]attery] < 40) ("set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln") DOELSE ("set dummy off")

Ich schätze, weil der set-Teil in Anführungszeichen steht...

[Gisbert]

Hallo wuast94,

ich bin kein Experte, habe jedoch viele DOIFs und notifiys erfolgreich umgesetzt.
Was mir auffällt (edit: wie bei mi.ke - Antwort hat sich überschnitten), sind die Anführungszeichen in den beiden Ausführungteilen, die ich so noch nie benutzt habe.

Ich schlage vor, dass du deine Definitionen (in code tags) postest, und zwar sowohl das DOIF als auch Device, bei welchem die Batterie überwacht werden soll, sowie die Definition des Push-Devices, gerne auch noch das dummy-Device.

Was passiert, wenn du in der Fhem-Kommandozeile "set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln" (inkl. deiner Anführungszeichen eingibst, wobei du $NAME und $EVENT durch etwas spezifisches (d.h. für das zu überwachende Device) ersetzt?

Viele Grüße Gisbert

[amenomade]

Dazu (die beiden hieroben haben Recht) kommt noch:

Code Auswählen Erweitern

([.*:[Bb]attery] < 40) Das ist syntaxisch falsch.  Du bekommst bestimmt eine Fehlermeldung

Code Auswählen Erweitern

timer_01_c01 error: Wrong timespec .*:attery: either HH:MM:SS or {perlcode}

Die Triggerung mit Regex ist für Eventtriggerung. Dies muss dann in Einführungszeichen geschrieben werden.
ABER : ein Event kann nicht kleiner oder grösser als 40 sein.

Code Auswählen Erweitern

([".*:[Bb]attery"] < 40) würde deswegen auch nicht funktionieren.

Schau mal in CommandRef bei https://fhem.de/commandref_DE.html#DOIF_aggregation
Es gibt viele Beispiele, wie man das implementieren kann.

[andies]

zB

Code Auswählen Erweitern


([18:00] and [?@"":battery:$_ ne "ok",""]) (set TelegramBot _msg Batteriewarnung am Gerät [@"":battery:$_ ne "ok",""])

Ist dann um 18:00, klappt auch.


Gesendet von iPad mit Tapatalk Pro

[wuast94]

Hallo wuast94,

ich bin kein Experte, habe jedoch viele DOIFs und notifiys erfolgreich umgesetzt.
Was mir auffällt (edit: wie bei mi.ke - Antwort hat sich überschnitten), sind die Anführungszeichen in den beiden Ausführungteilen, die ich so noch nie benutzt habe.

Ich schlage vor, dass du deine Definitionen (in code tags) postest, und zwar sowohl das DOIF als auch Device, bei welchem die Batterie überwacht werden soll, sowie die Definition des Push-Devices, gerne auch noch das dummy-Device.

Was passiert, wenn du in der Fhem-Kommandozeile "set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln" (inkl. deiner Anführungszeichen eingibst, wobei du $NAME und $EVENT durch etwas spezifisches (d.h. für das zu überwachende Device) ersetzt?

Viele Grüße Gisbert

Hier einmal das komplette DOIF:

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([.*:[Bb]attery] < 40) ("set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln") DOELSE ("set dummy off")

Was die definition eines zu überwachendes Gerätes angeht ist auch nur ein DOIF das sich aus 3 verschiedenen devices die readings holt und auf sich selbst setzt (zigbee Multisensor werden leider temp humidity und baro nicht in einem sensor sondern in 3 verschiedene übergeben). Und eines der Readings ist battery mit einem Zahlenwert von 0-100. deswegen kann ich auch nicht die Beispiele aus dem wiki nehmen da die alle von einem "ok" ausgehen.

das push device ist ein ganz normales pushover device und sollte hier auch nichts zur sache tun da das einwandfrei funktioniert. und wenn ich "set Push msg Gerät: bla Akkustand: 29 Bitte Akku wechseln" eingebe bekomme ich auch direkt die push nachricht. allerdings gebe ich das in fhem im commando Feld oben ohne Anführungszeichen ein.

[wuast94]

Dazu (die beiden hieroben haben Recht) kommt noch:

Code Auswählen Erweitern

([.*:[Bb]attery] < 40) Das ist syntaxisch falsch.  Du bekommst bestimmt eine Fehlermeldung

Code Auswählen Erweitern

timer_01_c01 error: Wrong timespec .*:attery: either HH:MM:SS or {perlcode}

Die Triggerung mit Regex ist für Eventtriggerung. Dies muss dann in Einführungszeichen geschrieben werden.
ABER : ein Event kann nicht kleiner oder grösser als 40 sein.

Code Auswählen Erweitern

([".*:[Bb]attery"] < 40) würde deswegen auch nicht funktionieren.

Schau mal in CommandRef bei https://fhem.de/commandref_DE.html#DOIF_aggregation
Es gibt viele Beispiele, wie man das implementieren kann.

und ja diese Fehlermeldung bekomme ich tatsächlich.

Code Auswählen Erweitern

timer_01_c01 error: Wrong timespec .*:attery: either HH:MM:SS or {perlcode}

aber es muss doch möglich sein in einem DOIF zahlen miteinander vergleichen zu können und auch ein wenn reading kleiner als dann das machen gehen

[Gisbert]

Hallo wuast94,

versuch mal das (ungetestet):

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([.*:[Bb]attery] < 40) (set Push msg Gerät: $NAME Akkustand: [$NAME:[Bb]attery] Bitte Akku wechseln) DOELSE (set dummy off)
Ggf. den 1. Ausführungsteil in doppelte Klammern:

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([.*:[Bb]attery] < 40) ((set Push msg Gerät: $NAME Akkustand: [$NAME:[Bb]attery] Bitte Akku wechseln)) DOELSE (set dummy off)
Falls beides nicht rennt, dann lass mal "[$NAME:[Bb]attery]" weg, und versuch es zuerst mal ohne den Readingswert.

Viele Grüße Gisbert

[amenomade]

Hallo wuast94,

versuch mal das (ungetestet):

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([.*:[Bb]attery] < 40) (set Push msg Gerät: $NAME Akkustand: [$NAME:[Bb]attery] Bitte Akku wechseln) DOELSE (set dummy off)
Ggf. den 1. Ausführungsteil in doppelte Klammern:

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([.*:[Bb]attery] < 40) ((set Push msg Gerät: $NAME Akkustand: [$NAME:[Bb]attery] Bitte Akku wechseln)) DOELSE (set dummy off)
Falls beides nicht rennt, dann lass mal "[$NAME:[Bb]attery]" weg, und versuch es zuerst mal ohne den Readingswert.

Viele Grüße Gisbert

So werden die Befehle funktionieren, aber die Bedingung ist immer noch falsch.

Ja, man kann Zahlen vergleichen, aber nicht ein Event mit einem Zahl.
Lies bitte nochmal CommandRef / Aggregation DOIF,wie schon erwähnt. Es sind viele Beispiele, sogar auch mit Zahlen
Etwas in der Art:

Code Auswählen Erweitern

([#"":[Bb]attery:$_ < 40]  >  0) Wenn Anzahl Devices mit Reading B/battery kleiner als 40 grösser als 0 ist

Code Auswählen Erweitern

(set TelegramBot _msg Batteriewarnung am Gerät [@"":battery:$_ ne "ok",""]) ..., sende Liste von diesen Devices

[MadMax-FHEM]

Wenn es kein DOIF sein muss, ist vielleicht auch das eine Alternative: https://forum.fhem.de/index.php/topic,82637.msg747514.html#msg747514

Kann auch mehr als "nur" OK etc.

Gruß, Joachim

[wuast94]

also habe dein zweites Bespiel mal ausprobiert .. auf ein setreading oder ein trigger reagiert es nicht.. allerdings wenn ich auf dem Dorf ein checkall mache bekomme ich einen push mit folgendem Inhalt:

Code Auswählen Erweitern

Gerät: $NAME Akkustand: Bitte Akku wechseln
Also schonmal mehr als vorher. habe mittlerweile die 40 auch auf 100 umgestellt um eher was zu triggern wenn ein echtes event rein kommt aber auch da bis jetzt nichts.

was die Geschichte mit dem geht so nicht angeht bin ich verwirrt .. die commandref in der deutschen version hat zb ganz am anfang folgendes stehen:

Code Auswählen Erweitern

define di_lamp DOIF ([06:00-09:00] and [sensor:brightness] < 40) (set lamp on) DOELSE (set lamp off)
wenn ich das richtig sehe und man sich die erste Kondition weg denkt wird da genau das gemacht was ich auch mache .. nur im Beispiel mit nem größer als und ich habe ein kleiner als.
Vlt hat es bei mir einfach noch nicht klick gemacht aber ich verstehe einfach nicht warum es nicht gehen sollte .. wenn ich das so vergleiche ist die syntax korrekt und alles passt und dennoch geht es nicht und gibt mir zusätzlich n timespec Fehler aus obwohl ich nicht mal ne zeit vergleiche oder verwende was mich noch mehr verwirrt.

[Damian]

Vergleiche sind normalerweise nur mit konkreten Readings möglich, z. B. wie:

Code Auswählen Erweitern

[sensor:brightness] < 40

Diese Angabe :

Code Auswählen Erweitern

[".*:[Bb]attery"] < 40

ist aber keine konkrete Angabe eines Readings, sondern eine Eventabfrage, die normalerweise nur wahr oder falsch ist, aber keinen Inhalt eines Readings liefert.

Es gibt die Möglichkeit, statt des Readings auch das Event auszuwerten, dazu muss man deinen Default-Wert angeben, z. B.

Code Auswählen Erweitern

[".*:[Bb]attery",0] < 40

[amenomade]

Das ist aber interessant.

Code Auswählen Erweitern

[".*:[Bb]attery",0] < 40
Damit wird auf Regex-Event getriggert, aber dann das Reading bewertet? Cool!

[Damian]

nein, es wird das Event ausgewertet.
Zitat aus der Commandref:

Allgemeine Ereignistrigger können ebenfalls so definiert werden, dass sie nicht nur wahr zum Triggerzeitpunkt und sonst nicht wahr sind, sondern Inhalte des Ereignisses zurückliefern. Initiiert wird dieses Verhalten durch die Angabe eines Default-Wertes.

Syntax:

["regex for trigger",<default value>]

Anwendungsbeispiel:

define di_warning DOIF ([":^temperature",0]< 0) (set pushmsg danger of frost $DEVICE)
attr di_warning do always

Perl-Modus:
define di_warning DOIF {if ([":^temperature",0]< 0) {fhem_set"pushmsg danger of frost $DEVICE}}

Damit wird auf alle Devices getriggert, die mit "temperature" im Event beginnen. Zurückgeliefert wird der Wert, der im Event hinter "temperature: " steht. Wenn kein Event stattfindet, wird der Defaultwert, hier 0, zurückgeliefert.
Ebenfalls kann ein Ereignisfilter mit Ausgabeformatierung angegeben werden.

Syntax:

["regex for trigger":"<regex filter>":<output>,<default value>]

Regex-Filter- und Output-Parameter sind optional. Der Default-Wert ist verpflichtend.

Die Angaben zum Filter und Output funktionieren, wie die beim Reading-Filter. Siehe: Filtern nach Ausdrücken mit Ausgabeformatierung

Wenn kein Filter, wie obigen Beispiel, angegeben wird, so wird intern folgende Regex vorbelegt: "[^\*: (.*)" Damit wird der Wert hinter der Readingangabe genommen. Durch eigene Regex-Filter-Angaben kann man beliebige Teile des Events herausfiltern, ggf. über Output formatieren und in der Bedingung entsprechend auswerten, ohne auf Readings zurückgreifen zu müssen.

Ich würde allerdings für diese Aufgabe die Lösung aus dem Post #5 nehmen.

[wuast94]

nein, es wird das Event ausgewertet.
Zitat aus der Commandref:
Ich würde allerdings für diese Aufgabe die Lösung aus dem Post #5 nehmen.

also hab es jetzt mal so

Code Auswählen Erweitern

defmod sys_bat_check DOIF ([".*:[Bb]attery",0] < 80)\
("set Push msg Gerät: $NAME Akkustand: $EVENT Bitte Akku wechseln") DOELSE ("set dummy off")
scheint aber immer noch nicht zu gehen