[Gelöst] MotorErr überwachen mittels DOIF

[Amenophis86]

Ok, ich gebe es auf. Versuche es seit Stunden und komme nicht weiter. Ähnlich der Batterie Überwachung möchte ich gerne das Reading "motorErr" meiner HM-CC-RT-DN überwachen. Natürlich kann man das auch mittels notify und auch auf das hminfo Device lauschen, aber so langsam geht es mir eher ums Prinzip, wie es mit DOIF geht.

Folgendes möchte ich:
- Alle Device überwachen. Diese heißen immer XY.HZ
- das Reading nennt sich motorErr
- bei keinem Fehler hat es den Status ok
- bei Fehler gibt es verschiedene Stati

Hier mal meine Lösungsvorschläge, welche alle nicht klappten, da immer cmd2 ausgeführt wurde:

Code Auswählen Erweitern


([".HZ$:motorErr"] and [$EVENT] eq "motorErr: ok")
(set Test ok)
DOELSEIF ([".HZ$:motorErr"] and [$EVENT] ne "motorErr: ok")
(set Test err)


Code Auswählen Erweitern

([".HZ:motorErr"] eq "motorErr: ok")
(set Test ok)
DOELSEIF ([".HZ:motorErr"] ne "motorErr: ok")
(set Test err)

Code Auswählen Erweitern

([.*HZ:motorErr] eq "ok")
(set Test ok)
DOELSEIF ([.*HZ:motorErr] ne "motorErr: ok")
(set Test err)

Ich brauche quasi eine Möglichkeit wie ich auf verschieden Devices und auf das gleiche Reading lauschen kann, wie in Beispiel 3. Oder ich muss verstehen, warum bei Beispiel 1 und 2 die Syntax falsch ist und immer wieder cmd2 ausgeführt wird.

[igami]

Ich kann dir zwar nicht bei deinem DOIF helfen, habe aber für solche Sammelmeldungen das Modul monitoring geschrieben.

[Amenophis86]

Liest sich zwar etwas komplex, aber werde es mal testen. Dafür müsste ich aber alle Error Meldungen von motorErr kennen, oder?

Und wie gesagt, geht es mir inzwischen auch ein bisschen ums Prinzip zu verstehen, warum mein DOIF nicht funktioniert

[Ellert]

Liest sich zwar etwas komplex, aber werde es mal testen. Dafür müsste ich aber alle Error Meldungen von motorErr kennen, oder?

Und wie gesagt, geht es mir inzwischen auch ein bisschen ums Prinzip zu verstehen, warum mein DOIF nicht funktioniert

In der Commandref zu DOIF gibt es doch das Beispiel der Batterieüberwachung, da müsstest Du doch nur die  Regexp anpassen.

Code Auswählen Erweitern

define di_battery DOIF ([":battery: low"] and [?$SELF:B_$DEVICE] ne "low")

    ({DebianMail('yourname@gmail.com', 'FHEM - battery warning from device: $DEVICE')}, setreading $SELF B_$DEVICE low)

DOELSEIF ([":battery: ok"] and [?$SELF:B_$DEVICE] ne "ok")

    (setreading $SELF B_$DEVICE ok)


attr di_battery do always

[Amenophis86]

Korrekt aber der MotorErr wirft verschiedene Error aus meine ich. Diese Error kenne und finde ich nicht. Daher muss ich auf ungleich "ok" prüfen und das schaffe ich nicht

EDIT:
ok, stehen in der CommandRef und können bei mir wie folgt sein:

Code Auswählen Erweitern

ok|ValveTight|adjustRangeTooLarge|adjustRangeTooSmall|communicationERR|unknown|lowBat|ValveErrorPosition

aber jetzt für jedes einen DOIF Strang bauen wäre ja auch doof. Es muss doch möglich sein auf != ok zu prüfen und das mit einem DOIF für alle Devices.

[Ellert]

Die Eventabfrage könnte doch so aussehen

Code Auswählen Erweitern

[":MotorError: (ValveTight|adjustRangeTooLarge|adjustRangeTooSmall|communicationERR|unknown|lowBat|ValveErrorPosition)"]
und statt

Code Auswählen Erweitern

[?$SELF:B_$DEVICE] ne "low"
nimmst Du

Code Auswählen Erweitern

[?$SELF:B_$DEVICE] !~ "ValveTight|adjustRangeTooLarge|adjustRangeTooSmall|communicationERR|unknown|lowBat|ValveErrorPosition"

Und das Batteriebeispiel ist für alle Geräte mir "battery" Reading.

[Amenophis86]

Das ist richtig und danke für den Vorschlag. Was ich mich allerdings aktuell noch frage ist folgendes:

Wie muss ich eine DOIF Bedingung schreiben, welche wie folgt sein soll:
- Lausche auf alle Devices mit "HZ" im Namen
- Lausche bei diesen Devices auf das Reading "motorErr"
- Wenn das Reading NICHT "ok" lautet, dann mache folgendes

Ist ähnlich deiner Abfrage, aber leicht abgewandelt. Und dafür sehe ich aktuell noch keine Lösung, was mich ehrlich gesagt wundert. Denn die quasi Bedingung

Code Auswählen Erweitern

