Akuelle 10_CUL_HM.pm ist bei mir fehlerhaft

[sepultura30]

Hallo,

die Aktuelle 10_CUL_HM.pm arbeit bei mir nicht sauber, im DOIF habe ich diesen Code drinnen:

([Flur_Oben] eq "on")
  (set Flur_Oben on-for-timer 120)

wenn jetzt das Licht angeht, dann springt das DOIF immer hin und her "cmd_1 cmd_2 cmd_1 cmd_2 u.s.w" und ich bekomme das Licht auch nicht mehr aus.
Mit dieser Version vom 20.02.2021 16:03:14 habe ich keine Probleme, die läuft sauber.

Grüße

Sandro

[frank]

zeig ein "list Flur_Oben"

[Otto123]

Hallo Sandro,

egal wie es mit den Versionen, das ist äußerst unsaubere Code!
Du musst 2 Dinge beachten:
1. STATE (state) bei einem Homematic Device zu verwenden ist äußerst kritisch! Schau Dir im Eventmonitor an was da passiert wenn Du schaltest!
2. Du setzt auf ein on (trigger) ein on-for-timer - der schon immer wieder ein state=on erzeugt. Das muss eigentlich schon immer eine  Schleife geben wie Du beschreibst. Wenn es das bisher nicht gemacht hat war es ein Fehler!

Nimm z.B. level der wird nicht neu gesetzt - etwas kritisch bleibt das (auf alle Fälle event-on-change-reading setzen).

Code: [Auswählen]

([Flur_Oben:level] == 100)   (set Flur_Oben on-for-timer 120)
Programmiere doch den Schalter direkt auf timer? Mit Template oder mit register direkt (hier beschrieben)
Code ist nur Prinzip und stimmt so nicht!

Code: [Auswählen]

set hminfo templateSet Flur_Oben autoOff self01:short 120
Gruß Otto

[sepultura30]

Hallo Otto,

besten Dank, so geht es und ich habe alles bei mir umgestellt.

Hallo Sandro,

egal wie es mit den Versionen, das ist äußerst unsaubere Code!
Du musst 2 Dinge beachten:
1. STATE (state) bei einem Homematic Device zu verwenden ist äußerst kritisch! Schau Dir im Eventmonitor an was da passiert wenn Du schaltest!
2. Du setzt auf ein on (trigger) ein on-for-timer - der schon immer wieder ein state=on erzeugt. Das muss eigentlich schon immer eine  Schleife geben wie Du beschreibst. Wenn es das bisher nicht gemacht hat war es ein Fehler!

Nimm z.B. level der wird nicht neu gesetzt - etwas kritisch bleibt das (auf alle Fälle event-on-change-reading setzen).

Code: [Auswählen]

([Flur_Oben:level] == 100)   (set Flur_Oben on-for-timer 120)
Programmiere doch den Schalter direkt auf timer? Mit Template oder mit register direkt (hier beschrieben)
Code ist nur Prinzip und stimmt so nicht!

Code: [Auswählen]

set hminfo templateSet Flur_Oben autoOff self01:short 120
Gruß Otto