Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Sun, Oct 8, 2017 at 3:26 PM, Nicolas George wrote: > Le septidi 17 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > > I think you're stretching the definition of "flaw" here. > > I addressed that in the paragraph you snipped: I agree that the > usability

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/8/17, Nicolas George wrote: > Le septidi 17 vendemiaire, an CCXXVI, Ronald S. Bultje a ecrit : >> I think you're stretching the definition of "flaw" here. > > I addressed that in the paragraph you snipped: I agree that the > usability considerations can be considered an

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le septidi 17 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > I think you're stretching the definition of "flaw" here. I addressed that in the paragraph you snipped: I agree that the usability considerations can be considered an enhancement, but the security considerations must be considered

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Sun, Oct 8, 2017 at 2:04 PM, Nicolas George wrote: > Le septidi 17 vendémiaire, an CCXXVI, Hendrik Leppkes a écrit : > > Usually those not happy with an existing solution should also offer up > > a replacement, even if only in concept (actual implementable concept, > >

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le septidi 17 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > Hm... Michael has proposed 3 solutions and I commented on each of them. I saw, but basically it amounts to agreeing with the solutions that remove the file output. I am very fine with that, but other people have disagreed.

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le septidi 17 vendémiaire, an CCXXVI, Hendrik Leppkes a écrit : > Usually those not happy with an existing solution should also offer up > a replacement, even if only in concept (actual implementable concept, > not just a slogan) That is the policy for enhancements, but not for flaws. When a flaw

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi Nicolas, On Sun, Oct 8, 2017 at 9:38 AM, Nicolas George wrote: > Le septidi 17 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > > > I think that we should refuse any new stats-gathering or similar filter > > > until a clean and working solution has been designed. > > I

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sun, Oct 8, 2017 at 5:05 PM, Nicolas George wrote: > Le septidi 17 vendémiaire, an CCXXVI, Paul B Mahol a écrit : >> Your lack of common sense have been noted. > > I want a clean solution. A solution that works for most uses. A solution > that works the same way for all

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le septidi 17 vendémiaire, an CCXXVI, Paul B Mahol a écrit : > Your lack of common sense have been noted. I want a clean solution. A solution that works for most uses. A solution that works the same way for all filters. And, if possible, a solution that is easy to secure. Is that lack of common

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/8/17, Nicolas George wrote: > Le septidi 17 vendemiaire, an CCXXVI, Ronald S. Bultje a ecrit : >> > I think that we should refuse any new stats-gathering or similar filter >> > until a clean and working solution has been designed. >> I don't think that is fair. I don't

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le septidi 17 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > > I think that we should refuse any new stats-gathering or similar filter > > until a clean and working solution has been designed. > I don't think that is fair. I don't agree with this approach. Your lack of alternative proposal

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Sun, Oct 8, 2017 at 9:06 AM, Nicolas George wrote: > I think that we should refuse any new stats-gathering or similar filter > until a clean and working solution has been designed. > I don't think that is fair. I don't agree with this approach. Ronald

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le sextidi 16 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > Calm down guys, I said in the next sentence I'm open to alternatives. The frustration in my message, if any, was not directed at you. But I think you should not only be "open" to alternatives but more "actively searching for"

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le sextidi 16 vendémiaire, an CCXXVI, Hendrik Leppkes a écrit : > If there is a command line to write a stats file with similar > information from this metadata - I don't think anyone would then > object to that. It was sarcastic: storing the stats in metadata is also a quick-and-dirty solution

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sat, Oct 07, 2017 at 11:39:40AM -0400, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 7, 2017 at 6:25 AM, Michael Niedermayer > wrote: > > > On Sat, Oct 07, 2017 at 12:12:15PM +0200, Michael Niedermayer wrote: > > > > > On Fri, Oct 06, 2017 at 10:03:16AM -0400,

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, Ronald S. Bultje wrote: > Hi, > > On Sat, Oct 7, 2017 at 6:25 AM, Michael Niedermayer > wrote: > >> On Sat, Oct 07, 2017 at 12:12:15PM +0200, Michael Niedermayer wrote: >> > > > On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Sat, Oct 7, 2017 at 6:25 AM, Michael Niedermayer wrote: > On Sat, Oct 07, 2017 at 12:12:15PM +0200, Michael Niedermayer wrote: > > > > On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: > > > >> I'm open to any alternative method you

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Sat, Oct 7, 2017 at 5:09 AM, Nicolas George wrote: > Le quintidi 15 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > > The same mechanism is present in ssim/psnr filters. > > This is true. But how do you think it came to happen? > Calm down guys, I said in the next

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sat, Oct 07, 2017 at 12:12:15PM +0200, Michael Niedermayer wrote: > On Sat, Oct 07, 2017 at 10:09:45AM +0200, Paul B Mahol wrote: > > On 10/7/17, James Almer wrote: > > > On 10/6/2017 7:09 PM, wm4 wrote: > > >> On Fri, 6 Oct 2017 18:02:44 -0300 > > >> James Almer

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sat, Oct 07, 2017 at 10:09:45AM +0200, Paul B Mahol wrote: > On 10/7/17, James Almer wrote: > > On 10/6/2017 7:09 PM, wm4 wrote: > >> On Fri, 6 Oct 2017 18:02:44 -0300 > >> James Almer wrote: > >> > >>> On 10/6/2017 5:44 PM, Paul B Mahol wrote: > On

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, Hendrik Leppkes wrote: > On Sat, Oct 7, 2017 at 11:24 AM, Nicolas George wrote: >> Le sextidi 16 vendemiaire, an CCXXVI, Paul B Mahol a ecrit : >>> There is metadata for each frame added and it can be used right now. >> >> Then we can remove that

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sat, Oct 7, 2017 at 11:24 AM, Nicolas George wrote: > Le sextidi 16 vendémiaire, an CCXXVI, Paul B Mahol a écrit : >> There is metadata for each frame added and it can be used right now. > > Then we can remove that file-writing option. Thanks for the > clarification. > If

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, Nicolas George wrote: > Le sextidi 16 vendemiaire, an CCXXVI, Paul B Mahol a ecrit : >> No. > > Very constructive, thanks. No. ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le sextidi 16 vendémiaire, an CCXXVI, Paul B Mahol a écrit : > No. Very constructive, thanks. -- Nicolas George signature.asc Description: Digital signature ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, Nicolas George wrote: > Le sextidi 16 vendemiaire, an CCXXVI, Paul B Mahol a ecrit : >> There is metadata for each frame added and it can be used right now. > > Then we can remove that file-writing option. Thanks for the > clarification. No.

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le sextidi 16 vendémiaire, an CCXXVI, Paul B Mahol a écrit : > There is metadata for each frame added and it can be used right now. Then we can remove that file-writing option. Thanks for the clarification. Regards, -- Nicolas George signature.asc Description: Digital signature

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, Nicolas George wrote: > Le quintidi 15 vendemiaire, an CCXXVI, Ronald S. Bultje a ecrit : >> The same mechanism is present in ssim/psnr filters. > > This is true. But how do you think it came to happen? > > It happened because somebody wanted some feature for their

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Le quintidi 15 vendémiaire, an CCXXVI, Ronald S. Bultje a écrit : > The same mechanism is present in ssim/psnr filters. This is true. But how do you think it came to happen? It happened because somebody wanted some feature for their use and implemented it (so far, very good), and since it

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/7/17, James Almer wrote: > On 10/6/2017 7:09 PM, wm4 wrote: >> On Fri, 6 Oct 2017 18:02:44 -0300 >> James Almer wrote: >> >>> On 10/6/2017 5:44 PM, Paul B Mahol wrote: On 10/6/17, Michael Niedermayer wrote: > On Fri,

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/6/2017 7:09 PM, wm4 wrote: > On Fri, 6 Oct 2017 18:02:44 -0300 > James Almer wrote: > >> On 10/6/2017 5:44 PM, Paul B Mahol wrote: >>> On 10/6/17, Michael Niedermayer wrote: On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote:

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Fri, 6 Oct 2017 18:02:44 -0300 James Almer wrote: > On 10/6/2017 5:44 PM, Paul B Mahol wrote: > > On 10/6/17, Michael Niedermayer wrote: > >> On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: > >>> Hi, > >>> > >>> On Thu, Oct 5,

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Fri, Oct 06, 2017 at 11:04:24PM +0200, Hendrik Leppkes wrote: > On Fri, Oct 6, 2017 at 10:21 PM, Michael Niedermayer > wrote: > > > > With this a filter graph can never be taken from an untrusted source > > > > > > The same could be said for any ffmpeg CLI command line

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Fri, Oct 6, 2017 at 10:21 PM, Michael Niedermayer wrote: > > With this a filter graph can never be taken from an untrusted source > > The same could be said for any ffmpeg CLI command line string, which people happily share on the web all the time, it can also contain

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/6/2017 5:44 PM, Paul B Mahol wrote: > On 10/6/17, Michael Niedermayer wrote: >> On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: >>> Hi, >>> >>> On Thu, Oct 5, 2017 at 7:52 PM, Michael Niedermayer >>> >>> wrote: >>> On

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On 10/6/17, Michael Niedermayer wrote: > On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: >> Hi, >> >> On Thu, Oct 5, 2017 at 7:52 PM, Michael Niedermayer >> >> wrote: >> >> > On Sat, Sep 30, 2017 at 03:51:41PM +, Ashish Singh

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Fri, Oct 06, 2017 at 10:03:16AM -0400, Ronald S. Bultje wrote: > Hi, > > On Thu, Oct 5, 2017 at 7:52 PM, Michael Niedermayer > wrote: > > > On Sat, Sep 30, 2017 at 03:51:41PM +, Ashish Singh wrote: > > > ffmpeg | branch: master | Ashish Singh

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

Hi, On Thu, Oct 5, 2017 at 7:52 PM, Michael Niedermayer wrote: > On Sat, Sep 30, 2017 at 03:51:41PM +, Ashish Singh wrote: > > ffmpeg | branch: master | Ashish Singh | Sat Sep > 16 02:35:58 2017 +0530| [148c8e88c43cfbabd6aee9f01ef30942cee9d359]

Re: [FFmpeg-devel] [FFmpeg-cvslog] avfilter: add vmafmotion filter

On Sat, Sep 30, 2017 at 03:51:41PM +, Ashish Singh wrote: > ffmpeg | branch: master | Ashish Singh | Sat Sep 16 > 02:35:58 2017 +0530| [148c8e88c43cfbabd6aee9f01ef30942cee9d359] | committer: > Ronald S. Bultje > > avfilter: add vmafmotion filter > > Signed-off-by: