Hallo,
zu deinem Python-Code:
'as' ist zum umbenennen da. Du nennst 'GPIO' in GPIO' um. 'signal' wird importiert aber nicht genutzt.
Grundsätzlich schreibt man keinen Code von oben nach unten. Das Hauptprogramm wird in Python aus einer Funktion mit dem Namen 'main' gesteuert. Daraus werden weitere Funktionen aufgerufen, Argumente übergeben und Rückgaben entgegen genommen.
Das heißt, das auf Modulebene kein ausführbarer Code stehen darf. Hier gehört nur Code hin, der Funktionen, Klassen und Konstanten definiert.
Egal welchen Taster du drückst, jedes mal wird '23' auf high gesetzt und '5' auf low. Ist das so gewünscht? Wenn ja, dann macht man das in einer Funktion und schreibt den gleichen Code nicht zwei mal, das gilt auch für die zwei 'else'-Zeilen.
'shell=True' nur verwenden, wenn es einen driftigen Grund dazu gibt. Normalerweise gilt 'shell=False', das ist auch der default-Wert. Hierbei geht es um sicherheitstechnische Gründe, das kannst du gerne mal nachlesen. Wenn du dabei auf Programmbeispiele stoßt, schau dir die genau an, manche löschen dir unerwünscht alle Daten, wenn du die einfach kopierst und laufen lässt.
Nach dem der 'subprocess'-Aufruf durch ist dann wurde an 'ultrastar' irgendwas wie "<subprocess popen object.."gebunden. Das ist zwar nicht "None", da du aber auch keine weiteren Wahrheitswerte abfragst, ist das auch mit 'None' eher überflüssig.
Wenn du den Erfolg eines 'subprocess'-Aufruf haben willst, kannst du zum Beispiel 'ultrastar.returncode' abfragen.
In Zeile 28 'photobooh' ist nicht definiert, da fehlt das 't'.
Anstatt 'GPIO' könntest du gpiozero verwenden, das ist meiner Meinung nach einfacher/schöner und lesbarer.
Ungetestet:
#!/usr/bin/env python3
import os
from subprocess import Popen
from time import sleep
from gpiozero import LED, Button
def start_photobooth():
Popen("absoluter/pfad/start-ultrastar-stop-chromium.sh", preexec_fn=os.setsid)
def start_ultrastar():
Popen("/absoluter/pfad/start-photobooth-stop-ultrastar.sh", preexec_fn=os.setsid)
def main():
status_led = LED(23)
wait_for_button = LED(5, initial_value=True)
ultrastar_button = Button(6)
photobooth_button = Button(24)
while True:
if ultrastar_button.is_pressed:
start_ultrastar()
status_led.on()
wait_for_button.off()
elif photobooth_button.is_pressed:
start_ultrastar()
status_led.on()
wait_for_button.off()
else:
status_led.off()
wait_for_button.on()
sleep(0.5)
if __name__ == "__main__":
main()
Display More
Grüße
Dennis
Edit: ich hatte den Beitrag angefangen, bevor hyle geschrieben hat, daher der doppelte Hinweis auf 'gpiozero'. Die absoluten Pfade hatte ich vergessen. 
