Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add option to tag DR value to files #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

solstice0
Copy link

Hello magicgoose, thank you for creating this great little tool, it is definitely an upgrade over dr14 t.meter which is where I saw this repo mentioned.

These changes add the option to tag the DR value to the scanned audio files, this works for flac, mp3, ogg, opus, m4a, and anything that uses APE tags. But note that your player will need to support extended tags, deadbeef or foobar for example. quod libet has partial support (flac, opus, vorbis, APE formats.)

A dependency has been added for the mutagen library, which is also now used to read metadata (this fixes the bug a I was seeing with ffmpeg not properly reading opus tags)

Understandable if you do not want to merge but its out here if anyone finds this useful.

Todo:

Raise error on Cuesheet or formats with no tagging support (WAV)

@magicgoose
Copy link
Owner

Thank you for the interest.

I consider this info to be the most useful to know before one has access to the files in question. In such a scenario, the data inside the files (in tags) is not visible.

I'd also like to keep the dependencies to a minimum.

This is why I would prefer to leave this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants