[FFmpeg-devel] [PATCH] avfilter: add ssim filter

[Ronald S. Bultje]

Hi,

On Tue, Jun 23, 2015 at 10:42 AM, Paul B Mahol <onemda at gmail.com> wrote:

> On 6/22/15, Paul B Mahol <onemda at gmail.com> wrote:
> > On 6/22/15, Nicolas George <george at nsup.org> wrote:
> >> Le quartidi 4 messidor, an CCXXIII, Paul B Mahol a ecrit :
> >>> Signed-off-by: Paul B Mahol <onemda at gmail.com>
> >>> ---
> >>>  Changelog                |   1 +
> >>>  doc/filters.texi         |  53 ++++++++
> >>>  libavfilter/Makefile     |   1 +
> >>>  libavfilter/allfilters.c |   1 +
> >>>  libavfilter/vf_ssim.c    | 340
> >>> +++++++++++++++++++++++++++++++++++++++++++++++
> >>>  5 files changed, 396 insertions(+)
> >>>  create mode 100644 libavfilter/vf_ssim.c
> >>
> >> Would it make sense to merge it with the psnr filter, sharing code and
> >> offering a common user interface for various quality measurements?
> >
> > Just to put it into same file?
> > Moving it into same filter would be also possible but how would then
> > new filter be named then?
>
> Looks like there are no new comments so I can safely assume current
> state as is is good enough.


Yeah lgtm.

Ronald