[FFmpeg-devel] [PATCH] avfilter/interlace: prevent over-sharpening with the complex low-pass filter

Michael Niedermayer michael at niedermayer.cc
Fri Sep 15 23:26:37 EEST 2017


On Fri, Sep 15, 2017 at 04:38:16PM +0200, Thomas Mundt wrote:
> 2017-09-13 23:35 GMT+02:00 Thomas Mundt <tmundt75 at gmail.com>:
> 
> > 2017-09-06 22:15 GMT+02:00 Thomas Mundt <tmundt75 at gmail.com>:
> >
> >> 2017-09-01 1:55 GMT+02:00 Thomas Mundt <tmundt75 at gmail.com>:
> >>
> >>> 2017-09-01 1:22 GMT+02:00 Michael Niedermayer <michael at niedermayer.cc>:
> >>>
> >>>> On Fri, Sep 01, 2017 at 01:18:11AM +0200, Michael Niedermayer wrote:
> >>>> > On Thu, Aug 31, 2017 at 10:40:12PM +0200, Thomas Mundt wrote:
> >>>> > > 2017-08-31 21:42 GMT+02:00 Michael Niedermayer
> >>>> <michael at niedermayer.cc>:
> >>>> > >
> >>>> > > > On Wed, Aug 30, 2017 at 03:54:08AM +0200, Thomas Mundt wrote:
> >>>> > > > > Hi,
> >>>> > > > >
> >>>> > > > > we did a transcoding cascade test at work were over-sharpening
> >>>> became
> >>>> > > > > visible with the complex low-pass filter. This patch rectifies
> >>>> the
> >>>> > > > > behaviour.
> >>>> > > > >
> >>>> > > > > Please comment...
> >>>> > > >
> >>>> > > > did you perform subjective and or objective tests ?
> >>>> > > >
> >>>> > > > objective being some metric like PSNR vs correctly sampled data
> >>>> > > > subjective being double blind tests with humans about which they
> >>>> prefer
> >>>> > > >
> >>>> > >
> >>>> > > The tests have been subjective with a team of video engineers and
> >>>> > > technicians with several test files.
> >>>> > >
> >>>> > > I did a quick SSIM/PSNR check with the first generation of an
> >>>> HD->SD file
> >>>> > > as a reference against the 6th generation.
> >>>> >
> >>>> > if its better then iam fine with it. No more comments from me
> >>>>
> >>>> ahh, forgot, you may want tp add the tests you did and their results
> >>>> to the commit message
> >>>
> >>>
> >>> Sure, new patch attached.
> >>>
> >>
> >> Ping
> >>
> >
> > Ping!
> >
> 
> Michael, is it possible for you to push this?

will push

also i see noone listed in MAINTAINERS for vf_(t)interlace
if noone objects, you might want to send a patch that adds you for
them. It seems you are the most active on these ATM

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I do not agree with what you have to say, but I'll defend to the death your
right to say it. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20170915/019dd6ff/attachment.sig>


More information about the ffmpeg-devel mailing list