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 Auto-dithering #38

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

BioBox
Copy link

@BioBox BioBox commented Aug 17, 2021

The script should automatically resample the flac if it has a bit-depth higher than 16. This is what's done in the standard crawler tool for certain RecommdED sites, and I don't want to check each dir to see if it's 24-bit. I also added the mutagen dependancy because it becomes very cumbersome to get the information I need otherwise. Finally, I adjusted the help info to remove mention of metaflac which is now optional only for replay-gain with flac, because the metaflac binary should already be included with the flac package anyways.

The mutagen module is only used to read the audio tags and not to write them, as it's better to use the encoders for that. I tested this commit by transcoding my entire music collection on my phone to MP3 V0. All of the tags are accounted for and it still sounds great.

Signed-off-by: Daniel J. Perry <dperry45@gatech.edu>
Signed-off-by: Daniel J. Perry <dperry45@gatech.edu>
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.

1 participant