Why does this scene not work. Email alert multiple times.

[rajp 14]

Hi.. I am using the following block scene to see if the weather outside is Rain or Storm and if so, it should set a delay of 24hours in the sprinkler panel. However, when it should run, it should only run at 12:00 mid day once. However, when it did run, it ended up sending out over a 1000 email alerts to me. So I turned off the scene.

 

What am I doing wrong???

 

Please login or register to see this attachment.

[boomx 43]

Try to uncheck the triggers

Please login or register to see this image.

/emoticons/default_wink.png" alt=";)" srcset="https://forum.fibaro.com/uploads/emoticons/[email protected] 2x" width="20" height="20" />

[rajp 14]

Try to uncheck the triggers

Please login or register to see this image.

/emoticons/default_wink.png" alt=";)" srcset="https://forum.fibaro.com/uploads/emoticons/[email protected] 2x" width="20" height="20" />

Ahh. Thanks. Totally missed that!!!!!

[boomx 43]

No problem

Please login or register to see this image.

/emoticons/default_smile.png" alt=":)" srcset="https://forum.fibaro.com/uploads/emoticons/[email protected] 2x" width="20" height="20" />

[Hoggle 1]

What is the use of the "triggers"?

What can they control ?

[rajp 14]

What is the use of the "triggers"?

What can they control ?

I believe in my scene, the 3 triggers are the Global Variable, Weather and the time. So if either of the components had a change in condition, it will trigger the scene. So if I wanted to trigger the scene alone on time then to deselect the Weather and Global variable trigger should only trigger on time.

 

Well, that is my understanding..

[rajp 14]

Try to uncheck the triggers

Please login or register to see this image.

/emoticons/default_wink.png" alt=";)" srcset="https://forum.fibaro.com/uploads/emoticons/[email protected] 2x" width="20" height="20" />

OK.. Unchecked those 2 triggers and then set it running. Yesterday, we had rain forecast and the scene again sent me around 40 or so messages around 2PM before I termnated the scene.. so that didn't fix it.. pretty confused as to why it keeps running past the 12:00 as I want it to run only at 12:00.

[1152 33]

I presume it is your use of "or" and how that plays into the rest of and statements, specifically rain versus the sprinkler delay. It is a different topic here on the forum addressing nesting like you have done, sometimes it plays out different than you expected. 

 

Try to convert it to lua and check if the logics int the statements then become more obvious. 

[rajp 14]

I presume it is your use of "or" and how that plays into the rest of and statements, specifically rain versus the sprinkler delay. It is a different topic here on the forum addressing nesting like you have done, sometimes it plays out different than you expected. 

 

Try to convert it to lua and check if the logics int the statements then become more obvious. 

 

This is the code converted to LUA.

 

--[[

%% properties

%% globals

--]]

local sourceTrigger = fibaro:getSourceTrigger();

function tempFunc()

local currentDate = os.date("*t");

local startSource = fibaro:getSourceTrigger();

if (

 ( fibaro:getValue(3, "WeatherConditionConverted") == "rain" )

or

 ( fibaro:getValue(3, "WeatherConditionConverted") == "storm" )

and

 ( tonumber(fibaro:getGlobalValue("sprinklerdelay")) ~= tonumber("0")  and  ((currentDate.day == 1 or currentDate.day == 2 or currentDate.day == 3 or currentDate.day == 4 or currentDate.day == 5 or currentDate.day == 6 or currentDate.day == 7) and string.format("%02d", currentDate.hour) .. ":" .. string.format("%02d", currentDate.min) == "12:00") )

)

then

    fibaro:call(319, "pressButton", "2");

    fibaro:setGlobal("sprinklerdelay", "0");

    fibaro:call(13, "sendDefinedEmailNotification", "325");

end

setTimeout(tempFunc, 60*1000)

end

if (sourceTrigger["type"] == "autostart") then

tempFunc()

else

local currentDate = os.date("*t");

local startSource = fibaro:getSourceTrigger();

if (

 ( fibaro:getValue(3, "WeatherConditionConverted") == "rain" )

or

 ( fibaro:getValue(3, "WeatherConditionConverted") == "storm" )

and

 ( tonumber(fibaro:getGlobalValue("sprinklerdelay")) ~= tonumber("0")  and  (currentDate.day == 1 or currentDate.day == 2 or currentDate.day == 3 or currentDate.day == 4 or currentDate.day == 5 or currentDate.day == 6 or currentDate.day == 7) )

or

startSource["type"] == "other"

)

then

    fibaro:call(319, "pressButton", "2");

    fibaro:setGlobal("sprinklerdelay", "0");

    fibaro:call(13, "sendDefinedEmailNotification", "325");

end

end