splinterhead / youtube-music-takeaway-sorter Goto Github PK

Error when I ran it first time

extracted_title = csv_row["Song Title"].strip()
KeyError: 'Song Title'

New column headers are now Song title, Album title, Duration seconds

Love this tool. Fuck Google

This should be containerised to allow running on environments without the correct dependencies, or to avoid cluttering up the environment

Run the test suites in Github actions to make sure they are runnable on clean environments

test_mover.py is completely disabled (commented out). The tests should be re-enabled to give confidence on the mover module

Steps:

1. Have an album with some tracks with metadata, some without
2. Set the config to copy only
3. Run the sorter

Expected Result:

• The tracks with metadata are only moved once

Actual Result:

• The tracks with metadata are moved first, then they are duplicated as part of the CSV sorting

Currently the sorter assumes all files are .mp3. It should attempt to identify other filetypes (.flac, .ogg) before defaulting to .mp3

Instead of poking about in the src directory we should be able to read env vars or a config file (toml. ini)

Feature: Panic if the basic root is not set

The sorter relies on CSV data to make assumptions about the data, and is designed to let other tools set the metadata if it's incorrect.
We should integrate a MusicBrainz search as part of the sorting.
This could happen per-file on the sort, or per-album after the move

Doing it before moving would avoid double-moving in the situation of incorrect data