DeepFaceLab icon indicating copy to clipboard operation
DeepFaceLab copied to clipboard

change video extract default output extension from `png` to `jpg`

Open mightbesimon opened this issue 2 years ago • 2 comments

In most cases, jpg is the preferred output extension for extraction. Even the help message itself states:

png is lossless, but extraction is x10 slower for HDD, requires x10 more disk space than jpg.

I think if the odd cases that require lossless png, they should require the additional step rather than have most cases require the extra step. This will also make bulk extraction easier too. Let me know if you agree with this perspective.

mightbesimon avatar Jun 24 '23 14:06 mightbesimon

Whoops, I didn't mean to push my additional modifications (batch frame extract). Reverted back to intended.

mightbesimon avatar Jun 24 '23 17:06 mightbesimon

png doesn't have to be lossless though does it?

marshalleq avatar Jul 23 '23 02:07 marshalleq