Problem mit DOIF

Starsurfer · 12 August 2019, 21:08:31

Moin,

ich bräuchte mal ein Hilfe bei einem DOIF:

Zur Zeit habe ich im Garten ein paar Gartenstrahler mit innr GU10 und Osram Gartenpoles.

Die Strahler werden zur Zeit über Bewegungsmelder angeschaltet und die Gartenpoles über ein Helligkeitssensor.

Meine Idee ist jetzt, über den Helligkeitssensor alle Lampen im Garten gedimmt einzuschalten und wenn Bewegung erkannt wird, die Lampen auf voller Helligkeit anzuschalten und wenn keine Bewegung mehr erkannt wird, wieder auf den Ursprünglichen Zustand zu schalten.

Dazu habe ich mir folgendes DOIF erstellt:

Code Auswählen

(([5:50-7:30] or [16:00-22:10] and [MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder_Hexen:"nomotion"] or [Bewegungsmelder_Schleppdach:"nomotion"]))
(set HUEDevice9 pct 10)
DOELSEIF
([MultiSensor_Einfahrt:brightness] < 40 and ([Bewegungsmelder_Hexen:"motion"] or [Bewegungsmelder_Schleppdach:"motion"])) 
(set HUEDevice9 pct 100)
DOELSE
(set HUEDevice9 off)

Das List dazu:

Code Auswählen

Internals:
   DEF        (([5:50-7:30] or [16:00-22:10] and [MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder_Hexen:"nomotion"] or [Bewegungsmelder_Schleppdach:"nomotion"]))
(set HUEDevice9 pct 10)
DOELSEIF
([MultiSensor_Einfahrt:brightness] < 40 and ([Bewegungsmelder_Hexen:"motion"] or [Bewegungsmelder_Schleppdach:"motion"])) 
(set HUEDevice9 pct 100)
DOELSE
(set HUEDevice9 off)
   FUUID      5d484da7-f33f-ae29-81bf-44385bb53101ea42
   MODEL      FHEM
   NAME       doif_Garten_Gardenpole
   NR         231
   NTFY_ORDER 50-doif_Garten_Gardenpole
   STATE      cmd_3
   TYPE       DOIF
   VERSION    19786 2019-07-05 21:47:08
   READINGS:
     2019-08-12 20:57:44   Device          MultiSensor_Einfahrt
     2019-08-12 16:25:23   cmd             3
     2019-08-12 16:25:23   cmd_event       doif_Garten_Gardenpole
     2019-08-12 16:25:23   cmd_nr          3
     2019-08-12 20:52:43   e_Bewegungsmelder_Hexen_events temperature: 21
     2019-08-12 20:49:57   e_Bewegungsmelder_Schleppdach_events nomotion
     2019-08-12 20:57:44   e_MultiSensor_Einfahrt_brightness 28.98
     2019-08-12 16:25:20   mode            enabled
     2019-08-12 16:25:23   state           cmd_3
     2019-08-12 18:30:59   timer_01_c01    13.08.2019 05:50:00
     2019-08-12 18:30:59   timer_02_c01    13.08.2019 07:30:00
     2019-08-12 18:30:59   timer_03_c01    13.08.2019 16:00:00
     2019-08-12 18:30:59   timer_04_c01    12.08.2019 22:10:00
   Regex:
     accu:
   attr:
     waitdel:
   condition:
     0          (::DOIF_time($hash,0,1,$wday,$hms) or ::DOIF_time($hash,2,3,$wday,$hms) and ::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40) and (::EventDoIf('Bewegungsmelder_Hexen',$hash,'nomotion',1) or ::EventDoIf('Bewegungsmelder_Schleppdach',$hash,'nomotion',1))
     1          ::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40 and (::EventDoIf('Bewegungsmelder_Hexen',$hash,'motion',1) or ::EventDoIf('Bewegungsmelder_Schleppdach',$hash,'motion',1))
   days:
   devices:
     0           MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     1           MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     all         MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
   do:
     0:
       0          set HUEDevice9 pct 10
     1:
       0          set HUEDevice9 pct 100
     2:
       0          set HUEDevice9 off
   helper:
     event      batVoltage: 3.18,battery: ok,brightness: 28.98,customData: 0,digitalInput: 0,humidity: 41,pressure: 1014.4,T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0,temperature: 20.3
     globalinit 1
     last_timer 4
     sleeptimer -1
     timerdev   MultiSensor_Einfahrt
     timerevent batVoltage: 3.18,battery: ok,brightness: 28.98,customData: 0,digitalInput: 0,humidity: 41,pressure: 1014.4,T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0,temperature: 20.3
     triggerDev MultiSensor_Einfahrt
     timerevents:
       batVoltage: 3.18
       battery: ok
       brightness: 28.98
       customData: 0
       digitalInput: 0
       humidity: 41
       pressure: 1014.4
       T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0
       temperature: 20.3
     timereventsState:
       batVoltage: 3.18
       battery: ok
       brightness: 28.98
       customData: 0
       digitalInput: 0
       humidity: 41
       pressure: 1014.4
       state: T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0
       temperature: 20.3
     triggerEvents:
       batVoltage: 3.18
       battery: ok
       brightness: 28.98
       customData: 0
       digitalInput: 0
       humidity: 41
       pressure: 1014.4
       T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0
       temperature: 20.3
     triggerEventsState:
       batVoltage: 3.18
       battery: ok
       brightness: 28.98
       customData: 0
       digitalInput: 0
       humidity: 41
       pressure: 1014.4
       state: T: 20.3 P: 1014.4 H: 41 B: 28.98 I: 0 X: 0
       temperature: 20.3
   internals:
   interval:
     0          -1
     1          0
     2          -1
     3          2
   intervalfunc:
   itimer:
   localtime:
     0          1565668200
     1          1565674200
     2          1565704800
     3          1565640600
   perlblock:
   readings:
     0           MultiSensor_Einfahrt:brightness 
     1           MultiSensor_Einfahrt:brightness 
     all         MultiSensor_Einfahrt:brightness 
   realtime:
     0          05:50:00
     1          07:30:00
     2          16:00:00
     3          22:10:00
   time:
     0          5:50
     1          7:30
     2          16:00:00
     3          22:10:00
   timeCond:
     0          0
     1          0
     2          0
     3          0
   timer:
     0          0
     1          0
     2          0
     3          0
   timers:
     0           0  1  2  3 
   trigger:
     all         Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
   triggertime:
     1565640600:
       localtime  1565640600
       hash:
     1565668200:
       localtime  1565668200
       hash:
     1565674200:
       localtime  1565674200
       hash:
     1565704800:
       localtime  1565704800
       hash:
   uiState:
   uiTable:
Attributes:
   DbLogExclude .*
   icon       icoUhr
   room       Zeitschaltuhren

Es ist jetzt kurz nach 21 Uhr, der Helligkeitssensor meldet knappe 28 LUX und die Bewegungsmelder stehen auf nomotion. Das Doif befindet sich aber in cmd_3.

Ich kann meinen Fehler leider nicht finden, vielleicht kann mir ja jemand auf die Sprünge helfen.

Danke

Otto123 · 12 August 2019, 21:20:29

Hi,

Du musst auf alle Fälle das erste or noch klammern.

Gruß Otto

amenomade · 12 August 2019, 21:24:53

Du kombinierst Status und Ereignissteuerung => Konkret wird dein DOIF hier nur durch Ereignisse "nomotion" von den Bewegungsmelder getriggert

Um 16:25 hat er cmd_3 geschaltet.
Um 20:49 hat Bewegungsmelder_Hexen "nomotion" gemeldet. Wie war damals MultiSensor_Einfahrt:brightness ? Wahrscheinlich noch über 40
28.98 kommt auf jeden Fall erst danach

amenomade · 12 August 2019, 21:26:12

Zitat von: Otto123 am 12 August 2019, 21:20:29
Hi,

Du musst auf alle Fälle das erste or noch klammern.

Gruß Otto

Naja... das habe ich auch am Anfang gedacht. Aber vielleicht will er das es morgens "auf jeden Fall" schaltet, und abends "nur" wenn die Helligkeit gering ist

Starsurfer · 12 August 2019, 21:40:20

Zitat von: amenomade am 12 August 2019, 21:26:12
Naja... das habe ich auch am Anfang gedacht. Aber vielleicht will er das es morgens "auf jeden Fall" schaltet, und abends "nur" wenn die Helligkeit gering ist

Nein beides nur wenn die Helligkeit unter 40 Lux ist.

Otto123 · 12 August 2019, 21:41:12

Zitat von: amenomade am 12 August 2019, 21:26:12
Naja... das habe ich auch am Anfang gedacht. Aber vielleicht will er das es morgens "auf jeden Fall" schaltet, und abends "nur" wenn die Helligkeit gering ist

Ja kann auch sein

