Re: possible issue in octave-forge package image (fftconv2)

[Carnë Draug]

On 4 July 2013 01:08, Alireza Nejati <address@hidden> wrote:
> In fftconv2.m in the image package, this is what line 47 currently looks
> like (in the latest version downloaded from sourceforge):
>
>      if ((nargin > 2) && ismatrix(varargin{3}))
>
> Since octave returns 1 for, say, ismatrix("same"), it doesn't recognize
> fftconv2(a,b,shape) properly. It thus returns errors for me. However, if I
> also add a isnumeric(varargin{3}) check, it works fine. This is what the
> corrected line looks like:
>
>     if ((nargin > 2) && ismatrix(varargin{3}) && isnumeric(varargin{3}))

Hi

that bug has already been fixed on the development version [1],
though the check we did was !ischar().

However, the function has another bug that went undiagnosed for a long
time. It returns the wrong results when you use different vectors for
the convolution in the rows and columns direction. Take a look a the
source for more details. If you could fix that one, that'd be great.

Carnë


[1] https://sourceforge.net/p/octave/image/ci/default/tree/inst/fftconv2.m