RASPI + Bewegungsmelder > Wiedergabe Audiodatei VLC Player

Papercraft · Feb 13th 2023

Hallo zusammen,

eines vorweg ich bin blutiger Anfänger und benötige deswegen etwas Hilfe bei folgendem Projekt.

Hardware:

Umsetzung auf dem Raspberry Pi 3 mit RPI HC-SR501 Sensor

Szene:

Beim betreten eines Raumes, soll per PIR die Wiedergabe einer Audiodatei (bspw. Hörbuch etc.) für X Minuten ausgelöst werden.

Die Wiedergabe soll bei späterem Betreten des Raumes fortgesetzt werden.

Folgende Code habe ich aktuell:

Code

import RPi.GPIO as GPIO
import time
import vlc


SENSOR_PIN = 4
GPIO.setmode(GPIO.BCM)
GPIO.setup(SENSOR_PIN, GPIO.IN)


def Play(channel):
    player = vlc.MediaPlayer("/home/pi/Desktop/script/sound.mp3")
    player.play()
    print('Es gab eine Bewegung! - Play')
    time.sleep(10) #Zeit zu Testzwecken auf 10 Sekunden gesetzt
    player.set_pause(1)
    print('Es gab seit X Minuten keine Bewegung! - Pause')


try:
    GPIO.add_event_detect(SENSOR_PIN, GPIO.RISING, callback=Play,)
    while True:
        continue
  
  
except KeyboardInterrupt:
    print ("Script wird beendet")
GPIO.cleanup()

Display More

Soweit , so gut... Es wird bei einer Bewegung also eine Datei wiedergegeben. Damit das Script aber fehlerfrei und in vollem Umfang läuft bräuchte ich bei den folgenden Punkten Hilfe:

1. Jedes mal wenn der Sensor eine weitere Bewegung registriert, wird scheinbar eine zusätzliche Instanz des Players geöffnet und die Datei wird parallel erneut abgespielt, läuft dann also doppelt, dreifach etc. im Kanon

Es wäre gut, wenn Bewegungen während der Wiedergabe ignoriert werden würden. Sodass nur eine Instanz des Players läuft

2. Wenn die Wiedergabe nach Ablauf der Zeit pausiert, soll die sie bei einer späteren Bewegung fortgesetzt werden.

Ich hoffe das ist machbar, und dass ihr mir helfen könnt

RTFM · Feb 13th 2023

Du musst erstmals die Einstellungen des VLC auf

- nur eine Instanz erlauben

- Play and Exit

- [Video Off]

abändern.

Servus !

Papercraft · Feb 13th 2023

Wenn ich die Einstellungen im Programm vornehme hat das leider keinen Effekt auf das Objekt

hyle · Feb 13th 2023

Nur so ein Gedanke... Erstelle das player Objekt mal ausserhalb der Funktion Play(). player musst Du dann natürlich der Funktion übergeben. Was sagt dann in der Funktion in der ersten Zeile print(player.get_state())?

Evtl. kann man damit den Status von player abfragen und entsprechend starten oder eben nicht.

Ich kann es leider gerade nicht selber testen.

Dennis89 · Feb 14th 2023

Hallo,

Quote from Papercraft

Sodass nur eine Instanz des Players läuft

dann solltest du, wie schon erwähnt, auch nur eine Instanz des Players erstellen. Du erstellst aber mit jeder erkannten Beweung eine neue Instanz -> Zeile 11.

Mit 'set_pause' kannst du pausieren (1) oder die Wiedergabe wieder fortzsetzen mit (0). Ich würde hier das "normale" 'pause' verwenden oder einheiltich mit 'set_pause' die Wiedergabe starten, pausieren und wieder starten.

Im Allgemeinen würde es sich anbieten anstatt 'GPIO' gpiozero zu benutzen. Damit schreibst du viel lesbareren Code und die haben schon viel Code vorbereitet. Es gibt Klassen zu Buttons, LED's, Bewegungsmelder etc. die man nur noch benutzen muss.

'as' ist zum umbennenen da, du nennst GPIO in GPIO um. Besser wäre from RPi import GPIO

'setmode', 'setup' so wie der try/except Code gehört in eine Funktion. Der darf nicht auf Modulebene (der Code ohne Einrückungen) stehen, denn das ist ausführbarere Code und der soll/darf keinen globalen Zustand haben.

Auf Modulebene werden nur Klassen, Funktionen, Konstanten (Wi dein 'SENSOR_PIN') und der Einstiegspunkt in die 'main'-Funktion definiert. Hier gehört auch dein Code rein.

Die Dauerschleife, die gar nichts macht und nur ununterbrochen durchrennt könnte man mit signal.pause() ersetzen. Dann wartet das Programm einfach bis was passiert. Oder du benutzt 'wait_for_'-Funktion der jeweiligen gpiozero-Klasse.

'GPIO.cleanup' sollte auch aufgerufen werden, wenn dein Skript beendet wird, dazu gibt es 'finally'. Auch das wird bei 'gpiozero' nicht benötigt, bzw. wird intern erledigt.

Funktionen werden nach ihrer Tätigkeit benannt und die Namen schreibt man klein_mi_unterstrich.

Der Pfad wäre auch ein Kandidat für eine Konstante und Pfade gibt man üblicherweise nicht als String an, da sie andere Eigenschaften wie Strings haben. Dafür gibt es das 'pathlib'-Modul. Wichtig ist das, wenn du mit Pfade arbeitest.

Die letzte 'print'-Ausgabe macht nicht viel Sinn, du hast keine Zeitmessung, die misst, wielange keine Bewegung mehr erkannt wurde. Wenn du die hättest gehört die auch nicht in die Funktion. Sonst würde die Funktion heißen 'play_media_and_after_that_do_meassurmet_time_until_next_detection' oder so ähnlich. Funktionen konzentrieren sich auf eine Aufgabe und werden so geschrieben, das man die eine Aufgabe auch unabhängig von all dem anderen Code testen kann.

Ungetestet:

Python

from time import sleep

import vlc
from gpiozero import MotionSensor

SENSOR_PIN = 4


def play_song(player):
    player.play()
    print("Es gab eine Bewegung! - Play")
    sleep(10)
    player.pause()


def main():
    motion_sensor = MotionSensor(SENSOR_PIN)
    player = vlc.MediaPlayer("/home/pi/Desktop/script/sound.mp3")
    while True:
        motion_sensor.wait_for_motion()
        play_song(player)


if __name__ == "__main__":
    main()

Display More

Grüße

Dennis

P.S. Wenn es zu Problemen mit dem Sonsor kommt, achte auf die Parameter die man bei 'MotionSensor' noch angeben kann. Und schau ob du damit dein Problem in den Griff bekommst. Aktuell ist alles auf den default-Werten.

Papercraft · Feb 14th 2023

Quote from Dennis89
Hallo,
dann solltest du, wie schon erwähnt, auch nur eine Instanz des Players erstellen. Du erstellst aber mit jeder erkannten Beweung eine neue Instanz -> Zeile 11.
Mit 'set_pause' kannst du pausieren (1) oder die Wiedergabe wieder fortzsetzen mit (0). Ich würde hier das "normale" 'pause' verwenden oder einheiltich mit 'set_pause' die Wiedergabe starten, pausieren und wieder starten.

Im Allgemeinen würde es sich anbieten anstatt 'GPIO' gpiozero zu benutzen. Damit schreibst du viel lesbareren Code und die haben schon viel Code vorbereitet. Es gibt Klassen zu Buttons, LED's, Bewegungsmelder etc. die man nur noch benutzen muss.

'as' ist zum umbennenen da, du nennst GPIO in GPIO um. Besser wäre from RPi import GPIO

'setmode', 'setup' so wie der try/except Code gehört in eine Funktion. Der darf nicht auf Modulebene (der Code ohne Einrückungen) stehen, denn das ist ausführbarere Code und der soll/darf keinen globalen Zustand haben.
Auf Modulebene werden nur Klassen, Funktionen, Konstanten (Wi dein 'SENSOR_PIN') und der Einstiegspunkt in die 'main'-Funktion definiert. Hier gehört auch dein Code rein.
Die Dauerschleife, die gar nichts macht und nur ununterbrochen durchrennt könnte man mit signal.pause() ersetzen. Dann wartet das Programm einfach bis was passiert. Oder du benutzt 'wait_for_'-Funktion der jeweiligen gpiozero-Klasse.
'GPIO.cleanup' sollte auch aufgerufen werden, wenn dein Skript beendet wird, dazu gibt es 'finally'. Auch das wird bei 'gpiozero' nicht benötigt, bzw. wird intern erledigt.
Funktionen werden nach ihrer Tätigkeit benannt und die Namen schreibt man klein_mi_unterstrich.
Der Pfad wäre auch ein Kandidat für eine Konstante und Pfade gibt man üblicherweise nicht als String an, da sie andere Eigenschaften wie Strings haben. Dafür gibt es das 'pathlib'-Modul. Wichtig ist das, wenn du mit Pfade arbeitest.
Die letzte 'print'-Ausgabe macht nicht viel Sinn, du hast keine Zeitmessung, die misst, wielange keine Bewegung mehr erkannt wurde. Wenn du die hättest gehört die auch nicht in die Funktion. Sonst würde die Funktion heißen 'play_media_and_after_that_do_meassurmet_time_until_next_detection' oder so ähnlich. Funktionen konzentrieren sich auf eine Aufgabe und werden so geschrieben, das man die eine Aufgabe auch unabhängig von all dem anderen Code testen kann.

