Posts by Dennis89

Hallo,

wieso ist in den Funktionen, die den Stand ändern sollen, nochmal ein 'event_detected'?

Bevor du das Ganze grafisch darstellst, würde ich mich erst einmal um die Logik kümmern.

Vergiss bitte gleich von Anfang an, das es globale Variablen gibt, Dieser Zustand macht ein Programm sehr unübersichtlich, schlecht wartbar und die Fehlersuche wird auch schwieriger. Wenn du dir Werte merken willst, brauchst du eine Klasse. Schau mal ob folgender Code deine Tore zählt:

from RPi import GPIO


TOR_A_PIN = 12
TOR_B_PIN = 16


class GoalCounter:
    def __init__(self):
        self.goals = {"A": 0, "B": 0}

    def add_goal(self, goal):
        self.goals[goal] += 1


def main():
    try:
        GPIO.setmode(GPIO.BCM)
        GPIO.setup(TOR_A_PIN, GPIO.IN, pull_up_down=GPIO.PUD_UP)
        GPIO.setup(TOR_B_PIN, GPIO.IN, pull_up_down=GPIO.PUD_UP)
        goal_counter = GoalCounter()
        while True:
            if GPIO.event_detected(TOR_A_PIN):
                goal_counter.add_goal("A")
            if GPIO.event_detected(TOR_B_PIN):
                goal_counter.add_goal("B")
            print(goal_counter.goals)
    finally:
        GPIO.cleanup()


if __name__ == "__main__":
    main()

Ist ungetestet, ich hoffe es läuft fehlerfrei durch.

Das ist wichtig um auch sicher zu gehen, dass dein Aufbau funktioniert.

Wenn du dir dann sicher bist, dann kannst du dich um die Grafik kümmern.

Grüße

Dennis

framp Ich weis, deswegen auch der Smily

Hallo,

framp 'Did you mean print(...)'

Grüße

Dennis

Hallo,

Quote from klaus1

SyntaxError: invalid syntax

Das sollte so aussehen:

