GPIO shutdown program help

[Worldfrog]

So I made a shutdown program, and the program worked the first few times, but appears to not work when I push the button that should execute code

Code: Select all

import RPi.GPIO as GPIO
import os
import time
import Tkinter
import tkMessageBox
GPIO.setmode(GPIO.BOARD)
GPIO.setup(16,GPIO.IN, pull_up_down=GPIO.PUD_DOWN
try:
while True:
if(GPIO.input(16)==1):
time.sleep(2);os.system("sudo shutdown -h now")
else:
#not necessary but added to show the program is alive
time.sleep(15);print("On")
except KeyboardInterrupt:
GPIO.cleanup();window=Tkinter.Tk();window.wm_withdraw();tkMessageBox.showinfo(title="Warning", message="Shutdown Program has been halted")

So I'm wondering if
A) My GPIO pin got messed up or
B) My code has something wrong

[Worldfrog]

Update: removing the time.sleep(15) before print("On") seems to have fixed it... why?

[gordon77]

How long are you pressing the button? You could need to press it for 15 seconds.

[Worldfrog]

How long are you pressing the button? You could need to press it for 15 seconds.

I was pressing for maybe a second at most. However, I agree, it probably wasn't working because of my sleep command which froze all my code.

[rpdom]

You seem to have lost all your indentation (spacing) on posting that code and you have multiple statements per line, which makes it difficult to follow.

Can you repost it with the leading spaces and the code on separate lines so it makes sense?

[gordon77]

How long are you pressing the button? You could need to press it for 15 seconds.

I was pressing for maybe a second at most. However, I agree, it probably wasn't working because of my sleep command which froze all my code.

sleep(15) will stop it doing anything for 15 seconds, even reading your switch.

[Worldfrog]

You seem to have lost all your indentation (spacing) on posting that code and you have multiple statements per line, which makes it difficult to follow.

Can you repost it with the leading spaces and the code on separate lines so it makes sense?

I can, here it is:

Code: Select all

import RPi.GPIO as GPIO
import os
import time
import Tkinter
import tkMessageBox
GPIO.setmode(GPIO.BOARD)
GPIO.setup(16,GPIO.IN, pull_up_down=GPIO.PUD_DOWN
try:
    while True:
        if(GPIO.input(16)==1):
            time.sleep(2)
            os.system("sudo shutdown -h now")
        else:
#not necessary but added to show the program is alive
            time.sleep(15);print("On")
except KeyboardInterrupt:
    GPIO.cleanup()
    window=Tkinter.Tk()
    window.wm_withdraw()
    tkMessageBox.showinfo(title="Warning", message="Shutdown Program has been halted")

But I've figured out my problem, as gordon said, my time.sleep(15) paused all my code