[FFmpeg-devel] [FFmpeg-cvslog] Adds ESPCN super resolution filter merged with SRCNN filter.

[Jean-Baptiste Kempf]

Hello,

On Mon, 2 Jul 2018, at 21:54, Pedro Arthur wrote:
> 2018-07-02 16:19 GMT-03:00 Jean-Baptiste Kempf <jb at videolan.org>:
> > On Mon, 2 Jul 2018, at 20:52, Pedro Arthur wrote:
> >> >> > Where do they come from, how can we recreate them?
> >> >> Paper link [1], and web page with reference matlab code [2].
> >> >
> >> > This code is not open source, and is not compatible with LGPLv2.1:
> >> >
> >> > "If you use/adapt our code in your work (either as a stand-alone tool or as a component of any algorithm),
> >> > you need to appropriately cite our ECCV 2014 paper or arXiv paper."
> >> >
> >> > Reimplementation of the code in a different language does not remove IP.
> >> The paper is properly cited in libavfilter/vf_sr.c.
> >>
> >> I not specialist in  IP rights but  there is nothing novel in the code
> >> (TBH there is no code at all, look at srcnn.m) it is just plain
> >> convolution, basic math machinery, we are just applying a bunch of
> >> filter convolution to an image.
> >
> > Well, either it is based on a paper and the matlab code, or it is not.
> > If it is, it's a big issue, since this is not open source compatible.
> >
> > If it is just convolution, then why is it based on this paper?
> Our code does basic convolutional neural network operations, the same
> as lots of other ML libraries, this is simple math no IP.

Yet, there are lots of table of unexplained data arrays. Where are those numbers coming from?

> The paper proposes a model (basically a filter layout) were it uses 3
> convolutional layers with filters(input x width x height x output)
> 1x9x9x64, 64x1x1x32, 32x5x5x1.
> Therefore the only thing you can consider novel is the filter
> dimension. Training also does not constitute IP as it just applying
> any optimization algorithm like Gradient Descent to the model.
> 
> Considering filter size as IP seems (to me) as absurd as considering
> an image size as IP.

Then what are the numbers?

> > What is the source of those numbers?
> They are the filters (or weights), result of training the model with a
> particular set of image data.

Where is the data set?

Sorry, you cannot dump 10kB of array and not explain where they come from.

9479955 was bad already with 3 convolution kernels dumped with almost no explanation.
And then basically reverted most of those kernels in bdf1bbd.
And then adding other unexplained numbers in d8c0bbb0.

But this one is beyond that...

(Not to mention the invalid license)

-- 
Jean-Baptiste Kempf -  President
+33 672 704 734