[dennis@dennis ~]$ source test/bin/activate
(test) [dennis@dennis ~]$ python
Python 3.11.3 (main, May 24 2023, 00:00:00) [GCC 12.3.1 20230508 (Red Hat 12.3.1-1)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import ssl
>>> 

Nur eben mit 'python3' falls 'python' bei dir auf Python2 zeigt.

Grüße

Dennis

Edit: Etwas zu langsam

Du nutzt aus 'bosma' immer nur den ersten Wert aus der zurückgegebenen Liste, wieso gibst du die anderen dann zurück?

Und man muss beim Vergleiche mit einem Wahrheitswert nicht extra noch mal den Wahrheitswert hinschreiben. 'if eintrag == True' ist das gleiche wie 'if eintrag'.

So hier mal der erste ungetestete Zwischenstand:

from pathlib import Path
import matplotlib.pyplot as plt
import pickle

PRIMES_FILE = Path("/home7neuernutzer/documents/primes_up_to_10_7")
H_UNTEN = 1
H_OBER = -1

EXPONENTEN_UNTER = 2
EXPONENTEN_OBER = 13


def listsave(liste, datei):
    with open(datei, "wb") as file:
        pickle.dump(liste, file)


def load_primes_from_file(file):
    with open(file, "rb", encoding="UTF-8") as file:
        return pickle.load(file)


def jacobisymbol(a, n):
    if n == 1 or n % 2 == 0:
        return
    if a == 2:
        return 1 if n % 8 in [1, 7] else -1
    elif a == -1:
        return pow(-1, (n - 1) // 2)
    elif a == 1:
        return 1
    elif a == 0:
        return 0
    if a >= n:
        return jacobisymbol(a % n, n)
    if a % 2 == 0:
        k = 0
        while a % 2 == 0:
            k += 1
            a = a // 2
        return jacobisymbol(2, n) ** k * jacobisymbol(a, n)
    faktor = pow(-1, ((n - 1) // 2) * (a - 1) // 2)
    return faktor * jacobisymbol(n, a)


def bosma(h, k, primes):
    # print("Bosmatest für {}*2^{}+1".format(h, k))
    for d in primes:
        jacobi = jacobisymbol((h % d) * 2 ** (k % (d - 1)) + 1, d)
        exp = h * 2 ** (k - 1)
        if jacobi == 0:
            return False
        elif jacobi == -1:
            n = 2 * exp + 1
            return pow(int(d), exp, n) == n - 1
    return False


def vektor_anlegen(laenge):
    return laenge * [False]


def anzahl_ermitteln(vektor):
    ywerte = []
    anzahl = 0
    for eintrag in vektor:
        if eintrag:
            anzahl += 1
        ywerte.append(anzahl)
    return ywerte


def liste_vorbereiten(vektor):
    y_rohdaten = []
    xwerte = []
    for x_werte, _, y_roh in vektor:
        xwerte.append(x_werte)
        y_rohdaten.append(y_roh)
    ywerte = anzahl_ermitteln(y_rohdaten)
    return xwerte, ywerte


def intervall_pruefen(untergrenze, obergrenze, exponent, liste, ausgabe=False):
    if untergrenze % 2 == 0:
        untergrenze += 1
        print(f"Untergrenze auf {untergrenze} angepasst")
    if obergrenze == 0 or obergrenze >= 2:
        obergrenze = 2**exponent - 1
        print(f"Obergrenze auf {obergrenze} angepasst")
    ergebnis = []
    anzahl = (obergrenze - untergrenze) // 2
    if ausgabe:
        for h in range(untergrenze, obergrenze + 1):
            print((h - 1) // 2 / anzahl, end="\r")
            b = bosma(h, exponent, liste)
            ergebnis.append((h, exponent, b))
        return ergebnis
    # Diese Stelle wird nur erreicht, wenn ausgabe=False gesetzt wurde
    for h in range(untergrenze, obergrenze + 1, 2):
        b = bosma(h, exponent, liste)
        ergebnis.append((h, exponent, b))
    return ergebnis


def plotten(untergrenze, exponenten, primzahlen):
    for k in exponenten:
        print(f"Aktueller Exponent: {k}")
        vektor = intervall_pruefen(untergrenze, 0, k, primzahlen, True)
        x_werte, y_werte = liste_vorbereiten(vektor)
        plt.plot(x_werte, y_werte)
    plt.show()


def main():
    primzahlen = load_primes_from_file(PRIMES_FILE)
    exponenten = list(range(EXPONENTEN_UNTER, EXPONENTEN_OBER))
    plotten(H_UNTEN, exponenten, primzahlen)


if __name__ == "__main__":
    main()

Das sind jetzt keine großen Änderungen, aber doch grundlegende Themen.

Grüße

Dennis

Hallo,

'time' wird importiert aber nicht genutzt.

Auf Modulebene (der Code ohne Einrückungen) darf kein ausführbarer Code stehen. Hier werden nur Klassen, Funktionen und Konstanten definiert. Es gibt eine Ausnahme, das ist eine 'if'-Abfrage als Einstiegspunkt in die 'main'-Funktion. Darin sollte dein Programm starten und von dort werden weitere Funktionen aufgerufen, Argumente übergeben und wieder entgegen genommen.

Gewöhne dir gleich an sprechende Namen zu verwenden und keine Abkürzungen. Das ist in der Mathematik manchmal etwas schwer, aber es ist auch sehr schwer Code zu lesen, der nur aus Namen mit einem Buchstaben besteht.

Wenn du mit Dateien arbeitest solltest du immer den absoluten Pfad verwenden. Ein Pfad ist aber kein einfacher String. Den ein Pfad hat andere Eigenschaften als ein String. Erstellen kannst du den Pfad mit 'pathlib.Path'.

Wenn du eine Datei öffnest, solltest du immer das encoding angeben. Es ist nicht garantiert, das automatisch das richtige genommen wird.

Bei 'range' musst du den 'step' nicht angeben, wenn der 1 sein soll, das ist der default-Wert.

Wieso belegst du 'h_ober' mit -1 und addierst bei der Benutzung 1 dazu? Dann kannst du doch gleich 'h_ober = 0' schreiben?

'h_unter', 'h_ober', 'exponenten_unter' und 'exponenten_ober' sollten Konstanten sein.

An 'plotten' wird 'obergrenze' übergeben, aber nicht benutzt.

Strings formatiert man aktuell eigentlich mit 'f'-Strings.

Wenn die 'obergrenze' 0 ist dann kann sie nicht auch größer 2 sein. Das dritte 'if' in 'intervall_prüfen' sollte ein 'elif' sein. Das heißt, wenn das vorherige True war, dann wird 'elif' gar nicht mehr geprüft. Komischer finde ich aber, das egal ob die Obergrenze 0 oder >2 ist, die Rechnung gleich ist. Zu was dann die Unterscheidung?

Den Sinn hinter 'ausgabe' verstehe ich nicht. Wann setzt man das True? Vielleicht wäre ein sprechenderer Name hier auch hilfreicher.

'B' ist auch so ein Name, aber nicht nur das er nichts aussagt, an der Stelle wollte ich noch sagen, das man Namen klein_mit_unterstrich schreibt. Ausnahmen sind Konstanten GANZ_GROSS und Klassen in PascalCase-Schreibweise.

In 'jacobisymbol' kannst du diesen Ausdruck:

    if a == 2:
        if n % 8 == 1 or n % 8 == 7:
            return 1
        else:
            return -1

kannst du etwas zusammen rücken:

    if a == 2:
        return 1 if n % 8 in [1, 7] else -1

Dann kannst du die 'if' an manchen Stellen wieder durch 'elif' ersetzen.

'while' braucht keine Klammern.

In 'bosma' kannst du

            if pow(int(D), exp, n) == n - 1:
                return [True, D]
            return [False, D]

wieder vereinfachen

return [True, D] if pow(int(D), exp, n) == n - 1 else [False, D]

und ein 'elif' einbauen.

Es ist verwirrend wenn eine Funktion zwei Arten von Rückgabewerte hat. Eine Liste oder eine negative Zahl. Normal gibt eine Funktion immer den gleichen Datentyp zurück.

'vektor_anlegen' wird nicht benutzt. Man muss nicht alles an einen Namen binden, wenn man den Namen nur benutzt um ihn danach zum Beispiel zurück zu geben. dann kannst du gleich return laenge * [False] schreiben.

In 'liste_vorbereiten' man will Index so wenig wie möglich benutzen, weil das relativ undurchsichtig ist. Du kannst direkt über den Inhalt der Tuple iterieren:

    for x_werte, _, y_roh in vektor:
        xwerte.append(x_werte)
        y_rohdaten.append(y_roh

In 'liste_vorbereiten' würde ich die Werte nicht als Liste zurück geben, sondern einfach zwei Werte und die dann auch so entgegen nehmen und dabei kannst du dann wieder auf den Index verzichten.

Soviel mal zum Allgemeinen Code. Für mehr Details ist es mir jetzt ehrlich gesagt zu spät und den Zwischenstand von dem hier geschriebenen in Form von Code, reiche ich morgen nach

Grüße

Dennis

Hallo,

vielleicht könnte man ja noch einwerfen, dass 'time' auch Funktionen zur Messung der Prozessdauer bietet, zum Beispiel process_time

Der letzte Code von Hofei:

import math
from time import process_time


def wurzel(k):
    return k, math.sqrt(k)


def main():
    start = process_time()
    ergebnisse = []
    for nr in range(1, 5001):
        ergebnisse.append(wurzel(nr))
    print(ergebnisse)
    print(f"Dauer: {process_time()- start}")


if __name__ == "__main__":
    main()

Dauer: 0.013281221000000003

Aus Interesse der gleiche Code, nur mit list-comprehension (weil ich das wirklich gerne nutze)

import math
from time import process_time


def wurzel(k):
    return k, math.sqrt(k)


def main():
    start = process_time()
    ergebnisse = [wurzel(nr) for nr in range(1, 5001)]
    print(ergebnisse)
    print(f"Dauer: {process_time()- start}")


if __name__ == "__main__":
    main()

Dauer: 0.009870833999999995

Und dann noch der multiprocessing-Code von __blackjack__

#!/usr/bin/env python3
from multiprocessing import Pool
import math
from time import process_time


def sqrt(value):
    return value, math.sqrt(value)


def main():
    start = process_time()
    with Pool() as pool:
        results = pool.map(sqrt, range(1, 5001))
    print(results)
    print(f'Dauer: {process_time() - start}')


if __name__ == "__main__":
    main()

Dauer: 0.039969237000000005

Alles mit der gleiche Hardware, allerdings ging gerade die Sonne unter

Grüße

Dennis

neuernutzer ich kann mich dem letzten Post von Hofei nur anschließen. So habe ich auch mit Python gestartet und auf diesem Weg lerne ich so auch regelmäßig weiter

Hallo,

bitte immer genau den Code posten, der einen Fehler geworfen hat und bitte die vollständige Fehlermeldung mit dazu posten.

Ich persönlich werde aus deinem Beitrag leider nicht schlau.

Grüße

Dennis

Edit: Fast vergessen 'global' ist nicht dafür bekannt Probleme zu lösen, eher um welche zu verursachen. Zumindest auf langfristige Sicht.

Hallo,

'pyautogui' nutzt für 'locateCenterOnScreen' pyscreeze.

Diese Funktion wird verwendet und hier findet man den Speicherort des Bildes und den Aufruf von 'scrot'.

Grüße

Dennis

Hallo,

und falls du bei der 'combobox' bleiben willst:

import tkinter as tk
from tkinter import ttk


class TestWindow(tk.Frame):
    def __init__(self, master):
        tk.Frame.__init__(self, master)
        choice = ttk.Combobox(self)
        choice["values"] = list(range(5))
        choice.grid(row=0, column=0)


def main():
    root = tk.Tk()
    root.title("Nur ein Test")
    app = TestWindow(root)
    app.pack()
    app.mainloop()


if __name__ == "__main__":
    main()

Weitere Infos:

https://docs.python.org/3/libr…obox#tkinter.ttk.Combobox

Grüße

Dennis

Quote from luemar

erstaunlicherweise werden beide Funktionen get_picture(ir_light, camera und

send_mail() unmittelbar nach Aktivierung des Scripts ausgeführt, unabhängig

von der eingegebenen Zeit und das Script bricht ab mit:

Das ist nicht erstaunlich, sondern das ist genau das was du programmiert hast. Wenn der Interpreter das erste mal deinen Code durchgeht, dann sieht er einen Funktionsaufruf und ruft die Funktion auf. Ein Klammerpaar nach einem Funktionsname bedeutet, dass die Funktion aufgerufen werden soll.

Stichwort: 'partial', falls 'sheudel da nicht was anbietet um Argumente mit zu übergeben. Schau mal in die Doku dazu.

Grüße

Dennis

Hallo,

'Button' hat gar kein 'was_held' :

https://gpiozero.readthedocs.i…ble/api_input.html#button

Du solltest nicht irgendwas an der 'Button'-Klasse hinzufügen oder überschreiben.

Du benötigst das auch nicht, weil du einfach 'value' abfragen kannst, da bekommst du je nach gedrückt oder nicht eine 1 oder eine 0 ausgegeben.

Was passiert, wenn du den Button jetzt 2 Sekunden gedrückt hälst?

#!/usr/bin/python3
from gpiozero import Button
from signal import pause


def test_on(button):
    print("Button wurde gedrückt")


def main():
    button = Button(11, hold_time=2)
    button.when_held = test_on
    pause()

if __name__ == "__main__":
    main()

Grüße

Dennis

Hallo,

'wait_for_motion' funktioniert so nicht. In der Doku steht dazu:

Quote

Pause the script until the device is activated, or the timeout is
reached.

https://gpiozero.readthedocs.i…ml#motionsensor-d-sun-pir

Dein Programm wird solange angehalten, bis eine Bewegung erkannt wird. Du musst dann die Funktion 'on_moition' danach aufrufen, wenn das Programm fortgesetzt wird. Wenn du das so verwenden willst, dann muss das in eine Schleife, damit das Programm danach wieder wartet.

def main():
    motion_sensor = MotionSensor(MOTION_PIN)
    ir_light = DigitalOutputDevice(RELAIS_PIN)
    push_bullet = Pushbullet(PUSHBULLET_API_KEY)
    with picamera.PiCamera() as camera:
        try:
            while True:
                motion_sensor.wait_for_motion()
                on_motion(push_bullet, ir_light, camera)
        except KeyboardInterrupt:
            pass

Grüße

Dennis

Hallo,

dein vorherigen Code mit dem 'relais' angepasst:

#!/usr/bin/python3
from email.mime.image import MIMEImage
from email.mime.multipart import MIMEMultipart
from email.mime.text import MIMEText
from email.header import Header
from functools import partial
from gpiozero import MotionSensor, LED
import logging
from pushbullet import Pushbullet
from signal import pause
import smtplib, sys, picamera
from socket import gethostname, gaierror
from time import strftime, sleep


logging.basicConfig(filename='/home/pi/mylog1.log', filemode='a',\
                    format='%(asctime)s - %(name)s - %(levelname)s - %(message)s',\
                    level=logging.INFO)

PATH_TO_PICTURE = "/home/pi/bild1.jpg"
MOTION_PIN = 11
RELAIS_PIN = 17

MAIL_FROM = "my email address"
MAIL_TO = "my email address"
SMTP_HOST = "mein Post Ausgangsserver"
SMTP_PORT =  465
SMTP_USER = "my user name"
SMTP_PASSWORD = "my password"
MAIL_SUBJECT = "von RPI 3 Home1"
MESSAGE = "Foto von RPI 3 Home1"

# Pushbullet
API_KEY = "my access token"
PHONE_NUMBER = "my mobile number"


def get_picture(camera, relais):
    camera.rotation = 90
    try:
        relais.on()
        sleep(0.5)
        camera.capture(PATH_TO_PICTURE, resize=(740, 480))
        relais.off()
        logging.info('Foto aufgenommen')
    except picamera.exc.PiCameraError:
        print('Camera Fehler')
        logging.info('Camera Fehler')
    camera.close()

def send_mail(mime, smtp):
    with open(PATH_TO_PICTURE, 'rb') as file:
        img = MIMEImage(file.read())
        mime.attach(img)
        logging.info('Foto angehaengt')
    try:
        smtp.login(SMTP_USER, SMTP_PASSWORD)
        smtp.sendmail(MAIL_FROM, MAIL_TO, mime.as_string())
        logging.info('e-mail gesendet')
        smtp.quit()
    except gaierror:
        print('email Fehler')
        logging.info('e-mail Fehler')


def push_to_bullet(push_bullet):
    device = push_bullet.devices[0]
    try:
        device = push_bullet.push_sms(device, PHONE_NUMBER, 'RPI 3 Home1 ausgeloest')
        logging.info('sms gesendet')
    except Exception:
        print('sms Fehler')


def on_motion(mime, smtp, push_bullet, camera, now, relais):
    print(f"PIR 3 Home2 um {now} ausgeloest")
    logging.info(f"PIR 3 Home1 um {now} ausgeloest")
    get_picture(camera, relais)
    push_to_bullet(push_bullet)
    send_mail(mime, smtp)


def on_no_motion():
    print('im Schlafmodus')
    logging.info('im Schlafmodus')


def main():
    motion_sensor = MotionSensor(MOTION_PIN)
    relais = LED(RELAIS_PIN)

    mime = MIMEMultipart()
    mime['From'] = MAIL_FROM
    mime['To'] = MAIL_TO
    mime['Subject'] = Header(MAIL_SUBJECT, 'utf-8')
    mime.attach(MIMEText(MESSAGE, 'plain', 'utf-8'))

    smtp = smtplib.SMTP_SSL(SMTP_HOST, SMTP_PORT)
    push_bullet = Pushbullet(API_KEY)
    camera = picamera.PiCamera()
    now = strftime('%d.%m.%Y %H:%M:%S')

    try:
        motion_sensor.when_motion = partial(on_motion, mime,smtp, push_bullet, camera, now, relais)
        motion_sensor.when_no_motion = on_no_motion
    except KeyboardInterrupt:
        sys.exit()

    pause()
if __name__ == "__main__":
    main()

Vielleicht kannst du so erkennen, was ich gemeint habe.

Grüße

Dennis

Es ist in der Funktion 'main' definiert, aber nicht in 'get_picture'. Wie geschrieben, du musst das noch übergeben. Schau dir an, was in der 'main' alles mit 'camera' gemacht wird.

Wenn du den Code von blackjack im Terminal startest und eine Bewegung erkannt wird, dann wird im Terminal nichts ausgegeben?

Grüße

Dennis

Auf der Github-Seite gibt es ein Beispielcode:

from time import sleep
from mfrc522 import SimpleMFRC522
reader = SimpleMFRC522()

try:
    while True:
        print("Hold a tag near the reader")
        id, text = reader.read()
        print("ID: %s\nText: %s" % (id,text))
        sleep(5)
except KeyboardInterrupt:
    GPIO.cleanup()
    raise

Was passiert wenn du den ausführst und die Karte an den Reader hälst? Bekommst du eine Ausgabe?

Grüße

Dennis

Hallo,

die Fehlermeldung ist, dass das Programm unerwartet mit Strg+C abgebrochen wurde. Dass da keine Meldung kommt, kann man noch einbauen, ist aber gerade nicht so wichtig.

Hast du mal den Code, denn du da ausführst?

Grüße

Dennis

P.S. Willkommen im Forum

Hallo,

sorry für die Zwischenfrage.

__blackjack__ ich bin neulich auf 'contextlib.suppress' gestoßen und ich finde das, nach dem ich jetzt weis was das macht, lesbarer wie die Ausnahmebehandlung mit 'pass'.

Also zum Beispiel in der aktuellen 'main' könnte man ja schreiben:

        with contextlib.suppress(KeyboardInterrupt):
            motion_sensor.when_motion = partial(
                on_motion, push_bullet, ir_light, camera
            )
            motion_sensor.when_no_motion = on_no_motion
            pause()

Ich schau mir ja relativ viele Codebeispiele von dir und anderen Programmierer an, aber das ist mir noch nie aufgefallen. Wieso wird das nicht benutzt?

Dank und Grüße

Dennis

Hallo,

bitte poste immer die Fehlermeldung dazu.

In Zeile 91wird vermutlich ein Fehler geworfen, weil es in deinem Code kein 'RELAIS_PIN' gibt. Sollte es aber, du kannst dir den 'MotionSensor' als Vorbild nehmen.

Der Pin ist eine Konstante und erst in der 'main' wird dann der Sensor oder in deinem Fall das Relaise definiert. Das wird dann wiederum an die Funktion übergeben, die es benötigt.

So wie jetzt ist dein Relaise global verfügbar. So etwas möchte man nicht haben, das kann ein sehr unordentliches und undurchsichtiges Programm zur Folge haben.

Grüße

Dennis

Hallo,

das wird ja nicht die ganze Klasse sein, den 'ki' und 'kp' wird gar nicht benutzt, aber ich vermute mal dass da noch irgendwo ein PI(D)-Regler auftauchen wird und da wäre 'ki' und 'kp' gebräuchliche Bezeichnungen in der Regelungstechnik: https://en.wikipedia.org/wiki/PID_controller

Sonst würde ich aber auch keine weiteren genutzten Abkürzungen verteidigen wollen

Grüße

Dennis