ich denke mit Deiner Anmerkung über die Ereignissteuerung liegst Du richtig. Damit spielt die Helligkeit keine triggernde Rolle mehr

amenomade · 12 August 2019, 21:42:35

Zitat von: Starsurfer am 12 August 2019, 21:40:20
Nein beides nur wenn die Helligkeit unter 40 Lux ist.

Dann musst Du, wie Otto geschrieben hat, noch mehr klammern. And bindet mehr als or. Ändert aber nichts an der Tatsache, dass dein DOIF eventgetriggert ist

Otto123 · 12 August 2019, 21:42:48

Und Umbau auf die Form [Bewegungsmelder_Hexen] eq "motion"

amenomade · 12 August 2019, 21:44:33

Zitat von: Otto123 am 12 August 2019, 21:42:48
Und Umbau auf die Form [Bewegungsmelder_Hexen] eq "motion"

oder ggf "nomotion".

Zur Erklärung, nehmen wir mal folgendes DOIF:

Code Auswählen

([du:"^on$"]) (set du4 on) DOELSE (set du4 off)
Ein Ereignis "on" auf Device "du" wird du4 auf on schalten
ABER alle andere Ereignisse auf "du" werden du4 wieder auf off schalten

Starsurfer · 12 August 2019, 21:48:12

So habe es jetzt so umgebaut:

Code Auswählen

(([5:50-7:30] or [16:00-22:10]) and ([MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder_Hexen] eq "nomotion" or [Bewegungsmelder_Schleppdach] eq "nomotion"))
(set HUEGroup2 pct 10,
set HUEDevice9 pct 10)
DOELSEIF
([MultiSensor_Einfahrt:brightness] < 40 and [Bewegungsmelder_Hexen] eq "motion" or [Bewegungsmelder_Schleppdach] eq "motion") 
(set HUEGroup2 pct 100,
set HUEDevice9 pct 100)
DOELSE
(set HUEGroup2 off,
set HUEDevice9 off)

List:

Code Auswählen

Internals:
   DEF        (([5:50-7:30] or [16:00-22:10]) and ([MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder_Hexen] eq "nomotion" or [Bewegungsmelder_Schleppdach] eq "nomotion"))
(set HUEGroup2 pct 10,
set HUEDevice9 pct 10)
DOELSEIF
([MultiSensor_Einfahrt:brightness] < 40 and [Bewegungsmelder_Hexen] eq "motion" or [Bewegungsmelder_Schleppdach] eq "motion") 
(set HUEGroup2 pct 100,
set HUEDevice9 pct 100)
DOELSE
(set HUEGroup2 off,
set HUEDevice9 off)
   FUUID      5d0be02c-f33f-ae29-77fc-0419a24f25ce354f
   MODEL      FHEM
   NAME       doif_Garten_Strahler
   NR         194
   NTFY_ORDER 50-doif_Garten_Strahler
   STATE      cmd_1
   TYPE       DOIF
   VERSION    19786 2019-07-05 21:47:08
   READINGS:
     2019-08-12 21:46:11   cmd             1
     2019-08-12 21:46:11   cmd_event       doif_Garten_Strahler
     2019-08-12 21:46:11   cmd_nr          1
     2019-08-12 21:46:07   mode            enabled
     2019-08-12 21:46:11   state           cmd_1
     2019-08-12 21:46:07   timer_01_c01    13.08.2019 05:50:00
     2019-08-12 21:46:07   timer_02_c01    13.08.2019 07:30:00
     2019-08-12 21:46:07   timer_03_c01    13.08.2019 16:00:00
     2019-08-12 21:46:07   timer_04_c01    12.08.2019 22:10:00
   Regex:
     accu:
   attr:
     cmdState:
     wait:
       0:
         0
       1:
         130
     waitdel:
   condition:
     0          (::DOIF_time($hash,0,1,$wday,$hms) or ::DOIF_time($hash,2,3,$wday,$hms)) and (::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40) and (::InternalDoIf($hash,'Bewegungsmelder_Hexen','STATE') eq "nomotion" or ::InternalDoIf($hash,'Bewegungsmelder_Schleppdach','STATE') eq "nomotion")
     1          ::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40 and ::InternalDoIf($hash,'Bewegungsmelder_Hexen','STATE') eq "motion" or ::InternalDoIf($hash,'Bewegungsmelder_Schleppdach','STATE') eq "motion"
   days:
   devices:
     0           MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     1           MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     all         MultiSensor_Einfahrt Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
   do:
     0:
       0          set HUEGroup2 pct 10, set HUEDevice9 pct 10
     1:
       0          set HUEGroup2 pct 100, set HUEDevice9 pct 100
     2:
       0          set HUEGroup2 off, set HUEDevice9 off
   helper:
     globalinit 1
     last_timer 4
     sleeptimer -1
     timerdev   
     timerevent 
     timerevents 
     timereventsState 
     triggerDev 
     DOIF_eventas:
       cmd_nr: 1
       cmd: 1
       cmd_event: doif_Garten_Strahler
       state: cmd_1
   internals:
     0           Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
     1           Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
     all         Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
   interval:
     0          -1
     1          0
     2          -1
     3          2
   intervalfunc:
   itimer:
   localtime:
     0          1565668200
     1          1565674200
     2          1565704800
     3          1565640600
   readings:
     0           MultiSensor_Einfahrt:brightness 
     1           MultiSensor_Einfahrt:brightness 
     all         MultiSensor_Einfahrt:brightness 
   realtime:
     0          05:50:00
     1          07:30:00
     2          16:00:00
     3          22:10:00
   time:
     0          5:50
     1          7:30
     2          16:00:00
     3          22:10:00
   timeCond:
     0          0
     1          0
     2          0
     3          0
   timer:
     0          0
     1          0
     2          0
     3          0
   timers:
     0           0  1  2  3 
   triggertime:
     1565640600:
       localtime  1565640600
       hash:
     1565668200:
       localtime  1565668200
       hash:
     1565674200:
       localtime  1565674200
       hash:
     1565704800:
       localtime  1565704800
       hash:
   uiState:
   uiTable:
Attributes:
   DbLogExclude .*
   do         resetwait
   icon       icoUhr
   room       Zeitschaltuhren
   wait       0:130

Zumindest ist er jetzt schon mal auf cmd_1 gesprungen. Muss gleich mal raus rennen und einen Bewegungsmelder aktivieren.

amenomade · 12 August 2019, 21:51:18

Schon mal besser, ja
Hoffe es regnet nicht

Und ich hoffe auch, dass pct 100 nicht genug Licht bringt, dass MultiSensor_Einfahrt:brightness über 40 springt

Starsurfer · 12 August 2019, 21:57:25

Zitat von: amenomade am 12 August 2019, 21:51:18
Schon mal besser, ja
Hoffe es regnet nicht
Und ich hoffe auch, dass pct 100 nicht genug Licht bringt, dass MultiSensor_Einfahrt:brightness über 40 springt

Nein regnet nicht

Nein, der steht auf der anderen Seite vom Haus

Allerdings springt er bei motion nicht in cmd_2

amenomade · 12 August 2019, 21:59:02

list?

amenomade · 12 August 2019, 22:00:21

Ahja übrigens

Code Auswählen

([MultiSensor_Einfahrt:brightness] < 40 and [Bewegungsmelder_Hexen] eq "motion" or [Bewegungsmelder_Schleppdach] eq "motion")

Da hast Du wiederum ein Klammer Problem.

EDIT: vergiss nicht, dass damit er aus cmd_1 rauskommt, BEIDE Bewegungsmelder auf "motion" sein müssen, sonst ist der "nomotion or nomotion" immer noch wahr

Starsurfer · 12 August 2019, 22:01:28

Nicht wundern, ist ein Bewegungsmelder mehr dazu gekommen, der steht hier auf dem Schreibtisch, damit ich nicht immer raus rennen muss.

List:

Code Auswählen

Internals:
   DEF        (([5:50-7:30] or [16:00-22:10]) and ([MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder] eq "nomotion" or [Bewegungsmelder_Hexen] eq "nomotion" or [Bewegungsmelder_Schleppdach] eq "nomotion"))
