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

[Carl Eugen Hoyos]

2017-10-01 20:44 GMT+02:00 Ronald S. Bultje <rsbultje at gmail.com>:
> Hi,
>
> On Sun, Oct 1, 2017 at 12:17 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
> wrote:
>
>> 2017-10-01 14:03 GMT+02:00 Ronald S. Bultje <rsbultje at gmail.com>:
>> > Hi,
>> >
>> > On Sat, Sep 30, 2017 at 3:40 PM, Carl Eugen Hoyos <ceffmpeg at gmail.com>
>> > wrote:
>>
>> >> Attached patch also support GBRP, I don't know if this is a good or
>> >> bad idea.
>> >
>> > I would personally probably err on the side of caution, but no strong
>> > opinions. If this works (I assume it does), I guess it's fine (?).
>>
>> Do you want me to commit (with or without GBRP)?
>>
>> Sorry, I really don't know if this is supposed to do something
>> useful or not.
>>
>
> vmafmotion in itself isn't very useful, but it's part of the native vmaf
> filter, which will be quite useful.
>
> I'd suggest to commit it without GBRP support for now. I'm not against GBRP
> support per se, I just don't know if the results are meaningful (since we'd
> base the metric on the G channel only), so I'd rather be conservative.

Pushed without GBRP, thank you!

Carl Eugen