added debug logs + formatted

Author: DevER-M

yami/control.py

@@ -76,28 +76,28 @@ def __init__(
         self.prev_button.grid(row=0, column=2, sticky="nsew", padx=5, pady=10)
         self.play_button.grid(row=0, column=3, sticky="nsew", padx=5, pady=10)
         self.next_button.grid(row=0, column=4, sticky="nsew", padx=5, pady=10)
+        logging.debug("initialized control bar")
 
     def play_pause(self, event=None):
         """Plays Or Pauses The Music"""
 
         if self.parent.music_list_player.get_state() == vlc.State.Playing:
             self.parent.music_list_player.pause()
-            logging.info("paused")
+            logging.debug("paused")
         else:
             self.parent.music_list_player.play()
-
-            logging.info("resumed")
+            logging.debug("resumed")
         self.update_play_button()
 
     def update_play_button(self):
         """Switches Play/Pause Icon"""
 
         if self.parent.music_list_player.get_state() == vlc.State.Playing:
             self.play_button.configure(image=self.pause_icon)
+            logging.debug("updated play button to pause")
         else:
             self.play_button.configure(image=self.play_icon)
-
-    
+            logging.debug("updated play button to play")
 
     # TRUNCATOR
     def set_music_title(self, title, artist):
@@ -107,6 +107,7 @@ def set_music_title(self, title, artist):
             truncated_title = title[: self.title_max_chars - 3] + "..."
         else:
             truncated_title = title
+        logging.debug("truncated title been set to %s", title)
         self.music_title_label.configure(
             text=truncated_title + " - " + artist.replace("/", ",")
         )

yami/cover_art.py

@@ -1,6 +1,7 @@
 """Frame For Cover Art"""
 
 import customtkinter as ctk
+import logging
 
 
 class CoverArtFrame(ctk.CTkFrame):
@@ -22,3 +23,4 @@ def __init__(self, parent):
         self.cover_art_label.grid(
             sticky="nsew",
         )
+        logging.debug("initialized covert art frame")

yami/music.py

@@ -19,7 +19,7 @@
 from .control import ControlBar
 from .cover_art import CoverArtFrame
 from .progress import BottomFrame
-from .util import GEOMETRY, TITLE, PlayerState, EVENT_INTERVAL,make_time_string
+from .util import GEOMETRY, TITLE, PlayerState, EVENT_INTERVAL, make_time_string
 
 
 ctk.set_default_color_theme("yami/data/theme.json")
@@ -49,30 +49,30 @@ def __init__(self: ctk.CTk, loop=None):
         )
 
         self.initialize_vlc()
-        
+
         # TKINTER SETUP
         self.setup_icons()
         self.setup_frames()
         self.setup_widget_packing()
 
         self.setup_keybindings()
 
-        
         self.event_manager = self.music_list_player.event_manager()
-        self.event_manager.event_attach(vlc.EventType.MediaListPlayerNextItemSet, self.change_info)
+        self.event_manager.event_attach(
+            vlc.EventType.MediaListPlayerNextItemSet, self.change_info
+        )
         self.update_loop()
         self.after(EVENT_INTERVAL, self.update)
 
     def update(self, event=None):
         if self.music_list_player.get_state() == vlc.State.Playing:
 
-            song_position=self.music.get_position()
+            song_position = self.music.get_position()
             self.bottom_frame.progress_bar.set(song_position)
