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

Carl Eugen Hoyos ceffmpeg at gmail.com
Sat Sep 30 21:19:37 EEST 2017


2017-09-30 19:47 GMT+02:00 Ronald S. Bultje <rsbultje at gmail.com>:
> Hi Carl,
>
> On Sat, Sep 30, 2017 at 1:31 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
> wrote:
>
>> Hi!
>>
>> 2017-09-15 22:47 GMT+02:00 Ashish Pratap Singh <ashk43712 at gmail.com>:
>>
>> > +static int query_formats(AVFilterContext *ctx)
>> > +{
>> > +    static const enum AVPixelFormat pix_fmts[] = {
>> > +        AV_PIX_FMT_YUV444P, AV_PIX_FMT_YUV422P,
>> > AV_PIX_FMT_YUV420P,
>> > +        AV_PIX_FMT_YUV444P10, AV_PIX_FMT_YUV422P10,
>> > AV_PIX_FMT_YUV420P10,
>>
>> Is the algorithm only defined for these formats and bit-depth
>> or are there just missing features?
>> Gray and gray10 come to mind...
>>
>
> Great question! I _believe_ that vmaf overall is luma-only, so it should be
> entirely independent of chroma.

Then imo, above function is just wrong, it should check for
non-rgb or similar (think of YUVA444 and friends).

Carl Eugen


More information about the ffmpeg-devel mailing list