Re: [eigen] On complex read-cost

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



I've done it.

bye,
-- 
Thomas Capricelli <orzel@xxxxxxxxxxxxxxx>
http://www.freehackers.org/thomas

In data mercoledì 26 maggio 2010 01:45:10, Benoit Jacob ha scritto:
: > Yes, indeed, good find, patch welcome!
> 
> http://eigen.tuxfamily.org/index.php?title=Developer's_Corner#Generating_a_patch
> 
> this needs to be fixed in both branches!
> 
> Benoit
> 
> 2010/5/25 Manoj Rajagopalan <rmanoj@xxxxxxxxx>:
> > Hi eigen developers,
> >
> >   The read-cost for complex<_Real> in NumTraits.h is hard-coded to 2 unlike
> > the AddCost and MulCost which are functions of the eponymous costs for the
> > corresponding real NumTraits. I suggest that
> > NumTraits<complex<_Real>>::ReadCost = 2*NumTraits<_Real>::ReadCost
> >
> >   If this change is made, it will correctly propagate read costs for
> > multi-component and multi-word arbitrary precision custom scalar.
> >
> > thanks,
> > Manoj
> >
> >
> >
> 
> 
> 



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