Hi, On Sun, Oct 1, 2017 at 12:17 PM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote:
> 2017-10-01 14:03 GMT+02:00 Ronald S. Bultje <rsbul...@gmail.com>: > > Hi, > > > > On Sat, Sep 30, 2017 at 3:40 PM, Carl Eugen Hoyos <ceffm...@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. Ronald _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