-            
-            self.control_bar.playback_label.configure(text=make_time_string(
-                song_position,
-                self.music.get_length() // 1000)
-                )
+
+            self.control_bar.playback_label.configure(
+                text=make_time_string(song_position, self.music.get_length() // 1000)
+            )
         self.after(EVENT_INTERVAL, self.update)
 
     def load_and_play_song(self, index):
@@ -86,12 +86,10 @@ def load_and_play_song(self, index):
 
         logging.info("playing %s", self.get_song_title())
 
-    def change_info(self,event=None):
-        
+    def change_info(self, event=None):
+
         self.cover_art_frame.cover_art_label.configure(
-            require_redraw=True, 
-            image=self.get_album_cover(), 
-            fg_color="#121212"
+            require_redraw=True, image=self.get_album_cover(), fg_color="#121212"
         )
         self.control_bar.set_music_title(  # truncates longer titles
             self.get_song_title(),
@@ -105,7 +103,7 @@ def play_next_song(self, _event=None):
         # UPDATE SELECTION
         self.playlist_frame.song_list.selection_clear(0, tk.END)
         self.playlist_frame.song_list.select_set(self.playlist_index)
-    
+
     def play_previous(self, event=None):
         self.music_list_player.previous()
         self.change_info()
@@ -114,10 +112,12 @@ def play_previous(self, event=None):
         self.playlist_frame.song_list.select_set(self.playlist_index)
 
     def get_song_length(self) -> int:
+        logging.debug("got song length")
         return self.music.get_length()
 
     def get_song_title(self) -> str:
         media = self.music_list_player.get_media_player().get_media()
+        logging.debug("got song title")
         try:
             if media.is_parsed():
                 return media.get_meta(0)
@@ -132,6 +132,7 @@ def get_album_cover(self) -> ctk.CTkImage | None:
         try:
             media = self.music_list_player.get_media_player().get_media()
             media.parse()
+            logging.debug("got album cover")
             return ctk.CTkImage(
                 self.round_corners(
                     Image.open(
@@ -148,6 +149,7 @@ def get_album_cover(self) -> ctk.CTkImage | None:
 
     def get_song_artist(self) -> str:
         media = self.music_list_player.get_media_player().get_media()
+        logging.debug("got song artist")
         try:
             if media.is_parsed():
                 return media.get_meta(1)
@@ -162,26 +164,28 @@ def get_song_position(self) -> float:
         return self.music.get_position()
 
     def round_corners(self, image, radius) -> Image.Image:
-        '''Rounds Album Cover'''
+        """Rounds Album Cover"""
         rounded_mask = Image.new("L", image.size, 0)
         draw = ImageDraw.Draw(rounded_mask)
         draw.rounded_rectangle((0, 0) + image.size, radius, fill=255)
 
         rounded_image = Image.new("RGBA", image.size)
         rounded_image.paste(image, (0, 0), mask=rounded_mask)
+        logging.debug("rounded album cover")
 
         return rounded_image
 
     def initialize_vlc(self):
-        '''initializes and creates 
-                :param `self.music_list_player`: vlc.MediaListPlayer
-                :param `self.music`:             vlc.MediaPlayer
-                :param `self.vlc_instance`:      vlc.Instance
-                :returns: some vlc attributes
-                '''
+        """initializes and creates
+        :param `self.music_list_player`: vlc.MediaListPlayer
+        :param `self.music`:             vlc.MediaPlayer
+        :param `self.vlc_instance`:      vlc.Instance
+        :returns: some vlc attributes
+        """
         self.music_list_player: vlc.MediaListPlayer = vlc.MediaListPlayer()
         self.music: vlc.MediaPlayer = self.music_list_player.get_media_player()
         self.vlc_instance: vlc.Instance = self.music_list_player.get_instance()
+        logging.debug("initialized vlc")
 
     def setup_icons(self):
         self.play_icon = ctk.CTkImage(Image.open("yami/data/play_arrow.png"))
@@ -190,7 +194,7 @@ def setup_icons(self):
         self.next_icon = ctk.CTkImage(Image.open("yami/data/skip_next.png"))
         self.folder_icon = ctk.CTkImage(Image.open("yami/data/folder.png"))
         self.music_icon = ctk.CTkImage(Image.open("yami/data/music.png"))
-        logging.info("icons setup")
+        logging.debug("icons setup")
 
     def setup_frames(self):
         self.topbar = TopBar(self)
@@ -200,23 +204,26 @@ def setup_frames(self):
         self.cover_art_frame = CoverArtFrame(self)
 
     def setup_keybindings(self):
-        '''
+        """
         :param `<F9>`: play next
         :param `<F8>`: play previous
         :param `<Space>`:  play or pause
-        '''
+        """
 
         self.bind("<F10>", self.play_next_song)
         self.bind("<F8>", self.play_previous)
         self.bind("<F9>", self.control_bar.play_pause)
         self.bind("<space>", self.control_bar.play_pause)
+        self.bind("<Control-o>", self.topbar.choose_folder)
+        logging.debug("setup keybinds")
 
     def setup_widget_packing(self):
         self.topbar.pack(side=tk.TOP, fill=tk.X)
         self.bottom_frame.pack(side=tk.BOTTOM, fill=tk.X)
         self.control_bar.pack(side=tk.BOTTOM, fill=tk.X)
         self.playlist_frame.pack(side=tk.RIGHT)
         self.cover_art_frame.pack(side=tk.LEFT, padx=10)
+        logging.debug("widgets packed")
 
     def update_loop(self):
         self.loop.call_soon(self.loop.stop)

yami/playlist.py

@@ -2,6 +2,7 @@
 
 import tkinter as tk
 import customtkinter as ctk
+import logging
 
 
 class PlaylistFrame(ctk.CTkFrame):
@@ -34,11 +35,13 @@ def __init__(self, parent):
         self.song_list.config(yscrollcommand=self.scrollbar.set)
         self.song_list.bind("<Double-1>", self.play)
         self.song_list.bind("<Return>", self.play)
+        logging.debug("initialized playlist frame")
 
     # SELECTION CALLBACK
     def play(self, event):
         try:
             index = event.widget.curselection()[0]
+            logging.debug("selected index %s to play", index)
             self.parent.load_and_play_song(index)
-        except IndexError:
-            pass
+        except Exception as e:
+            logging.exception(e)

yami/topbar.py

@@ -54,12 +54,13 @@ def __init__(self, parent):
         self.open_folder.grid(row=0, column=1, sticky="w", pady=5, padx=10)
         self.music_downloader.grid(row=0, column=2, sticky="w", pady=5, padx=10)
         self.yami.grid(row=0, column=3, sticky="w", pady=5, padx=10)
-
-        # BINDINGS
-        self.parent.bind("<Control-o>", self.choose_folder)
+        logging.debug("initialized topbar")
 
     # FOR ADDING SONGS TO PLAYLIST
+    """TODO MAKE IT SMALLER AND SIMPLER"""
+
     def choose_folder(self, _event=None):
+
         self.parent.current_folder = filedialog.askdirectory(
             title="Select Music Folder"
         )
@@ -68,12 +69,11 @@ def choose_folder(self, _event=None):
 
         # CLEAR PLAYLIST AND LISTBOX
         self.parent.playlist_frame.song_list.delete(0, tk.END)
-        # self.parent.playlist.clear()
         self.parent.media_list: vlc.MediaList = (
             self.parent.vlc_instance.media_list_new()
         )
+
         self.parent.music_list_player.set_media_list(self.parent.media_list)
-        vlc.MediaList.add_media
 
         # FILTER MUSIC FILES
         for root, _, files in os.walk(self.parent.current_folder):
@@ -82,9 +82,10 @@ def choose_folder(self, _event=None):
             for file in music_files:
                 file_path = os.path.join(root, file)
                 media = self.parent.vlc_instance.media_new(file_path)
+                artistname, title = self.get_name_and_title_of_media(media)
                 self.parent.media_list.add_media(media)
                 self.parent.playlist_frame.song_list.insert(
-                    "end", f"• {Path(file).stem}"
+                    "end", f"• {title} - {artistname}"
                 )
         os.chdir(self.parent.current_folder)
 
@@ -97,6 +98,20 @@ def prompt_download(self):
         if song_url:
             self.parent.loop.create_task(self.download_song(song_url))
 
+    def get_name_and_title_of_media(self, media):
+        """gets song artist name and title of song"""
+
+        logging.debug("got song artist name + title of song for playlist")
+        try:
+            if media.is_parsed():
+                return media.get_meta(1), media.get_meta(0)
+            else:
+                media.parse()
+                return media.get_meta(1), media.get_meta(0)
+        except Exception as e:
+            logging.exception(e)
+            return ""
+
     async def download_song(self, song_url):
         try:
             logging.info("searching %s", song_url)

yami/util.py

@@ -13,7 +13,7 @@ class PlayerState(Enum):
     STOPPED = 3
 
 
-def make_time_string(song_position,song_length):
+def make_time_string(song_position, song_length):
     curtime = song_position * song_length
 
     cur_minutes = int(curtime // 60)
@@ -23,4 +23,3 @@ def make_time_string(song_position,song_length):
     song_sec = int(song_length % 60)
 
     return f"{cur_minutes:02d}:{cur_seconds:02d} / {song_min:02d}:{song_sec:02d}"
-