[Gelöst] DOIF Bedingung wird immer als falsch evaluiert

[SCKWorld2012]

Hi zusammen,

ich wende mich mit einem vermutlich trivialen Problem an euch. Hier der Kontext:
Für ein Uni-Projekt muss ich eine Sturmwarnung mit FHEM umsetzen. Dazu nutze ich ein DOIF. Es soll ausgelöst werden, wenn die Werte meines Windsensors gewisse Schwellenwerte überschreiten. Die Schwellenwerte sind als selectNumbers-Eingabeflächen über webCmd realisiert und werden zu Testzwecken auf 0 gesetzt.
Zudem braucht meine Bedingung noch einen weiteren Teil: das DOIF soll nur einmal pro Tag ausgelöst werden, wobei der Zeitpunkt des "wieder scharf schaltens" auch angepasst werden können soll. Daher habe ich ein weiteres Eingabefeld, dieses mal einen uzsu-Toggle definiert. Das Feld fungiert als Flag, ob an diesem Tag schon einmal eine Warnung gesendet wurde. Den Wert setze ich mit einem at zum Wunschzeitpunkt zurück. Auch diesen Wert stelle ich zu Testzwecken natürlich händisch richtig ein.
Diese beiden Bedingungen verbinde ich mit einem "and".

Mein eigentliches Problem besteht nun darin, dass die Bedingung meines DOIFs immer als falsch evaluiert zu werden scheint, da das DOIF als state immer "CMD_2" hat. Besonders seltsam daran ist, dass beide Teile der Bedingung für sich genommen funktionieren, zusammen jedoch nicht mehr.

Kann mir evtl. jemand von euch sagen, was ich falsch mache?
Hier die Definition meines DOIFs:

define WindMonitoring DOIF (([MyWindSensor:windSpeed] ge [$SELF:thresholdWindSpeed] or [MyWindSensor:windGust] ge [$SELF:thresholdWindGust]) and [$SELF:alreadyNotified] eq "NEIN") (set StormWarningPushMessage message [$SELF:notifyText],set WindMonitoring alreadyNotified JA)

[Hermann123]

Hallo SCKWorld2012,

schick doch mal das ganze define...möglichst per Code einfügen Button....

[SCKWorld2012]

hi Hermann123,

das war das ganze define für das DOIF. Die restlichen Definitionen sollten hier ja nicht relevant sein, oder? Wie gesagt, für sich genommen funktionieren die Teile der Bedingung ja. Hier nochmal der Code "richtig" formatiert

Code Auswählen Erweitern

define WindMonitoring DOIF (([MyWindSensor:windSpeed] ge [$SELF:thresholdWindSpeed] or [MyWindSensor:windGust] ge [$SELF:thresholdWindGust]) and [$SELF:alreadyNotified] eq "NEIN") (set StormWarningPushMessage message [$SELF:notifyText],set WindMonitoring alreadyNotified JA)

[MadMax-FHEM]

Ich kenne ja DOIF zu wenig aber:

müsste das

set WindMonitoring alreadyNotified JA

nicht mittels setreading gemacht werden?

Also:

Code Auswählen Erweitern


setreading WindMonitoring alreadyNotified JA


Weil es soll doch das reading alreadyNotified des DOIF (WindMonitoring) auf JA gesetzt werden?

(wer setzt es denn dann zurück?)
EDIT: und wer setzt es denn überhaupt mal auf NEIN? Weil sonst stimmt ja die Abfrage eq "NEIN" schon nicht...

Und: besser ein list statt nur dem define, also list Devicename (in deinem Fall list WindMonitoring)...

Gruß, Joachim

[SCKWorld2012]

Hi Joachim,

ich kenne mich auch nicht aus, aber das Ausprobieren des "Do" Teils für sich hat funktioniert, dementsprechend habe ich darüber nicht weiter nachgedacht. Vielleicht kann da ja einer der alteingesessenen Fhem-Experten etwas Klarheit schaffen.
Das eigentliche Problem löst das aber leider noch nicht :/

Gruß
Simon

[Hermann123]

Ich denke das Problem wird dann sein, das Du die Reading oder EventReadings
   
   [$SELF:thresholdWindSpeed]
   [$SELF:notifyText]

nicht definierst hast.

Schau mal in der Refenz unter "Erzeugen berechneter Readings ohne Events" oder "Erzeugen berechneter Readings mit Events".
Vielleicht ist das ja die Lösung....

[Beta-User]

...kenne DOIF auch nicht, würde aber annehmen, dass es die "Doppelbedingung" ist: Es können praktisch nie zwei Ereignisse gleichzeitig eintreffen. Steht irgendwo in dieser überbordenden commandref zu DOIF...

[SCKWorld2012]

Hi Hermann,