([.*HZ.*:.*motorErr.*] ne "ok") geht ja nicht. Und das muss doch irgendwie möglich sein, ohne alle Varianten explizit zu nennen, wie du es gemacht hast.
Wie gesagt, deine Lösung ist top, aber aus Prinzip frage ich mich noch, wie es richtig heißen müsste um das von mir geschriebene zu definieren

[Damian]

Code Auswählen Erweitern

([.*HZ.*:.*motorErr.*] ne "ok")

funktioniert so nicht, weil dein Ereignistrigger nur wahr oder nicht wahr ist. So etwas lässt sich aber inzwischen mit der erweiterten Syntax 
["regex for trigger",<default value>] für Ereignistrigger, initiiert durch einen Default-Wert, realisieren (siehe https://fhem.de/commandref_DE.html#DOIF_Ereignissteuerung_ueber_Auswertung_von_Events).

Übrigens, die ganze .*-Orgie kannst du bei DOIF weglassen

Es sollte so funktionieren:

Code Auswählen Erweitern

([HZ:motorErr,""] ne "ok")

[Amenophis86]

Das .* nicht geht war mir ja bewusst, sollte nur zeigen auf was es hinaus laufen sollte.

Und das man ([DEVICE:Reading,""] ne "ok") schreiben kann war mir nicht bewusst. Hatte ich so aus der CommandRef nicht entnommen, weil immer nur überall DEVICE stand und nie auch noch das Reading quasi dabei. Aber danke für die Antwort, werde es mal testen. Hätte mich auch gewundert, wenn es da nix gegeben hätte

EDIT:
Dein Vorschlag kann aber nicht funktionieren, da ja ich ja auf alle Device lauschen will, welche HZ im Namen oder am Anfang haben.

[Damian]

([DEVICE:Reading,""] ne "ok")

ist nicht das Gleiche, wie

(["DEVICE:Reading",""] ne "ok")

Im ersten Fall ist ein bestimmtes Reading gemeint, hier sind keine Regex erlaubt. Im zweiten Fall handelt es sich aufgrund der Anführungszeichen um einen Ereignistrigger mit Regex und keine Readingabfrage wie im ersten Fall.

[Damian]

EDIT:
Dein Vorschlag kann aber nicht funktionieren, da ja ich ja auf alle Device lauschen will, welche HZ im Namen oder am Anfang haben.

Doch!, schon probiert?

[Amenophis86]

jop und hat nicht funktioniert. Teste es aber gerne nochmal

[Damian]

jop und hat nicht funktioniert. Teste es aber gerne nochmal

Wenn es nicht klappt, dann poste mal die vermeintliche Event-Zeile und ein List deines DOIF-Moduls

[Amenophis86]

Das DOIF:

Code Auswählen Erweitern


Internals:
   CFGFN
   DEF        ([HZ:motorErr,""] ne "ok") (set HZ.Test $EVENT)
   NAME       Test.DOIF
   NR         306
   NTFY_ORDER 50-Test.DOIF
   STATE      initialized
   TYPE       DOIF
   Readings:
     2017-04-17 12:00:52   cmd             0
     2017-04-17 12:00:52   state           initialized
   Condition:
     0          ReadingValDoIf($hash,'HZ','motorErr','""') ne "ok"
   Devices:
     0           HZ
     all         HZ
   Do:
     0:
       0          set HZ.Test $EVENT
     1:
   Helper:
     globalinit 1
     last_timer 0
     sleeptimer -1
   Itimer:
   Readings:
     0           HZ:motorErr
     all         HZ:motorErr
   Regexp:
     All:
   State:
Attributes:


Eventzeile:

Code Auswählen Erweitern

2017-04-17 12:04:26 dummy HZ.Test motorErr: defekto

EDIT:
Verstehe auch nicht ganz, wieso [HZ:motorErr,""] auf alle Device mit HZ im Namen lauschen soll. Das ist doch ne klare Angabe auf das Device HZ und auf das Reading motorErr, oder verstehe ich das immer noch falsch? Oder hast du dich im Beispiel verschrieben und ich sollte ["HZ:motorErr",""] in der DOIF Abfrage nehmen?

EDIT2:
Ok, du scheinst dich verschieben zu haben:

Es sollte so funktionieren:

Code Auswählen Erweitern

([HZ:motorErr,""] ne "ok")

mit

Code Auswählen Erweitern

(["HZ:motorErr",""] ne "ok") funktioniert es. Hier wird also auf HZ im DEVICE Namen gelauscht und auf "motorErr" im Readingsnamen. Und warum genau brauche ich den default-Wert "" am Ende der Suche noch?

[Damian]

EDIT2:
Ok, du scheinst dich verschieben zu haben:mit

Code Auswählen Erweitern

(["HZ:motorErr",""] ne "ok") funktioniert es. Hier wird also auf HZ im DEVICE Namen gelauscht und auf "motorErr" im Readingsnamen. Und warum genau brauche ich den default-Wert "" am Ende der Suche noch?

Ja, du hast Recht.

Weil der Default-Wert eine neue Funktionalität bei Ereignistriggern impliziert. Es muss ja einen Wert zurückliefern, nämlich den Default-Wert, wenn kein Ereignis passt und noch ein weiterer Trigger in der Bedingung verknüpft wird.

z. B.

((["HZ:motorErr",""] ne "ok") or [08:00])

mit der Definition ohne Default-Wert: ["HZ:motorErr"] ist das Ding immer nur wahr oder nicht wahr.