On 12 November 2012 08:59, Sébastien Villemot <sebastien@debian.org> wrote:
> Rafael Laboissiere <rafael@laboissiere.net> writes:
>
>> I prepared version 2.0.0-1 in experimental for upload. I added two patches in
>> this version:
>
> Uploaded.
>
>> 1. fftconv2-usage.patch: This removes one of the bogus usage of fftconv2. I
>> discussed this with the upstream author Carnë Draug in the octave-dev mailing
>> list. Feel free to revert my patch if you think it is not appropriate.
>
> Since your patch fixes #664716, I kept it. I don't know how upstream
> wants to solve the issue.
Talking to Carnë in IRC, it seems clear that the problem isn't that
the test is wrong but rather that fftconv2 is wrong here. It seems
like an actual bug.
OF list: fftconv2's vector-vector-matrix three-arg call seems
incorrect and seems to have been incorrect for some time. Can anyone
please confirm? These two calls should both be equal:
octave:1> conv2(1:3, 1:2, [1 2; 3 4; 5 6])
ans =
1 4 4
5 18 16
14 48 40
19 62 48
15 48 36
octave:2> fftconv2(1:3, 1:2, [1 2; 3 4; 5 6])
ans =
1 4 7 6
5 18 31 24
11 36 61 42
10 32 54 36
The upper result is correct. You can nobtain this result with
fftconv2(fftconv2([1 2; 3 4; 5 6,1:2),1:3), so I think this should be
fairly easy to fix.
Patches, anyone?
- Jordi G. H.
------------------------------------------------------------------------------
Monitor your physical, virtual and cloud infrastructure from a single
web console. Get in-depth insight into apps, servers, databases, vmware,
SAP, cloud infrastructure, etc. Download 30-day Free Trial.
Pricing starts from $795 for 25 servers or applications!
http://p.sf.net/sfu/zoho_dev2dev_nov
_______________________________________________
Octave-dev mailing list
Octave-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/octave-dev