Lights going on-off-on with 2 motion sensors

[Rikbast 3]

Hi,

 

I have a small piece op code:

 

if (LastModifiedMotionKitchen < LastModifiedMotionHallway) and (LightStatusHallway == '0') then

fibaro:call(203, "pressButton", "2")

end

 

203 = VD with on/off status (Philips Hue lamp), button 2 starts the Hue scene

 

So, I leave the kitchen (motion detected) en go into the hallway (motion detected) and light in hallway is off. It turns on but after a second or so, light goes off again. Then when I move in the hallway the light goes on again.

I can't grasp what is going on here. I would not expect the on-off-on sequence as the conditions were full filled.

 

Anyone an idea? They are FIBARO motion sensors. Maybe settings on the motion sensors?

thx!

[Thomasn 77]

I think it is because every movement detected by the pirs will make your script press the on/off switch. You need to make a script that only press the button when you want it. How about making a loop based on time?

[Rikbast 3]

1 hour ago, Thomasn said:

I think it is because every movement detected by the pirs will make your script press the on/off switch. You need to make a script that only press the button when you want it. How about making a loop based on time?

 

But the statement only runs when the lamp is off, and suppose it is on, so it should pass this statement and not pressing the 'Off' button, that should happen when I leave the room.

 

Can you give an example what you mean by timed loop? Any help is appreciated.

[Thomasn 77]

I get it that your statements only runs when it is true. But globals can take a second to load - so the statement could be true long enough to run again upon movement.

 

I have made a pir program here: 

Please login or register to see this link.

 

[Rikbast 3]

I made a mistake in my code: LastModifiedMotionKitchen and LastModifiedMotionHallway was indeed the variable 'lastmodified', but should have been 'lastbreached'.

Now it works. For timer I use ((os.time()) -  tonumber(LastModifiedBreachedKitchen) >= Timer) instead of 'Sleep'. I don't know if this is better or not, but works well.

 

I sure will have a look at your website with examples, looks very nice! Thank you for the help