Logic Licht blinken lassen...

[Msinn]

Ja das geht besser: Du kannst das so formatieren das man es lesen kann

[mumpf]

Not ... gibt einen bool Wert (True oder False), jedoch nicht wie Du erwartest 1 oder 0

Hi, daran kann es nicht liegen, da int und bool zuweisungskompatibel sind. Ich vermute, dass die Blinkbefehle gar nicht aufgerufen werden, weil Du

if sh.OG.Info_Bewegung_GangWC_OG() == 1:

schreibst, aber eigentlich das Item

if sh.OG.Info_Bewegung_GangWC_OG.OnOff() == 1:

meinst.

Gruß, Waldemar

[rok123]

Hi, hab jetzt noch ein Problem mit meiner Logik.
Wurde jetzt erweitert, dass das Licht (wenn es eingeschaltet war) nach dem blinken wieder an ist.
Jedoch bleibt das Licht jetzt immer an. Vielleicht findet ihr ja den Fehler (bin mir nicht sicher wie das time.sleep arbeitet, hält es die ganze Logik an?

if sh.OG.Info_Bewegung_GangWC_OG() == 1:
    if sh.OG.pm_wc_og_HKL() == 1:
        if sh.EG.Essplatz.Deckenlicht.OnOff(1):
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
        else:
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(2)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)


    if sh.OG.pm_gang_og_HKL() == 1:
        if sh.EG.Essplatz.Deckenlicht.OnOff(0):
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
        elif sh.EG.Essplatz.Deckenlicht.OnOff(1):
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(0)
            sh.EG.Essplatz.Deckenlicht.OnOff(0)
            time.sleep(3)
            sh.EG.deckenlicht_wohnzimmer.OnOff(1)
            sh.EG.Essplatz.Deckenlicht.OnOff(1)

[bmx]

Warum schreibst Du 'if sh.EG.Essplatz.Deckenlicht.OnOff(1):' und nicht 'if sh.EG.Essplatz.Deckenlicht.OnOff() == 1:' ?

[Tom Bombadil]

Nur mal so eine Idee: Sollte das nicht in der aktuellen Version auch direkt über Items statt über eine separate Logik lösbar sein (Stichwort: Schreiben auf 'fremde' Items über eval/autotimer)? Dann könnte man die Funktionalität direkt auf dem ein-/auszuschaltenden Item bzw. dessen Sub-Items abbilden, das wäre vielleicht in Summe lesbarer.

Muss ich mich mal reindenken, aber nicht mehr heute ...

/tom

[rok123]

bmx danke, jetzt funktioniert es.