Problem mit Python / Beschaltung *einfach*

[sunshineh (Gast)]

Hallo,

leider bekomme ich jede Minute einen Alarm, obwohl die Lichtschranke gar 
nicht schließt.

Hier ist mein Code. Für nen Tipp wäre ich dankbar, da ich grad den Wald 
vor lauter Bäumen nicht sehe...

#!/usr/bin/env python
# -*- coding: utf-8 -*-
import RPi.GPIO as GPIO
import smtplib
import time
BtnPin = 11
Gpin   = 12
Rpin   = 13
fromaddr = '[e-mail]'
toaddrs  = '[e-mail]'
msg = """From: Raspberry Pi Alarmsystem
To: [e-mail]
Subject: Die Lichtschranke wurde unterbrochen
Achtung: Ihre Lichtschranke wurde unterbrochen, bitte pruefen sie die Ursache!
"""
username = [e-mail]'
password = 'xxx
def setup():
    GPIO.setmode(GPIO.BOARD)       # Numbers GPIOs by physical location
    GPIO.setup(Gpin, GPIO.OUT)     # Set Green Led Pin mode to output
    GPIO.setup(Rpin, GPIO.OUT)     # Set Red Led Pin mode to output
    GPIO.setup(BtnPin, GPIO.IN, pull_up_down=GPIO.PUD_UP)    # Set BtnPin's mode is input, and pull up to high level(3.3V)
    GPIO.add_event_detect(BtnPin, GPIO.BOTH, callback=detect, bouncetime=200)
def Led(x):
    if x == 1:
        GPIO.output(Rpin, 1)
        GPIO.output(Gpin, 0)
    if x == 0:
        GPIO.output(Rpin, 0)
        GPIO.output(Gpin, 1)
def Print(x):
    if x == 0:
        print '    ***********************'
        print '    *   Button Pressed!   *'
        print '    ***********************'
def SendMail():
        server = smtplib.SMTP('smtp.gmail.com:587')
        server.starttls()
        server.login(username,password)
        server.sendmail(fromaddr, toaddrs, msg)
        server.quit()
        time.sleep(60)
        print("Mail gesendet")
def WriteFile():
        date = time.strftime("%x")
        zeit = time.strftime("%X2")
        fileout = open("/home/pi/Desktop/Lichtschranke.txt","a")
        time.sleep(0.1)
        fileout.write(date+";"+zeit+"; Die Lichtschranke wurde unterbrochen \n")
        time.sleep(0.1)
        fileout.close()
        print("Log-File geschrieben")
def detect(chn):
    WriteFile()        
    print(GPIO.input(BtnPin))
    SendMail()

def loop():
    while True:
        pass
def destroy():
    GPIO.output(Gpin, GPIO.HIGH)       # Green led off
    GPIO.output(Rpin, GPIO.HIGH)       # Red led off
    GPIO.cleanup()                     # Release resource
if __name__ == '__main__':     # Program start from here
    setup()
    try:
        loop()
    except KeyboardInterrupt:  # When 'Ctrl+C' is pressed, the child program destroy() will be  executed.
        destroy()

[Simon T. (simon_t399)]

Hallo,

was passiert denn, wenn du in "detect" mal alles sein lässt und 
stattdessen dort einfach mal etwas ausgibtst?

Alternativ weiß ich auch nicht ganz genau, was das 
"GPIO.setup(BtnPin..." tut, wenn man einen Pull-Up-Widerstand extern 
eingebaut hat (hast du ja laut dem Bild) und gleichzeitig noch den im 
Chip einschaltet. Also "pull_up_down" ggf. mal weglassen.

[Karl (Gast)]

Versuch es erstmal mit einem Schalter, anstatt einer Lichtschranke. 
Warum GPIO.BOTH? Was soll das time.sleep(60) in der SendMail()-Funktion?

Wenn man mit Python arbeitet, sollte man mal ein Tutorial über OOP 
lesen, das macht es übersichtlicher.

[sunshineh (Gast)]

Hallo,

danke für die Antworten. Ich habe bisher noch nichts mit Python gemacht 
und diesen Code aus verschiedenen Teilen im Internet 
zusammengeschustert. Wenn etwas schlecht daran ist, bitte ich um 
Verbesserungsvorschläge oder Tipps zu guten Tutorials.


Die 60 Sekunden hatte ich nur eingefügt, da ich nicht ständig eine 
Email-Benachrichtigungen bekommen wollte. Ich habe nun gemerkt, dass 
dies ein einer schlechten Verbindung gelegen hat und nicht am Code.

Zudem hab ich nun den Eingang der Lichtschranke mit einem Pulldown 
Widerstand belegt und nun bekomme ich (meist) wirklich nur noch eine 
Benachrichtigung, wenn der die Schranke schließt.

Aktuelle Probleme:
1. Beim Unterbrechen der Lichtschranke erhalte ich innerhalb der ersten 
Millisekunden mehrfach die Info, dass die Lichtschranke durchbrochen 
wurde.
Wie bekomme ich dieses "prellen" raus?

2. Zudem möchte ich mich dagegen absichern, wenn die Lichtschranke 
dauernd geschlossen ist (also kaputt ist oder wer was dazwischen 
stellt). Auch in diesem Fall möchte ich eine Benachrichtigung erhalten.
(....Ich habe dies ansatzweise so gemacht, dass ich in der 
Interrupt-Funktion eine while Schleife eingebaut habe, so dass man erst 
1min warten musste, und dann nochmals prüfte, dass hat aber nicht 
hingehauen)

