What is the proper way to write this program?

[gweempose]

I want to write a program that will turn all my exterior lights on 15 minutes before sunset and turn them all off at sunrise. It seems like there are many ways to accomplish this. Would this work?

 

If

Sunset - 15 minutes to Sunrise (next day)

Then

Turn on front porch lights
Turn on dog run lights
Turn on patio lights

Else

Turn off front porch lights
turn off dog run lights
turn off patio lights

 

Or does it make more sense to have two separate programs. One to turn them on and the other to turn them off with no "else" statement?

[andyf0]

The program is fine, however you might want to create an "ALL" Scene that contains the porch lights, dog lights and patio lights.

 

Then you can just say

Set Scene "ALL" On

&

Set Scene "ALL" Off

[Michel Kohanim]

Hi,

 

Just a caveat, if you have thermostats, sending All On/All Off will cause the thermostat to go to an undefined state. I strongly advise against using it.

 

With kind regards,

Michel

[andyf0]

Michel, I was referring to a scene name "ALL" and doing a Set Scene ALL off. Not suggesting sending an All Off. How confusing I made it with my choice of scene name, sorry!

[gweempose]

Thanks for the help, guys! Now if I can just get my ISY to once again communicate with my patio lights, I'll be all set.