-
Notifications
You must be signed in to change notification settings - Fork 58
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
Multi-pass encoding #6
Comments
Added in a PR for multiple passes, afaik anything beyond 2 is inconsequential, so I hard-coded that in for now in the #19 |
a rendering engine that uses more threads
We will need to rework this into the |
Yeah, should be much easier once there is an options drop-down for it that we can pull from, or whatever. |
Quite new to ffmpeg myself and encoding in general, but can someone confirm/clarify something for me: By default, when you encode in x264, ffmpeg uses a crf with a factor of 23 or so. Once you specify what bitrate you want, you can do multi-pass encoding. In that case for our new GUI we would need to incorporate a radio button to switch between those two options as they are incompatible with each other. Or a separate "bitrate"/"quality" tab where we can specify the various options for CRF and Multi-pass encoding, like: Setting for both: CRF: Multi-pass: |
Perhaps we should separate the encoder settings widget into its own file with a separate ui file, so it's easier to manage this complexity as more settings are added. |
Currently broken for any component using the Video class which doesn't properly re-open a pipe to FFmpeg during the second pass. Should be an easy fix. The bigger issue is that multipass encoding only works with specific bitrates
I separated 1 encoder pass into its own method, and tried adding a 2-pass encoding option based on @ardeaf's pull request #19 Branch here: https://github.com/djfun/audio-visualizer-python/tree/feat-2-pass-encoding It seems to work at some video bitrates (1200 with my test audio) and will produce slightly different videos, but I don't really know how to tell if I'm doing it right nor how to tell which bitrates would actually work. We have to detect that ahead-of-time. If anyone still wants this feature, please let me know :) |
Currently broken for any component using the Video class which doesn't properly re-open a pipe to FFmpeg during the second pass. Should be an easy fix. The bigger issue is that multipass encoding only works with specific bitrates
I took a swing at implementing this myself already, but it was more difficult than i anticipated.
The ability to do multi-pass encoding would help to improve the quality:filesize ratio, though.
The text was updated successfully, but these errors were encountered: