HaikuArchives / ffmpegGUI

GUI for FFmpeg
MIT License
23 stars 10 forks source link

Cropping issues #46

Closed humdingerb closed 1 year ago

humdingerb commented 1 year ago

Did some crop testing with a file Test.mp4, resolution 608 x 1080, always cropping 200 pixels.

andimachovec commented 1 year ago

The code to generate the cropping commandline options (in ffguiwin::BuildLine()) probably needs to be reworked completely. It`s even stated in a comment above it. At the moment it stacks several filters upon each other, I´m not really surprised this doesn´t work correctly ;-)

andimachovec commented 1 year ago

@humdingerb : Could you check out my fix_cropping branch and test if it works with your examples?

humdingerb commented 1 year ago

Yep, seems to work fine!

andimachovec commented 1 year ago

Great, happy to hear that.

andimachovec commented 1 year ago

is fixed and can be closed