Ungetestet:
Python
from time import sleep

import vlc
from gpiozero import MotionSensor

SENSOR_PIN = 4


def play_song(player):
    player.play()
    print("Es gab eine Bewegung! - Play")
    sleep(10)
    player.pause()


def main():
    motion_sensor = MotionSensor(SENSOR_PIN)
    player = vlc.MediaPlayer("/home/pi/Desktop/script/sound.mp3")
    while True:
        motion_sensor.wait_for_motion()
        play_song(player)


if __name__ == "__main__":
    main()
Display More
Grüße
Dennis

P.S. Wenn es zu Problemen mit dem Sonsor kommt, achte auf die Parameter die man bei 'MotionSensor' noch angeben kann. Und schau ob du damit dein Problem in den Griff bekommst. Aktuell ist alles auf den default-Werten.
Display More

Wow, der Code funktioniert einwandfrei

Vielen Dank, dass du dir die Zeit genommen hast mir alles so ausführlich zu erklären, jetzt ist mir alles etwas klarer... aber ich muss wohl noch sehr viel lernen

Dennis89 · Feb 14th 2023

Das freut mich.

Quote from Papercraft

aber ich muss wohl noch sehr viel lernen

Python hat ein offizielles Tutorial, es hilft wenn man das mal durchgearbeitet hat:

https://docs.python.org/3/tutorial/

Grüße

Dennis

Papercraft · May 31st 2023

Quote from Dennis89

Das freut mich.

Python hat ein offizielles Tutorial, es hilft wenn man das mal durchgearbeitet hat:
https://docs.python.org/3/tutorial/

Grüße
Dennis

Display More

Ich muss mich hier leider doch nochmal um -Hilfe bitten.

Ich habe schon vor einiger Zeit festgestellt, dass das Script zwar eigentlich richtig funktioniert, wenn allerdings das Ende des Liedes erreicht ist, wird die Wiedergabe bei erneuter registrierter Bewegung aber nicht einfach wieder vom Anfang gestartet. Stattdessen bleibt der Raspi stumm. Die Bewegungen werden weiterhin registriert, also hat sich das Script schonmal nicht aufgehängt.

Ich bekomme das leider nicht selbständig in den Griff

DeaD_EyE · May 31st 2023

Die normale MediaPlayer-Instanz fängt nicht wieder automatisch beim ersten Track an, wenn das Ende erreicht ist.
Ich habe es hier mal als Klasse implementiert. Der Code ist nicht komplett getestet.

Python

from time import sleep

from gpiozero import MotionSensor
from vlc import Media, MediaList, MediaListPlayer, PlaybackMode


class Player:
    """
    Wrapper für vlc.MediaListPlayer
    """

    def __init__(self, tracks, loop=False):
        """
        tracks := liste mit Pfaden zu den Medien (Audio/Video)
        loop := Medien in einer Endlosschleife abspielen

        >>> player = Player(["track1.mp3"])
        >>> player.play()
        """
        media_list = MediaList()

        for track in tracks:
            media_list.add_media(Media(track))

        self._player = MediaListPlayer()
        self._player.set_media_list(media_list)

        # ist jetzt für dieses Projekt wahrscheinlich unnötig
        # ich lasse es dennoch drin, vielleicht kann es jemand anderes gebrauchen
        if loop:
            self._player.set_playback_mode(PlaybackMode.loop)

    def play(self):
        return self._player.play()

    def pause(self):
        return self._player.pause()

    def stop(self):
        return self._player.stop()


SENSOR_PIN = 4


def main():
    motion_sensor = MotionSensor(SENSOR_PIN)
    media = ["/home/pi/Desktop/script/sound.mp3"]
    player = Player(media)

    while True:
        motion_sensor.wait_for_motion()
        player.play()
        sleep(10)
        player.pause()

        # oder um wieder zum Anfang zu springen und zu stopppen:
        # player.stop()


if __name__ == "__main__":
    main()

Display More

Verwende zum Testen am besten eine sehr kurze Audiodatei. 10 Sekunden sind schon genug zum Testen.

Papercraft · May 31st 2023

Muchas gracias . Die ersten Tests waren erfolgreich.

Ich teste mal etwas längere Tracks und schau mal ob es stabil läuft... Ihr seid meine Helden

Users Viewing This Thread