-
Notifications
You must be signed in to change notification settings - Fork 20
Added instructions for audio transcoding #24
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
Conversation
|
I would suggest running flac with |
|
Also it would be pretty nice to expand on multichannel files a bit because certain channel layouts will fail to encode with the current examples. |
FichteFoll
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry it took so long to review this. I started reviewing this months ago but somehow didn't end up finishing it and then it got buried in my open tabs and lack of time, which I'm now trying to catch up on.
Please separate all code blocks from text with a blank line.
For semantic line feeds, it is very rare to not have a line break after a comma and usually only for small enumerations or the likes of "Additionally,", whose proper name I forgot. Some sentences in here have odd line breaks.
| examples for qaac, flac | ||
| As said earlier, only qaac will require configuration, | ||
| and everythig else can simply be extracted wherever you like. | ||
| But, for the ease of use, we suggest adding them to PATH, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not on top of the entire guide's contents, but is "adding to PATH" explained anywhere? Or maybe we can add a glossary entry for an all-caps PATH with explanations without having to repeat or reference this all the time?
|
|
||
| ### Decoding audio with ffmpeg and piping it out | ||
|
|
||
| Basic ffmpeg usage is very simple—you just need to specify the input file, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No comma here.
| The command above will decode the source file | ||
| and save a resulting WAV file on your drive. | ||
| You can then use this WAV file to encode a FLAC or AAC file, | ||
| but there is a faster and more convinient way to do that—piping. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| but there is a faster and more convinient way to do that—piping. | |
| but there is a faster and more convenient way to do that: piping. |
| any information about the length of the file, | ||
| or that information is incorrect, | ||
| which makes it stop the encoding before the file actually ends. | ||
| With this option FLAC will ignore the header |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| With this option FLAC will ignore the header | |
| With this option, FLAC will ignore the header |
| and place the whole command after the ffmpeg command, | ||
| like this: | ||
| ``` | ||
| ffmpeg -i "input.dts" -acodec pcm_s24le -f wav -| flac -8 --ignore-chunk-sizes - -o "output.flac" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
24bit flac is padded to 32 bits per sample, which means 25% of the resulting file contain no information. Encoding dts to wav 24-bit seems reasonable, but at this point we should definitely mention this fallacy with flac. See also #2.
| opusenc --bitrate 160 --vbr --ignorelength "input.wav" "output.opus" | ||
| ``` | ||
| As you may have noticed, | ||
| opusenc CLI uses bitrate control, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think there should be a comma here?
| ``` | ||
| As you may have noticed, | ||
| opusenc CLI uses bitrate control, | ||
| insted of some kind constant quality mode, like in qaac. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| insted of some kind constant quality mode, like in qaac. | |
| instead of some kind constant quality mode, like in qaac. |
| As you may have noticed, | ||
| opusenc CLI uses bitrate control, | ||
| insted of some kind constant quality mode, like in qaac. | ||
| This doesn't mean it works in constant bitrate mode—[the target bitrate is calibrated against the internal constant quality targets |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's a really long link text. Please try to rephrase this.
| so that over a typical music collection, | ||
| something very close to the target bitrate will be achieved][opus_hydrogen]. | ||
| This makes choosing the proper target quality | ||
| a little bit funky—you will have to set different bitrate for stereo and for multichannel files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| a little bit funky—you will have to set different bitrate for stereo and for multichannel files. | |
| a little bit funky. | |
| You will have to set different bitrate for stereo and for multichannel files. |
A period here is fine.
| a little bit funky—you will have to set different bitrate for stereo and for multichannel files. | ||
| As per HydrogenAudio recommendation, | ||
| the recommended bitrate for stereo is 160 kbps, | ||
| while for 5.1 something in the neighbourhood of 320-384 kbps should be OK. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| while for 5.1 something in the neighbourhood of 320-384 kbps should be OK. | |
| while for 5.1 something in the range of 320-384 kbps should be OK. |
If the two spaces at the end of the line are intended, use <br> to add a line break instead. Those are resistent to removal of trailing whitespace.
|
i can help |
|
here #62 |
|
Superseded by #62 (which has been merged). |
I know that @LightArrowsEXE already wrote this, but his instructions are not in line with what was written on the "Preparation and Necessary Software" site (using ffmpeg flac instead of libflac), he repeated a lot of things already written in the aforementioned site, and also added a lot of information that are completely unnecessary, like explaining the difference between lossy and lossless (basic knowledge that is IMO not needed in this guide, plus it's also mentioned in "Preparation and Necessary Software") or comparing file sizes for different flac levels, or explaining what "FLAC" means, lol. I also disagree with some of his statements, which I commented in his pull request #21.