.vscode/settings.json

@@ -24,7 +24,6 @@
         "encyclopaedia",
         "ENDC",
         "Gitea",
-        "isrc",
         "levenshtein",
         "metallum",
         "musify",

development/actual_donwload.py

@@ -6,8 +6,8 @@ logging.getLogger().setLevel(logging.DEBUG)
 
 if __name__ == "__main__":
     commands = [
-        "s: #a Crystal F",
+        "s: #a Psychonaut 4",
-        "d: 20",
+        "d: 0",
     ]
 
     

music_kraken/audio/metadata.py

@@ -112,6 +112,8 @@ def write_metadata_to_target(metadata: Metadata, target: Target, song: Song):
             USLT(encoding=3, lang=u'eng', desc=u'desc', text=uslt_val)
         )
 
+        mutagen_file = mutagen.File(target.file_path)
+
     id3_object.add_metadata(metadata)
     id3_object.save()
 

music_kraken/objects/parents.py

@@ -301,33 +301,6 @@ class OuterProxy:
 
         return r
 
-    @property
-    def root_collections(self) -> List[Collection]:
-        if len(self.UPWARDS_COLLECTION_STRING_ATTRIBUTES) == 0:
-            return [self]
-
-        r = []
-        for collection_string_attribute in self.UPWARDS_COLLECTION_STRING_ATTRIBUTES:
-            r.extend(self.__getattribute__(collection_string_attribute))
-
-        return r
-
-    def _compile(self, **kwargs):
-        pass
-
-    def compile(self, from_root=False, **kwargs):
-        # compile from the root
-        if not from_root:
-            for c in self.root_collections:
-                c.compile(from_root=True, **kwargs)
-            return
-
-        self._compile(**kwargs)
-
-        for c_attribute in self.DOWNWARDS_COLLECTION_STRING_ATTRIBUTES:
-            for c in self.__getattribute__(c_attribute):
-                c.compile(from_root=True, **kwargs)
-
     TITEL = "id"
     @property
     def title_string(self) -> str:

music_kraken/objects/song.py

@@ -376,25 +376,6 @@ class Album(Base):
             r += f" with {len(self.song_collection)} songs"
         return r
 
-    def _compile(self):
-        self.analyze_implied_album_type()
-        self.update_tracksort()
-
-    def analyze_implied_album_type(self):
-        # if the song collection has only one song, it is reasonable to assume that it is a single
-        if len(self.song_collection) == 1:
-            self.album_type = AlbumType.SINGLE
-            return
-
-        # if the album already has an album type, we don't need to do anything
-        if self.album_type is not AlbumType.OTHER:
-            return
-
-        # for information on EP's I looked at https://www.reddit.com/r/WeAreTheMusicMakers/comments/a354ql/whats_the_cutoff_length_between_ep_and_album/
-        if len(self.song_collection) < 9:
-            self.album_type = AlbumType.EP
-            return
-
     def update_tracksort(self):
         """
         This updates the tracksort attributes, of the songs in
@@ -544,9 +525,6 @@ class Artist(Base):
             self.label_collection.extend(object_list)
             return
 
-    def _compile(self):
-        self.update_albumsort()
-
     def update_albumsort(self):
         """
         This updates the albumsort attributes, of the albums in
@@ -557,6 +535,9 @@ class Artist(Base):
 
         :return:
         """
+        if len(self.main_album_collection) <= 0:
+            return
+
         type_section: Dict[AlbumType, int] = defaultdict(lambda: 2, {
             AlbumType.OTHER: 0,  # if I don't know it, I add it to the first section
             AlbumType.STUDIO_ALBUM: 0,
@@ -599,7 +580,7 @@ class Artist(Base):
             album_list.extend(sections[section_index])
 
         # replace the old collection with the new one
-        self.main_album_collection._data = album_list
+        self.main_album_collection: Collection = Collection(data=album_list, element_type=Album)
 
     INDEX_DEPENDS_ON = ("name", "source_collection", "contact_collection")
     @property

music_kraken/objects/target.py

@@ -1,7 +1,7 @@
 from __future__ import annotations
 
 from pathlib import Path
-from typing import List, Tuple, TextIO, Union, Optional
+from typing import List, Tuple, TextIO, Union
 import logging
 import random
 import requests
@@ -31,10 +31,7 @@ class Target(OuterProxy):
     }
 
     @classmethod
-    def temp(cls, name: str = str(random.randint(0, HIGHEST_ID)), file_extension: Optional[str] = None) -> P:
+    def temp(cls, name: str = str(random.randint(0, HIGHEST_ID))) -> P:
-        if file_extension is not None:
-            name = f"{name}.{file_extension}"
-
         return cls(main_settings["temp_directory"] / name)
 
     # This is automatically generated

music_kraken/pages/abstract.py

@@ -361,7 +361,6 @@ class Page:
         return download_result
 
     def _download_song(self, song: Song, naming_dict: NamingDict):
-        song.compile()
         if "genre" not in naming_dict and song.genre is not None:
             naming_dict["genre"] = song.genre
 
@@ -382,7 +381,7 @@ class Page:
             song.target_collection.append(new_target)
 
         r = DownloadResult(1)
-        temp_target: Target = Target.temp(file_extension=main_settings["audio_format"])
+        temp_target: Target = Target.temp()
 
         found_on_disc = False
         target: Target

music_kraken/pages/youtube_music/super_youtube.py

@@ -145,8 +145,6 @@ class SuperYouTube(Page):
         _sponsorblock_connection: Connection = Connection()
         self.sponsorblock = python_sponsorblock.SponsorBlock(silent=True, session=_sponsorblock_connection.session)
 
-        super().__init__(*args, **kwargs)
-
     def get_source_type(self, source: Source) -> Optional[Type[DatabaseObject]]:
         _url_type = {
             YouTubeUrlType.CHANNEL: Artist,

music_kraken/pages/youtube_music/youtube_music.py

@@ -193,7 +193,8 @@ class YoutubeMusic(SuperYouTube):
 
         self.start_millis = get_current_millis()
 
-        self._fetch_from_main_page()
+        if self.credentials.api_key == "" or DEBUG_YOUTUBE_INITIALIZING:
+            self._fetch_from_main_page()
 
         SuperYouTube.__init__(self, *args, **kwargs)
 
@@ -214,8 +215,6 @@ class YoutubeMusic(SuperYouTube):
         self.download_values_by_url: dict = {}
         self.not_download: Dict[str, DownloadError] = {}
 
-        super().__init__(*args, **kwargs)
-
     def _fetch_from_main_page(self):
         """
         ===API=KEY===
@@ -737,9 +736,8 @@ class YoutubeMusic(SuperYouTube):
                 raw_headers=True,
                 disable_cache=True,
                 headers=media.get("headers", {}),
-                chunk_size=main_settings["chunk_size"],
+                # chunk_size=media.get("chunk_size", main_settings["chunk_size"]),
                 method="GET",
-                timeout=5,
             )
         else:
             result = DownloadResult(error_message=str(media.get("error") or self.not_download[source.hash_url]))

pyproject.toml

@@ -69,7 +69,7 @@ dependencies = [
     "toml~=0.10.2",
     "typing_extensions~=4.7.1",
 
-    "python-sponsorblock~=0.1",
+    "python-sponsorblock~=0.1.dev1",
     "youtube_dl",
 ]
 dynamic = [