Re: [eigen] eigen sqrt compilation error

[ Thread Index | Date Index | More Archives ]

I say keep the core and array separate and provide eigen, dense and
sparse only as a convenience header.

On Fri, Jun 19, 2009 at 12:07 PM, Benoit Jacob<jacob.benoit.1@xxxxxxxxx> wrote:
> 2009/6/19 Patrick Mihelich <patrick.mihelich@xxxxxxxxx>
>> On Fri, Jun 19, 2009 at 5:50 AM, Benoit Jacob <jacob.benoit.1@xxxxxxxxx> wrote:
>>> The #include<Eigen/Array> increases compilation times by 2.5% and the
>>> memory usage is not affected.
>> 2.5 percent on a simple test program?
> Simple, but realistic, not just an abstract test program. It makes a
> decent number of instantiations, so i think that it is a real use
> case. So real users who would lose 2.5% of compilation speed if we
> just merged Array into Core.
> There's another little thing that bugs me: if we merge Array into Core
> for the reason that it doesn't slow compilation down much, then in the
> future we'll be more reluctant to add more features to Array (like how
> we're conservative with Core). As long as Array is separate, we feel
> more free...
> Finally I wonder how much of a use there still is for a combined
> Core+Array, now that there is the Dense header.
> (None of what i say here is a stong opinion...)
> Benoit

Rohit Garg

Senior Undergraduate
Department of Physics
Indian Institute of Technology

Mail converted by MHonArc 2.6.19+