Re: [eigen] stable norm

[ Thread Index | Date Index | More Archives ]

On Wed, Jun 9, 2010 at 9:13 AM, Hauke Heibel <hauke.heibel@xxxxxxxxxxxxxx> wrote:

There is a glitch in stable norm. I think these two lines should be removed

 VERIFY(!isFinite( ei_abs(big)/RealScalar(0)));

The test seems useless since any number divided by zero will yield Inf
and thus the test is always true.

Is that right, or am I overseeing someting?

yes, the goal of these lines was simply to test the function isFinite itself, not the fact that x/0 returns Inf.

So I still think it is nice to keep them.



- Hauke

Mail converted by MHonArc 2.6.19+