[sunshineh (Gast)]

Das mit dem Prellen habe ich raus, habe die Bouncetime auf 2000 erhöht 
nun bräuchte ich nur noch eine Lösung für den Dauerkontakt.

[Cihan (Gast)]

sunshineh schrieb:
> nur noch eine Lösung für den Dauerkontakt
Dafür verwendet man einen Timeout oder Watchdog.
Was willst Du wann im Falle des Dauerkontaktes machen bzw. auslösen?

[Sascha W. (sascha-w)]

Hallo,

sunshineh schrieb:
> Das mit dem Prellen habe ich raus, habe die Bouncetime auf 2000 erhöht
> nun bräuchte ich nur noch eine Lösung für den Dauerkontakt.
in deiner loop(), die im übrigen sowieso ein time.sleep(1) enthalten 
sollte (schau dir mal die Prozessorauslastung an wenn dein Programm 
läuft) fragst du den Zustand des Eingangs noch mal ab. Wenn der Eingang 
aktiv ist erhöhst du einen Zähler, wenn der Eingang inaktiv ist setzt du 
den Zähler auf Null. Erreicht der Zähler z.B. 10 dann ist der Eingang 
schon 10s aktiv und du kannst dein gewünschtest Ereignis (Mail) 
auslösen.

Sascha

[sunshineh (Gast)]

Sorry, ich kriegs nicht hin. Wie muss ich meine Main-Schleife anpassen, 
damit es geht?

[sunshineh (Gast)]

Also nun bin ich schonmal soweit, dass ich in der Main die SleepTime 
eingebunden habe. Nun dachte ich mir, dass ich jedesmal dort den Zustand 
der Lichtschranke abfrage und in einer Variablen hochzähle

#!/usr/bin/env python
# -*- coding: utf-8 -*-
import RPi.GPIO as GPIO
import smtplib
import time
BtnPin = 13
Gpin   = 12
Rpin   = 11
cnt = 0

def setup():
    GPIO.setmode(GPIO.BOARD)       # Numbers GPIOs by physical location
    GPIO.setup(Gpin, GPIO.OUT)     # Set Green Led Pin mode to output
    GPIO.setup(Rpin, GPIO.OUT)     # Set Red Led Pin mode to output
    GPIO.setup(BtnPin, GPIO.IN, pull_up_down=GPIO.PUD_DOWN)    # Set BtnPin's mode is input, and pull down to low
    GPIO.add_event_detect(BtnPin, GPIO.BOTH, callback=detect, bouncetime=1000)
    
def Led(x):
    if x == 1:
        GPIO.output(Rpin, 1)
        GPIO.output(Gpin, 0)
    if x == 0:
        GPIO.output(Rpin, 0)
        GPIO.output(Gpin, 1)
def Print(x):
    if x == 0:
        print '    ***********************'
        print '    *   Button Pressed!   *'
        print '    ***********************'
def SendMail(x):
        server = smtplib.SMTP('smtp.gmail.com:587')
        server.starttls()
        server.login(username,password)
        if x == 0:      # server.sendmail(fromaddr, toaddrs, msg1)     
                print("Lichtschranke unterbrochen > Mail gesendet")
        if x == 1:      # server.sendmail(fromaddr, toaddrs, msg2)
                print("Lichtschranke immer noch unterbrochen > erneut Mail gesendet")
        server.quit()
def WriteFile():
        date = time.strftime("%x")
        zeit = time.strftime("%X2")
        fileout = open("/home/pi/Desktop/Lichtschranke.txt","a")
        time.sleep(0.1)
        fileout.write(date+";"+zeit+"; Die Lichtschranke wurde unterbrochen \n")
        time.sleep(0.1)
        fileout.close()
        print("Log-File geschrieben")
def detect(chn):
        time.sleep(0.1)
        print("in der detect ", GPIO.input(BtnPin)) 
        if GPIO.input(BtnPin) == GPIO.LOW:      # Lichtschranke wurde unterbrochen
                WriteFile()
                Led(1)
                print("detect") # SendMail(0)
        if GPIO.input(BtnPin) == GPIO.HIGH:     # Lichtschranke wieder frei
                print("wieder frei")
                Led(0)
def cntUnterbrochen(u):
        if u == 0:
                cnt = cnt + 1
                print("cnt aufaddieren = ", cnt)
        if u == 1:
                cnt = 0
                print("cnt rucksetzen = ", cnt)
def loop():
        while True:
                time.sleep(1)
                cntUnterbrochen(GPIO.input(BtnPin))
                print("main")
              
def destroy():
    GPIO.output(Gpin, GPIO.HIGH)       # Green led off
    GPIO.output(Rpin, GPIO.HIGH)       # Red led off
    GPIO.cleanup()                     # Release resource
if __name__ == '__main__':     # Program start from here
    setup()
    try:
        loop()
    except KeyboardInterrupt:  # When 'Ctrl+C' is pressed, the child program destroy() will be  executed.
        destroy()

Allerdings bekomm ich hier die Meldung
cnt = cnt + 1
UnboundLocalError: local variable "cnt" referenced before assignment

[Nosnibor (Gast)]

Er hält cnt für eine lokale Variable in cntUnterbrochen(), weil er's 
nicht besser weiß. Das gibt natürlich Ärger (uninitialisierte Variable).

"global cnt" am Anfang der Funktion klärt das Mißverständnis auf.

[sunshineh (Gast)]

Funktioniert, danke!