[FFmpeg-trac] #2014(avfilter:closed): Crop and Scale are location sensitive, fails if scale comes before crop

FFmpeg trac at avcodec.org
Wed Dec 12 16:38:16 CET 2012


#2014: Crop and Scale are location sensitive, fails if scale comes before crop
-------------------------------------+------------------------------------
             Reporter:  ramitbhalla  |                    Owner:
                 Type:  defect       |                   Status:  closed
             Priority:  normal       |                Component:  avfilter
              Version:  unspecified  |               Resolution:  invalid
             Keywords:  crop scale   |               Blocked By:
             Blocking:               |  Reproduced by developer:  1
Analyzed by developer:  1            |
-------------------------------------+------------------------------------
Changes (by saste):

 * status:  new => closed
 * reproduced:  0 => 1
 * component:  undetermined => avfilter
 * keywords:   => crop scale
 * analyzed:  0 => 1
 * resolution:   => invalid


Comment:

 Replying to [comment:5 Cigaes]:
 > Replying to [comment:4 ramitbhalla]:
 > > I agree, never said silently, throw a warning but don't fail the
 conversion.
 >
 > When something is wrong, a warning is not enough. If you want the
 conversion to succeed, provide it with correct parameters.
 >
 > > Why would you crop after scaling?
 >
 > Because the arithmetic may be easier, depending on circumstances.
 >
 > If you want your cropping to take effect before scaling, put the crop
 filter before the scale filter.

 Filter specification is location-sensitive by design, the user should be
 aware that most filtering operations are not commutative, and thus should
 provide a meaningful filterchain.
 Pretending "clever" guessworking from a program is a doomed idea.

 Issuing a more helpful error message is another issue (which I'm going to
 fix), but is unrelated to the present ticket subject.

-- 
Ticket URL: <https://ffmpeg.org/trac/ffmpeg/ticket/2014#comment:6>
FFmpeg <http://ffmpeg.org>
FFmpeg issue tracker


More information about the FFmpeg-trac mailing list