Das deutsche Python-Forum

Sirius3 hat geschrieben: ↑Donnerstag 26. Oktober 2023, 13:15
Wahrscheinlich ist Pin 18 wirklich der einzige Pin, an dem ein high-Signal anliegt. Wie hast Du denn Deine Taster verschaltet? Hast Du die passenden Pull-Down-Widerstände?

hyle hat geschrieben: ↑Freitag 27. Oktober 2023, 21:04 Sind für die GPIOs hardwareseitig Pulldown-Widerstände verbaut? Im Skript sind die internen Pulldowns jedenfalls nicht aktiviert. Also nur um diese mögliche Fehlerquelle auszuschließen. Davon abgesehen ist es sinnvoller gegen GND zu schalten, Pullup zu verwenden und auf fallende Flanke zu reagieren. Dieser Weg ist wesentlich zuverlässiger als andersherum.

Um das auch noch anzusprechen: gpiozero ist wesentlich angenehmer und einfacher als RPi.GPIO.

__deets__ hat geschrieben: ↑Freitag 3. November 2023, 11:58 Ich halte die Pulldowns für einen roten Hering.

Einige Beispiele für verschiedene Baugruppen eines modularen Systems:
Zentralbaugruppe (Sitz des Hauptprozessors)
Anschaltungen: Baugruppen zum Anbinden eines Feldbus-Systems
Busmodule (Schraub- oder Crimpklemmen, Alarmierung)
Digitale Eingangskarte (meist Potentialtrennung mittels Optokoppler)
Digitale Ausgangskarte (offener Kollektor oder Relais)
Analoge Eingangskarte
Analoge Ausgangskarte

Sirius3 hat geschrieben: ↑Donnerstag 26. Oktober 2023, 13:15 Kommentare sollten einen Mehrwert bieten, ein "Import der Bobliotheken" for import-Zeilen oder ein "Cleanup" vor dem cleanup-Aufruf sind nicht sehr hilfreich.
`as` bei Import ist dazu da, dass man Namen beim Importieren umbenennen kann, wie `import tkinter as tk` um einen schönen kurzen Namen zu haben, oder `from datetime import datetime as DateTime`, um sich an die Namenskonvention zu halten, Du aber importierst GPIO und nennst es in GPIO um. Das `as` ist also unsinnig, korrekt wäre `from RPi import GPIO as gpio` um sich an die Namenskonvention zu halten.
subprocess.call mit shell=True benutzt man nicht. Kopierten Code möchte man auch nicht haben, statt dessen möchte man Schleifen benutzen.
Man benutzt keine kryptischen Nummern, sondern definiert sich die PIN-Nummern als Konstanten am Anfang der Datei, und man schreibt keine literalen Strings wie Dateinamen mitten in den Code, sondern zieht diese Information auch als Konstanten an den Anfang der Datei. Konstanten schreibt man komplett GROSS.
Um Pins zu überwachen, benutzt man die entpsrechenden Methoden und schreib sich nicht selbst eine Busy-Loop.
cleanup muß verläßlich innerhalb eines finally-Blocks aufgerufen werden:

Code: Alles auswählen

import subprocess
from queue import Queue
from RPi import GPIO as gpio

PIN_TO_VIDEO = {
    12: "/home/XYZ/Desktop/XYZ/XYZ1.mp4",
    16: "/home/XYZ/Desktop/XYZ/XYZ2.mp4",
    18: "/home/XYZ/Desktop/XYZ/XYZ3.mp4",
}
VLC_COMMAND = ["vlc", "--fullscreen", "--no-video-title-show"]

def initialize():
    queue = Queue()
    gpio.setmode(gpio.BOARD)
    for pin in PIN_TO_VIDEO:
        gpio.setup(pin, gpio.IN)
        gpio.add_event_detect(pin, gpio.RISING, callback=queue.put, bouncetime=200)
    return queue

def main():
    try:
        queue = initialize()
        while True:
            pin = queue.get()
            subprocess.call([*VLC_COMMAND, PIN_TO_VIDEO[pin]])
    finally:
        gpio.cleanup()

if __name__ == "__main__":
    main()

Wahrscheinlich ist Pin 18 wirklich der einzige Pin, an dem ein high-Signal anliegt. Wie hast Du denn Deine Taster verschaltet? Hast Du die passenden Pull-Down-Widerstände?