Tracebackfehler, kann jemand helfen?

andy.z · May 28th 2023

Hallo,

ich hab mir letztes Jahr ein Phytonprogramm zur Steuerung der Gartenbewässerung geschrieben.

Leider mach ich seit dem nichts mehr mit Phyton.

Jetzt habe ich aber folgendes Problem:

Ich bekomme bei Start mit Thonny folfgende Fehler angezeigt:

Python 3.9.2 (/usr/bin/python3)

>>> %Run gartenbewaesserung_final.py

Traceback (most recent call last):

File "/usr/lib/python3.9/ast.py", line 50, in parse

return compile(source, filename, mode, flags,

File "/home/pi/Gartenbewässerung/gartenbewaesserung_final.py", line 32

schedule.every().day.at("08:00").do(bewaessern_an, REL_5)

^

IndentationError: unindent does not match any outer indentation level

Hier das Phytonfile:

#!/usr/bin/env python3

from gpiozero import LED

from time import sleep

import schedule

import time

def bewaessern_an(relais):

relais.on()

def bewaessern_aus(relais):

relais.off()

def main():

REL_1=LED(12) # Rasenbewässerung 1, beim Wintergarten/Rasenmäher

REL_2=LED(13) # Rasenbewässerung 2, beim Bambus/Rosen

REL_5=LED(21) # Bewässerung 3, Einfahrt

# schedule.every().day.at("22:00").do(bewaessern_an, REL_1)

# schedule.every().day.at("22:05").do(bewaessern_aus, REL_1)

# schedule.every().day.at("22:30").do(bewaessern_an, REL_2)

# schedule.every().day.at("22:35").do(bewaessern_aus, REL_2)

# schedule.every().day.at("03:00").do(bewaessern_an, REL_1)

# schedule.every().day.at("03:05").do(bewaessern_aus, REL_1)

# schedule.every().day.at("03:30").do(bewaessern_an, REL_2)

# schedule.every().day.at("03:35").do(bewaessern_aus, REL_2)

# schedule.every().day.at("04:00").do(bewaessern_an, REL_1)

# schedule.every().day.at("04:05").do(bewaessern_aus, REL_1)

# schedule.every().day.at("04:30").do(bewaessern_an, REL_2)

# schedule.every().day.at("04:35").do(bewaessern_aus, REL_2)

schedule.every().day.at("08:00").do(bewaessern_an, REL_5)

schedule.every().day.at("08:05").do(bewaessern_aus, REL_5)

schedule.every().day.at("10:00").do(bewaessern_an, REL_5)

schedule.every().day.at("10:05").do(bewaessern_aus, REL_5)

schedule.every().day.at("12:00").do(bewaessern_an, REL_5)

schedule.every().day.at("12:05").do(bewaessern_aus, REL_5)

schedule.every().day.at("14:00").do(bewaessern_an, REL_5)

schedule.every().day.at("14:05").do(bewaessern_aus, REL_5)

schedule.every().day.at("18:00").do(bewaessern_an, REL_5)

schedule.every().day.at("18:05").do(bewaessern_aus, REL_5)

schedule.every().day.at("22:00").do(bewaessern_an, REL_5)

schedule.every().day.at("22:05").do(bewaessern_aus, REL_5)

while True:

schedule.run_pending()

time.sleep(1)

if __name__ == "__main__":

main()

Hat jemand eine Ahnung wo's hängt?

LG

Andreas

bennetr · May 28th 2023

So können wir dir leider nicht helfen, kopiere den bitte noch einmal, und zwar in einem Codeblock, damit die Einrückung beibehalten wird. Und packe den Stacktrace am besten auch in einen Codeblock.

Aber was man schonmal sagen kann, dass wahrscheinlich in Zeile 32 was mit der Einrückung falsch ist

Gnom · May 28th 2023

Hast du die Fehlermeldung überhaupt gelesen? Da steht doch Eindrückungsfehler in Zeile 32.

File "/home/pi/Gartenbewässerung/gartenbewaesserung_final.py", line 32

schedule.every().day.at("08:00").do(bewaessern_an, REL_5)

^

IndentationError: unindent does not match any outer indentation level

Die Zeile und vermutlich alle folgenden sind wohl falsch eingerückt.

keepfear · May 28th 2023

Ich hab mal das LED durch DigitalOutputDevice geändert und deinen Variablen neue Namen verpasst.

Python

#!/usr/bin/env python3
from gpiozero import DigitalOutputDevice
from time import sleep
import schedule


def bewaessern_an(relais):
    relais.on()


def bewaessern_aus(relais):
    relais.off()


def main():
    wintergarten = DigitalOutputDevice(12) 
    bambus_rosen = DigitalOutputDevice(13) 
    einfahrt = DigitalOutputDevice(21) 
    
    schedule.every().day.at("22:00").do(bewaessern_an, wintergarten)
    schedule.every().day.at("22:05").do(bewaessern_aus, wintergarten)
    schedule.every().day.at("22:30").do(bewaessern_an, bambus_rosen)
    schedule.every().day.at("22:35").do(bewaessern_aus, bambus_rosen)
    schedule.every().day.at("03:00").do(bewaessern_an, wintergarten)
    schedule.every().day.at("03:05").do(bewaessern_aus, wintergarten)
    schedule.every().day.at("03:30").do(bewaessern_an, bambus_rosen)
    schedule.every().day.at("03:35").do(bewaessern_aus, bambus_rosen)
    schedule.every().day.at("04:00").do(bewaessern_an, wintergarten)
    schedule.every().day.at("04:05").do(bewaessern_aus, wintergarten)
    schedule.every().day.at("04:30").do(bewaessern_an, bambus_rosen)
    schedule.every().day.at("04:35").do(bewaessern_aus, bambus_rosen)
    schedule.every().day.at("08:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("08:05").do(bewaessern_aus, einfahrt)
    schedule.every().day.at("10:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("10:05").do(bewaessern_aus, einfahrt)
    schedule.every().day.at("12:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("12:05").do(bewaessern_aus, einfahrt)
    schedule.every().day.at("14:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("14:05").do(bewaessern_aus, einfahrt)
    schedule.every().day.at("18:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("18:05").do(bewaessern_aus, einfahrt)
    schedule.every().day.at("22:00").do(bewaessern_an, einfahrt)
    schedule.every().day.at("22:05").do(bewaessern_aus, einfahrt)

    while True:
        schedule.run_pending()
        sleep(1)

if __name__ == "__main__":
    main()

Display More

DeaD_EyE · May 28th 2023

Wahrscheinlich hast du beim Bearbeiten die Einrückung falsch gemacht.

Das Problem, dass ich aus 10 Meter Entfernung sehe, ist die Wiederholung von schedule.every().day.at(....).do(..., ...).

Das macht eine nachträgliche Änderung unheimlich nervig, da man immer wieder das Gleiche wiederholen muss und Copy&Paste ist keine Lösung.

Hier eine verbesserte Variante, wo das nachträgliche Bearbeiten nicht ganz so schiefgehen kann.

Python

#!/usr/bin/env python3

from time import sleep

import schedule
from gpiozero import DigitalOutputDevice


def bewaessern_an(relais):
    relais.on()


def bewaessern_aus(relais):
    relais.off()


def main():
    wintergarten = DigitalOutputDevice(12)
    bambus_rosen = DigitalOutputDevice(13)
    einfahrt = DigitalOutputDevice(21)

    aufgaben = [
        ("22:00", bewaessern_an, wintergarten),
        ("22:05", bewaessern_aus, wintergarten),
        ("22:30", bewaessern_an, bambus_rosen),
        ("22:35", bewaessern_aus, bambus_rosen),
        ("03:00", bewaessern_an, wintergarten),
        ("03:05", bewaessern_aus, wintergarten),
        ("03:30", bewaessern_an, bambus_rosen),
        ("03:35", bewaessern_aus, bambus_rosen),
        ("04:00", bewaessern_an, wintergarten),
        ("04:05", bewaessern_aus, wintergarten),
        ("04:30", bewaessern_an, bambus_rosen),
        ("04:35", bewaessern_aus, bambus_rosen),
        ("08:00", bewaessern_an, einfahrt),
        ("08:05", bewaessern_aus, einfahrt),
        ("10:00", bewaessern_an, einfahrt),
        ("10:05", bewaessern_aus, einfahrt),
        ("12:00", bewaessern_an, einfahrt),
        ("12:05", bewaessern_aus, einfahrt),
        ("14:00", bewaessern_an, einfahrt),
        ("14:05", bewaessern_aus, einfahrt),
        ("18:00", bewaessern_an, einfahrt),
        ("18:05", bewaessern_aus, einfahrt),
        ("22:00", bewaessern_an, einfahrt),
        ("22:05", bewaessern_aus, einfahrt),
    ]

    for uhrzeit, funktion, ort in aufgaben:
        schedule.every().day.at(uhrzeit).do(funktion, ort)

    while True:
        schedule.run_pending()
        sleep(1)


if __name__ == "__main__":
    main()

Display More

Eine weitere Verbesserung wäre es, wenn man Aufgaben mit Startzeit, Dauer, Einschalt-Funktion und Ausschaltfunktion erstellt.

Das würde die Liste um die Hälfte reduzieren. Ich will es aber nicht komplizierter machen, als e schon ist.

Gnom · May 28th 2023

Wenn es ein Problem ist, dass (!!!) du aus 10 m Entfernung siehst, dann setz dich doch näher an den Bildschirm!

__blackjack__ · May 28th 2023

Wobei man Funktion und Ort noch leicht zusammenfassen kann, denn die beiden Funktionen machen ja nicht wirklich etwas ausser `on()`/`off()` aufzurufen:

Python

#!/usr/bin/env python3
from time import sleep

import schedule
from gpiozero import DigitalOutputDevice


def main():
    wintergarten = DigitalOutputDevice(12)
    bambus_rosen = DigitalOutputDevice(13)
    einfahrt = DigitalOutputDevice(21)

    for uhrzeit, funktion in [
        ("22:00", wintergarten.on),
        ("22:05", wintergarten.off),
        ("22:30", bambus_rosen.on),
        ("22:35", bambus_rosen.off),
        ("03:00", wintergarten.on),
        ("03:05", wintergarten.off),
        ("03:30", bambus_rosen.on),
        ("03:35", bambus_rosen.off),
        ("04:00", wintergarten.on),
        ("04:05", wintergarten.off),
        ("04:30", bambus_rosen.on),
        ("04:35", bambus_rosen.off),
        ("08:00", einfahrt.on),
        ("08:05", einfahrt.off),
        ("10:00", einfahrt.on),
        ("10:05", einfahrt.off),
        ("12:00", einfahrt.on),
        ("12:05", einfahrt.off),
        ("14:00", einfahrt.on),
        ("14:05", einfahrt.off),
        ("18:00", einfahrt.on),
        ("18:05", einfahrt.off),
        ("22:00", einfahrt.on),
        ("22:05", einfahrt.off),
    ]:
        schedule.every().day.at(uhrzeit).do(funktion)

    while True:
        schedule.run_pending()
        sleep(1)


if __name__ == "__main__":
    main()

Display More

hyle · May 28th 2023

Und ich würde zudem auch nach Orten, statt nach Zeit sortieren (wenn nicht sogar für jeden Ort einen eigene Zeitliste erstellen). So wäre es für mich zumindest übersichtlicher und vor allem kann man ohne groß suchen zu müssen, einen Ort raus nehmen oder einen neuen Ort hizufügen.

__blackjack__ · May 28th 2023

Wenn ich das richtig sehe sind das auch immer on/off-Paare die genau 5 Minuten auseinander liegen. Das heisst eigentlich bräuchte man auch nur die Hälfte der Information in der Liste, weil man aus den on-Zeiten die off-Zeiten berechnen kann.

hyle · May 28th 2023

Quote from __blackjack__

Das heisst eigentlich bräuchte man auch nur die Hälfte der Information in der Liste, weil man aus den on-Zeiten die off-Zeiten berechnen kann.

Genau den Gedanken hatte ich vorhin auch, aber vergessen zu schreiben.

__blackjack__ · May 28th 2023

Ungetestet:

Python

#!/usr/bin/env python3
from datetime import datetime as DateTime, timedelta as TimeDelta
from time import sleep

import schedule
from gpiozero import DigitalOutputDevice


def add_minutes(time_text, minutes):
    return format(
        DateTime.strptime(time_text, "%H:%M") + TimeDelta(minutes=minutes),
        "%H:%M",
    )


def main():
    wintergarten = DigitalOutputDevice(12)
    bambus_rosen = DigitalOutputDevice(13)
    einfahrt = DigitalOutputDevice(21)

    for irrigation, start_times in [
        (wintergarten, ["22:00", "04:00", "03:00"]),
        (bambus_rosen, ["22:30", "03:30", "04:30"]),
        (einfahrt, ["08:00", "10:00", "12:00", "14:00", "18:00", "22:00"]),
    ]:
        for start_time in start_times:
            end_time = add_minutes(start_time, 5)
            schedule.every().day.at(start_time).do(irrigation.on)
            schedule.every().day.at(end_time).do(irrigation.off)

    while True:
        schedule.run_pending()
        sleep(1)


if __name__ == "__main__":
    main()

Display More

hyle · May 28th 2023

Ich will ja nicht meckern, aber bin ein bissel hin- und hergerissen. Einerseits finde ich es echt großartig (!!!), was in diesem Thread aus einem simplen Einrückungsfehler entstanden/gewachsen ist, aber andererseits bin ich nicht sicher ob ihr nicht viel zu gutmütig für diese (digitale) Welt seid!

SCNR!

andy.z · May 31st 2023

Vielen Dank für die großartige Unterstützung.

Wie geschrieben, hab ich das Programm letztes Jahr mit sehr viel externer Hilfe geschrieben.

Was der Fehlercode bedeutet, hat sich mir leider nicht erschlossen - egal wie nah oder fern ich vorm Monitor gesessen bin.

Aber ja, es war der Einrückungsfehler, der kam dadurch zustande, da ich im Winter die Rasenbewässerung (REL 1 und REL 2) mit "#"deaktiviert habe.

Beim löschen von "#" kams dann wohl zu dem Einrückungsfehler.

Falls es da jemand gibt, der mir helfen kann und mag, ich hätte da eigentlich noch einiges zu verbessern an dem Programm.

Am Pi4 ist noch ein kapazitiver Feuchtigkeitsmesser angeschlossen und im Garten verbuddelt, der eigentlich mit integriert werden sollte - so nach dem Motto, ist der Rasen zu feucht, braucht nicht gesprengt werden und ein Außentemperaturmesser ist auch angeschlossen, quasi als Frostmelder, ab wann sich die Bewässerung durch das öffnen der Ventile entleeren soll.

Die Werte des Feuchtigkeitssensors, sowie die Temperatur kann ich aus seperaten Programm schon auslesen, es hapert halt an den Verknüpfungen ....

SG

Andreas

Users Viewing This Thread