danke für deine Antwort. Die Readings definiere ich unmittelbar im Anschluss zum DOIF. Ich habe hier nur den Code dazu nicht eingestellt. Wie gesagt, die einzelnen Teile der Bedingung funktionieren, auch die Readings werden anstandslos ausgelesen und der "notifyText" Text in meine Nachricht eingefügt.

[SCKWorld2012]

Hi Beta-User,

ich nutze in dem DOIF doch gar kein Event? Oder habe ich da das Konzept falsch verstanden? Beim Aufbau der Bedingungen und deren Kombination habe ich mich am Fhem-Wiki orientiert, da sah das ganz ähnlich aus.

[Otto123]

Hi,

der do Teil funktioniert für sich alleine, aber eben nicht in dem DOIF das hier:
set WindMonitoring alreadyNotified JA
heisst ja eigentlich
set $SELF alreadyNotified JA

Das triggert sich selbst!
[$SELF:alreadyNotified] eq "NEIN"
Das wird entweder verhindert oder nicht ausgeführt.
Eventuell hilft [?$SELF:alreadyNotified] eq "NEIN" und sleep 1;set $SELF alreadyNotified JA

Aber da bin ich nicht sicher.

Gruß Otto

[SCKWorld2012]

Hi Otto,

habe jetzt den fraglichen Teil jetzt mal aus dem "Do" Block herausgenommen, sodass es sich nicht mehr selbst Triggern dürfte.
Vorher habe ich mal versucht nur den den

Code Auswählen Erweitern

[$SELF:alreadyNotified] eq "NEIN" Teil in der Bedingung stehen zu lassen - so hat es funktioniert.

Die zusammegesetzte Bedingung funktioniert aber immer noch nicht 

Immerhin scheint es kein allzu offensichtlicher Fehler zu sein - dann habe ich ja immerhin meine Hausaufgaben in Sachen Recherche gemacht.

Gruß
Simon

[MadMax-FHEM]

ich nutze in dem DOIF doch gar kein Event? Oder habe ich da das Konzept falsch verstanden?

Wenn ich aber das hier lese: JA, falsch verstanden und besser mal nachlesen/nachholen

fhem ist (eigentlich) Event-Gesteuert.

Also es passiert etwas (auch ein "Timer" "passiert") und dann wird reagiert: notify/DOIF/FileLog/...

Kein Event, keine Reaktion...

Und: fhem ist single-Threaded, d.h. es gibt immer nur EINEN Event zur Zeit (wichtig bei DOIF, wo man schon mal mehrere Events "abfragen" will)...

Und: poste doch (trotzdem) mal das gesamte DOIF (also list WindMonitoring und evtl. auch von weiteren beteiligten Devices)...

Gruß, Joachim

[Beta-User]

Immerhin scheint es kein allzu offensichtlicher Fehler zu sein - dann habe ich ja immerhin meine Hausaufgaben in Sachen Recherche gemacht.

Wir haben das ständig...

[MyWindSensor:windSpeed] und [MyWindSensor:windGust] sind zumindest nach meiner Interpretation der Einleitung zu DOIF Ereignistrigger...:

Kombinierte Ereignis- und Zeitsteuerung

define di_lamp DOIF ([06:00-09:00] and [sensor:brightness] < 40) (set lamp on) DOELSE (set lamp off)

Aber wie schon geschrieben: DOIF kapiere ich auch nicht und löse sowas dann via notify.

PS: Du bist der Student, der eine Aufgabe erhalten hat. Etwas vertiefte Recherche ist DEIN JOB, nicht unserer .

[Otto123]

Die zusammengefügten Bedingungen ...
da empfehle ich auch immer eine Logiktabelle, damit man sieht was man erwarten kann. Du hast drei Bedingungen. Man muss die Klammer erst abarbeiten.
(A oder B) und C
0 0 -> 0    0 0 -> 0
1 0 -> 1    1 0 -> 0
0 1 -> 1    0 1 -> 0
1 1 -> 1    1 1 -> 1

C ist also für deinen Fall dominierend  A oder B (oder beide) muss wahr sein und C muss wahr sein für den CMD_1 Fall und zwar zur gleichen Zeit.
Alle Deine Bedingungen triggern und lösen das DOIF aus. Eventuell ist es gut auf einen Trigger zu verzichten und da nur Abfrage zu machen.

[MadMax-FHEM]

Muss das (Zeile 3):

0 1 -> 1    0 1 -> 0

nicht (auch) so:

0 1 -> 1    1 0 -> 0

EDIT: oder auch so 0 1 -> 1    1 1 -> 1

sein?

Oder verstehe ich die Tabelle falsch?
(und fehlen nicht generell noch ein paar Zeilen?)

EDIT: wobei ja klar ist worauf du hinaus willst/wolltest

Gruß, Joachim