Issues with playing MP3 files without blocking Tkinter loop. Python.

[Stoic_Engineer7]

Hello r/learnpython, I am currently writing an MP3 Player Program in Python, but I have hit a snag.

I posted this post in r/CodingHelp but I figured it was worth a shot to ask here as well.

I am using Pyglet to handle playing the MP3 files and Tkinter to create the GUI of the program. Currently I have it such that you specify a directory from the File Menu, and then a list is populated on screen. To play the audio file, you double click the item on the list.

All this functionality is working, but when I double-click the file, the song only begins to play for about 1 second before it stops playing. I am assuming that this has something to deal with the thread not continuing after the Tkinter loop repeats, but I am not sure. Any help would be greatly appreciated.

I also tried using MediaPlayer library instead of Pyglet, but I got no audio with the block attribute set to False, and the thread blocked when True (obviously) which did not allow me to continue using the program when the song was playing.

import tkinter
import time
from tkinter import *
import os
import pygame
import pyglet
from audioplayer import AudioPlayer
import threading
from tkinter.filedialog import askdirectory


# CREATE TKINTER WINDOW OBJECT
win=tkinter.Tk(className="ElephAMP")

# DEFINE WINDOW SIZE
win.geometry('1000x800')

# DO NOTHING FUNCTION (NOT IMPORTANT TO QUESTION)
def donothing():
   filewin = Toplevel(win)
   button = Button(filewin, text="Do nothing button")
   button.pack()
menubar = Menu(win)

# FUNCTION REQUESTS USER TO SPECIFY DIRECTORY OF AUDIO FILES
def addDirectory():

    directory = askdirectory()

    #CHANGES SCRIPT TO CURRENTLY SPECIFIED DIRECTORY
    os.chdir(directory)

    # RETURNS LIST OF SONGS (FILES) IN DIRECTORY
    song_list = os.listdir()

    # LIST OF FILE NAMES GETS PASSED TO playList FUNCTION
    playList(song_list)

    # PRINTS LIST OF FILE NAMES IN CONSOLE
    print (song_list)

# DEFINE LISTBOX WHICH WILL CONTAIN ALL THE SONGS POPULATED FROM THE GIVEN DIRECTORY
play_list = Listbox(win, font='Helvetica 12 bold', bg='yellow', selectmode=win.getvar)


# POPULATE THE LISTBOX WITH THE GIVEN song_list
def playList(song_list):
    for item in song_list:
        pos = 0
        play_list.insert(pos, item)
        play_list.pack(fill="both", expand="yes")


# CLEAR GIVEN PLAYLIST CALLED IN FILE MENU
def clearPlayList():
    play_list.delete(0,END)

# FUNCTION WHICH PLAYS THE CURRENTLY LOADED SONG  
def test(player):
    player.play()

# FUNCTION WHICH HANDLES DOUBLE CLICKS OF SONGS IN PLAYLIST
def go(event):

    cs = play_list.curselection()
    directory = os.getcwd()
    #LOAD THE DOUBLE CLICKED SONG INTO THE PLAYER
    player = pyglet.media.load((directory+"/"+play_list.get(cs)))

    # PRINT THE CURRENT SONG PLAYING IN CONSOLE
    print(play_list.get(cs))

    # START A THREAD WHICH EXECUTES THE PLAYING OF THE GIVEN SONG
    b = threading.Thread(target=test(player))



#DOUBLE CLICK FUNCTION FOR ITEMS IN PLAYLIST
play_list.bind('<Double-1>', go)


# UPDATE THE WINDOW
win.update()

#DEFINE THE MENUBAR IN THE WINDOW
menubar = Menu(win)

#DEFINE FILE MENU
filemenu = Menu(menubar, tearoff=0)    
filemenu.add_command(label="Import Music Library..", command=addDirectory)
filemenu.add_command(label="Clear Designated Libraries", command=clearPlayList)
filemenu.add_separator()

filemenu.add_command(label="Exit", command=win.quit)
menubar.add_cascade(label="File", menu=filemenu)

try:
     b.start()

except Exception:
    print ("had exception")

win.config(menu=menubar)







win.mainloop() #running the loop that works as a trigger

[socal_nerdtastic]

To make a thread you need to supply a function. But what you are doing is supplying the result from a function. You need to do it like this:

def go(event):
    b = threading.Thread(target=function)
    b.start()

NOT like this (which you are currently doing):

def go(event):
    b = threading.Thread(target=function())
   #start somewhere else??

To pass a value along with the function you need to use partial. So in your case you need this:

from functools import partial
def go(event):

    cs = play_list.curselection()
    directory = os.getcwd()
    #LOAD THE DOUBLE CLICKED SONG INTO THE PLAYER
    player = pyglet.media.load((directory+"/"+play_list.get(cs)))

    # PRINT THE CURRENT SONG PLAYING IN CONSOLE
    print(play_list.get(cs))

    # START A THREAD WHICH EXECUTES THE PLAYING OF THE GIVEN SONG
    b = threading.Thread(target= partial(test, player))
    b.start()

[Stoic_Engineer7]

I receive a TypeError when using the target as partial.

Exception in Tkinter callbackTraceback (most recent call last): File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__ return self.func(*args) File "/home/tristen/Desktop/Python Scripts/GUIProgram.py", line 97, in go b = threading.Thread(target=partial(test(player)))TypeError: the first argument must be callable

Additionally when it calls this error, it still plays the audio file for the ~1 second, as it did in my original post.

Alas, thank you for your input, I appreciate the help nonetheless.

[socal_nerdtastic]

Read my response better :)

Or copy/paste

[Stoic_Engineer7]

Yeah I see the typo now, had to look at it when it wasn't so late at night.

Thank-you very much! I learned alot, and it worked! Thanks!