Re: [eigen] beta1: Nullary tests failing

[ Thread Index | Date Index | More lists.tuxfamily.org/eigen Archives ]


2010/6/30 Carlos Becker <carlosbecker@xxxxxxxxx>:
> On Wed, Jun 30, 2010 at 6:45 PM, Benoit Jacob <jacob.benoit.1@xxxxxxxxx>
> wrote:
>>
>> 2010/6/30 Benoit Jacob <jacob.benoit.1@xxxxxxxxx>:
>>  Try this:
>>
>> replace this line,
>>  VERIFY( (row_vector-col_vector.transpose()).norm() <
>> NumTraits<Scalar>::epsilon() );
>>
>> by this:
>>  VERIFY( row_vector.isApprox(col_vector.transpose(),
>> NumTraits<Scalar>::epsilon()));
>>
>> this does a relative fuzzy check.
>>
>
> Ok this works fine and the tests pass. Should I push this change, or maybe
> wait for Hauke's opinion ?

You can push, this is a safe change. epsilon only is meaningful for
relative comparisons.
Benoit



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/