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
Option to skip the conversion step #168
Comments
@Rubecks, yeah, we can probably do this. I'll have to do some research into which tools are available for embedding metadata of different audio formats. I think, so far, Mutagen – https://github.com/quodlibet/mutagen should suffice for as many formats as freyr needs. But since that's written in Python, we need to get #21 merged to aid bridging node and python. |
Yes indeed, this feature would be more than nice. |
Hi! I'm a new user, and notice something odd in the conversion process. The download file is This is the command I used: freyr get -b 256 https://open.spotify.com/track/5r5cp9IpziiIsR6b93vcnQ?si=295f54a7150349c2 |
Ignore my comment, right after I came across #625. It was related to atomicParsley. Bumping the version that solved the issue.
|
Converting a lossy file to another lossy format is a bad practice, the audio quality reduces 2 times (YouTube converts the upload on their side, and freyr converts it again), 3 times if the uploader sent a lossy audio too.
Sure, converting to AAC 320kbps is probably ok for YouTube's audio quality, but then you are creating bigger files for no reason. One of the songs I used to test it was 3.94 MB when downloaded, and 9.48 MB after conversion. Trying to reduce the bitrate to match the source size would be gambling with the audio quality too, since the same bitrate with different formats gives different results; a 128 kbps opus has higher quality than a 128 kbps aac.
Could you add an option to keep the original file, probably opus, and do everything else (tagging, organizing, etc...)?
The text was updated successfully, but these errors were encountered: