Re: [math] Re: SingularValueDecompositionImpl

2011-08-15 Thread Gilles Sadowski
Hi. > > Also, I think that further cleanup is possible (cf. "wantu" and "wantv" > > checks that don't seem very useful). > > This is residual from the JAMA code and is not required with our current > interface. I removed them in r1157281. > I wonder if we could implement something around this.

Re: [math] Re: SingularValueDecompositionImpl

2011-08-15 Thread Chris Nix
> Also, I think that further cleanup is possible (cf. "wantu" and "wantv" > checks that don't seem very useful). This is residual from the JAMA code and is not required with our current interface. I wonder if we could implement something around this. I can conceive of cases where users may only

Re: [math] Re: SingularValueDecompositionImpl

2011-08-12 Thread Greg Sterijevski
Thanks! There is an additional method I am moving in. Its the inverse of the condition number. public double getInverseConditionNumber() { return singularValues[FastMath.min(m,n) - 1] / singularValues[0]; } This addition stems from an issue (MATH-602) I submitted awhile back. Name

Re: [math] Re: SingularValueDecompositionImpl

2011-08-12 Thread Gilles Sadowski
On Thu, Aug 11, 2011 at 04:31:15PM -0500, Greg Sterijevski wrote: > At least three with some code I checked in last night. The point is that > there is no reason to replicate the same thing over and over again. I understand the point. I've replaced those 2 occurrences which I detected (revision 11

Re: [math] Re: SingularValueDecompositionImpl

2011-08-11 Thread Greg Sterijevski
At least three with some code I checked in last night. The point is that there is no reason to replicate the same thing over and over again. -Greg On Thu, Aug 11, 2011 at 6:33 AM, Gilles Sadowski < gil...@harfang.homelinux.org> wrote: > > > In the SVD class I notice: > > > > > > FastMath.max(m,

Re: [math] Re: SingularValueDecompositionImpl

2011-08-11 Thread Gilles Sadowski
> > In the SVD class I notice: > > > > FastMath.max(m, n) > > > > all over the place. Since these values are known when the constructor is > > called and are final, would anyone object to making the result a private > > instance variable? I see only 2 places: lines 557 and 569. [In one of them it

[math] Re: SingularValueDecompositionImpl

2011-08-10 Thread Phil Steitz
On 8/10/11 3:51 PM, Greg Sterijevski wrote: > Hello All, > > In the SVD class I notice: > > FastMath.max(m, n) > > all over the place. Since these values are known when the constructor is > called and are final, would anyone object to making the result a private > instance variable? +1 (note chan