[SciPy-user] Re: bug in diagsvd

Pearu Peterson pearu at scipy.org
Tue Jun 1 12:55:46 CDT 2004



On Tue, 1 Jun 2004, Kirill Lapshin wrote:

> > Thanks for the patch, this is now fixed in CVS.
> 
> Wow. You rock. Less then an hour from bugreport to fix.
> 
> One nitpick though: the reason why this bug crept in on first place is 
> missing unit test on diagsvd. If you would add even a simple two line 
> test, like one Carlo posted, I would feel much much safer. If you want I 
> can propose a patch.

A very simple unittest for diagsvd is in CVS now. There are always missing 
unittests in various scipy modules. For instance, when you execute 
scipy.test() then it will show which modules completely lack unittests. 
So, any addition to scipy unittest site will be greately appreciated.

Thanks,
Pearu



More information about the SciPy-user mailing list