(set HUEGroup2 pct 10,
set HUEDevice9 pct 10)
DOELSEIF
(([MultiSensor_Einfahrt:brightness] < 40) and ([Bewegungsmelder] eq "motion" or [Bewegungsmelder_Hexen] eq "motion" or [Bewegungsmelder_Schleppdach] eq "motion")) 
(set HUEGroup2 pct 100,
set HUEDevice9 pct 100)
DOELSE
(set HUEGroup2 off,
set HUEDevice9 off)
   FUUID      5d0be02c-f33f-ae29-77fc-0419a24f25ce354f
   MODEL      FHEM
   NAME       doif_Garten_Strahler
   NR         194
   NTFY_ORDER 50-doif_Garten_Strahler
   STATE      cmd_1
   TYPE       DOIF
   VERSION    19786 2019-07-05 21:47:08
   READINGS:
     2019-08-12 22:00:08   Device          Bewegungsmelder
     2019-08-12 22:00:08   cmd             1
     2019-08-12 22:00:08   cmd_event       Bewegungsmelder
     2019-08-12 22:00:08   cmd_nr          1
     2019-08-12 22:00:08   e_Bewegungsmelder_STATE nomotion
     2019-08-12 21:55:03   mode            enabled
     2019-08-12 22:00:08   state           cmd_1
     2019-08-12 21:55:03   timer_01_c01    13.08.2019 05:50:00
     2019-08-12 21:55:03   timer_02_c01    13.08.2019 07:30:00
     2019-08-12 21:55:03   timer_03_c01    13.08.2019 16:00:00
     2019-08-12 21:55:03   timer_04_c01    12.08.2019 22:10:00
   Regex:
     accu:
   attr:
     cmdState:
     wait:
       0:
         0
       1:
         130
     waitdel:
   condition:
     0          (::DOIF_time($hash,0,1,$wday,$hms) or ::DOIF_time($hash,2,3,$wday,$hms)) and (::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40) and (::InternalDoIf($hash,'Bewegungsmelder','STATE') eq "nomotion" or ::InternalDoIf($hash,'Bewegungsmelder_Hexen','STATE') eq "nomotion" or ::InternalDoIf($hash,'Bewegungsmelder_Schleppdach','STATE') eq "nomotion")
     1          (::ReadingValDoIf($hash,'MultiSensor_Einfahrt','brightness') < 40) and (::InternalDoIf($hash,'Bewegungsmelder','STATE') eq "motion" or ::InternalDoIf($hash,'Bewegungsmelder_Hexen','STATE') eq "motion" or ::InternalDoIf($hash,'Bewegungsmelder_Schleppdach','STATE') eq "motion")
   days:
   devices:
     0           MultiSensor_Einfahrt Bewegungsmelder Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     1           MultiSensor_Einfahrt Bewegungsmelder Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
     all         MultiSensor_Einfahrt Bewegungsmelder Bewegungsmelder_Hexen Bewegungsmelder_Schleppdach 
   do:
     0:
       0          set HUEGroup2 pct 10, set HUEDevice9 pct 10
     1:
       0          set HUEGroup2 pct 100, set HUEDevice9 pct 100
     2:
       0          set HUEGroup2 off, set HUEDevice9 off
   helper:
     event      nomotion
     globalinit 1
     last_timer 4
     sleeptimer -1
     timerdev   Bewegungsmelder
     timerevent nomotion
     triggerDev Bewegungsmelder
     DOIF_eventas:
       cmd_nr: 1
       cmd: 1
       cmd_event: Bewegungsmelder
       state: cmd_1
     timerevents:
       nomotion
     timereventsState:
       state: nomotion
     triggerEvents:
       nomotion
     triggerEventsState:
       state: nomotion
   internals:
     0           Bewegungsmelder:STATE Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
     1           Bewegungsmelder:STATE Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
     all         Bewegungsmelder:STATE Bewegungsmelder_Hexen:STATE Bewegungsmelder_Schleppdach:STATE 
   interval:
     0          -1
     1          0
     2          -1
     3          2
   intervalfunc:
   itimer:
   localtime:
     0          1565668200
     1          1565674200
     2          1565704800
     3          1565640600
   readings:
     0           MultiSensor_Einfahrt:brightness 
     1           MultiSensor_Einfahrt:brightness 
     all         MultiSensor_Einfahrt:brightness 
   realtime:
     0          05:50:00
     1          07:30:00
     2          16:00:00
     3          22:10:00
   time:
     0          5:50
     1          7:30
     2          16:00:00
     3          22:10:00
   timeCond:
     0          0
     1          0
     2          0
     3          0
   timer:
     0          0
     1          0
     2          0
     3          0
   timers:
     0           0  1  2  3 
   trigger:
   triggertime:
     1565640600:
       localtime  1565640600
       hash:
     1565668200:
       localtime  1565668200
       hash:
     1565674200:
       localtime  1565674200
       hash:
     1565704800:
       localtime  1565704800
       hash:
   uiState:
   uiTable:
Attributes:
   DbLogExclude .*
   do         resetwait
   icon       icoUhr
   room       Zeitschaltuhren
   wait       0:130