[Kdenlive-devel] Fwd: [Kdenlive 0002235]: Crop applies before any other effect

[Dan Dennedy]

Thought this new comment was interesting and might get overlooked.
simple change.

---------- Forwarded message ----------
From: Mantis Bug Tracker <webmaster at kdenlive.org>
Date: Wed, Jul 18, 2012 at 10:31 AM
Subject: [Kdenlive 0002235]: Crop applies before any other effect
To: dan at dennedy.org



A NOTE has been added to this issue.
======================================================================
http://www.kdenlive.org/mantis/view.php?id=2235
======================================================================
Reported By:                eSyr
Assigned To:
======================================================================
Project:                    Kdenlive
Issue ID:                   2235
Category:                   Effects
Reproducibility:            always
Severity:                   minor
Priority:                   low
Status:                     new
Build/Install Method:       Distribution package
======================================================================
Date Submitted:             2011-07-12 20:52 CEST
Last Modified:              2012-07-18 19:31 CEST
======================================================================
Summary:                    Crop applies before any other effect
Description:
Crop applies before any other effect

Steps to Reproduce:
 * Add several effects, like "Rotate and Shear" or "Corners"
 * Add Crop effect after them
======================================================================

----------------------------------------------------------------------
 (0007170) ttill (developer) - 2011-08-10 15:47
 http://www.kdenlive.org/mantis/view.php?id=2235#c7170
----------------------------------------------------------------------
Yes.
The MLT maintainer (Dan) wants it to be that way because it is also used for
some internal stuff.
Maybe another Crop filter should be created and the current one then hidden.

----------------------------------------------------------------------
 (0007183) ddennedy (developer) - 2011-08-10 17:37
 http://www.kdenlive.org/mantis/view.php?id=2235#c7183
----------------------------------------------------------------------
Use Scale and Tilt.

Crop uses the video/image at its full resolution prior to any automatic scaling
and padding to make it fit in the project resolution. Then, all other effects
apply. For best quality, it is best to crop the image at its full resolution
before scaling an input down to project resolution. This is useful for high
resolution photos or HD-to-SD conversion, especially when using the center
option to eliminate padding.

Perhaps we should rename 'Crop' to 'Edge Crop' and 'Scale and Tilt' to 'Crop,
Scale, and Position' (and its parameters).

----------------------------------------------------------------------
 (0008164) PeterA (reporter) - 2012-07-18 19:31
 http://www.kdenlive.org/mantis/view.php?id=2235#c8164
----------------------------------------------------------------------
I stumbled over this bug a few weeks ago. I had a tight deadline, so I had not
time to look up the bugtracker. I made up a workaround (working for my case),
using a Title-Clip with a black rectangles on each side and "Composite"
transition. But this is not very nice and works only you want to "crop to
black".

Now with more time, I found this bug report and tested ddennedy's tip, using the
"Scale and Tilt" filter. Result: It did perfectly crop, what I wanted to be
cropped.

So I definitely vote for renaming "Scale and Tilt" to "Crop, Scale, and
Position". Since I would never have found this solution without this bugreport.

Issue History
Date Modified    Username       Field                    Change
======================================================================
2011-07-12 20:52 eSyr           New Issue
2011-07-12 20:52 eSyr           File Added: kdenlive_crop_3.png

2011-07-12 21:16 eSyr           File Added: kdenlive_crop_1.png

2011-08-10 15:47 ttill          Issue Monitored: ddennedy
2011-08-10 15:47 ttill          Note Added: 0007170
2011-08-10 17:37 ddennedy       Note Added: 0007183
2012-07-18 19:31 PeterA         Note Added: 0008164
======================================================================




-- 
+-DRD-+