octave-maintainers
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Test failure for fftfilt.m


From: Daniel J Sebald
Subject: Re: Test failure for fftfilt.m
Date: Sat, 16 Mar 2013 17:28:55 -0500
User-agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111108 Fedora/3.1.16-1.fc14 Thunderbird/3.1.16

On 03/16/2013 05:18 PM, Ed Meyer wrote:

I fixed the tests last fall according to what I consider the right way to
do tests like this. As I have said several times before tests should not
in general use absolute tolerances, rather they should take into account
the size of the data. So for example instead of

assert (fftfilt (b, r * x),r * r * [1, 1, 0, 0, 0, 0, 0, 0, 0, 0],eps)

I used

%! y = r*r*[1 1 0 0 0 0 0 0 0 0];
%! tol = 2*norm(y)*eps;
%! assert (fftfilt (b, r*x), y, tol);

There are a number of tests like this (eig.cc for example) with absolute
tolerances which should be using norms. But don't take my word for it,
see, e.g. Golub & Van Loan, "Matrix Computations".

Yes, I'm a bit sketchy, but I remember looking at the work you had done. I thought you checked it in. But this file log would suggest it wasn't:

http://hg.savannah.gnu.org/hgweb/octave/log/7d0848353eaf/scripts/signal/fftfilt.m

Dan


reply via email to

[Prev in Thread] Current Thread [Next in Thread]