Re: [eigen] Feature request - Prevent dynamic resizing of a matrix |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen Archives
]
- To: eigen@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [eigen] Feature request - Prevent dynamic resizing of a matrix
- From: Timothy Hunter <tjhunter@xxxxxxxxxxxx>
- Date: Wed, 28 Jan 2009 11:04:50 -0800
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:sender:received:in-reply-to :references:date:x-google-sender-auth:message-id:subject:from:to :content-type:content-transfer-encoding; bh=kPEoGxpbOAgJkoklfInf+GI2a3Kz5mupwGFT3Buwq54=; b=g+lUaAYm24m/MR+//xYhGsuTZ+PMhfrnxg3d4GCIAvWdCRqjEOBQ36VmnXZyPdWj7f j6cBtTiS7UyUIr96MJ0LtrM1W9qmI5UrmCPpuW3jqzi/jmqfgcGftn44cd25oA99eyej 0VWsLP1IsWPgj/SUMNoPoppmGqOzsS8Nu0uow=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:sender:in-reply-to:references:date :x-google-sender-auth:message-id:subject:from:to:content-type :content-transfer-encoding; b=DSSE90gNZFQEoieJqyIarwkenw9Yfbc+5tDFSLBh61mPiLyYJAhIrohGbvaOs00/7j psKblOTQ/c2SnVBu6W1Qy7WfeNTYM9+uqgJKQ8taG8l4saN69pRC4qnChKnvpZGmUn9v 9oB+sQAkJAxzfuenOqTnikqOCLb/GpB0M0Gd4=
I do not think using a #DEFINE in this case is a bad idea, as it is
mostly a debugging feature. I am having a hard time to find a case in
production code where you would mix the two behaviors.
Cheers
Tim
On Wed, Jan 28, 2009 at 9:51 AM, Benoit Jacob <jacob.benoit.1@xxxxxxxxx> wrote:
> 2009/1/28 Keir Mierle <mierle@xxxxxxxxx>:
>> This seems like the best way to me, defaulting to Eigen::Resizeable.
>
> Actually, Gael rejected this idea today on IRC with a valid argument.....
>
>> Does
>> adding more template parameters like this adversely affect compilation
>> time?
>
> ..... and his argument was indeed that we shouldn't duplicate template
> types -- it's a golden rule in Eigen to minimize the number of
> template types as this is indeed important for compilation time,
> compiler memory usage, and most importantly, object code size.
>
> Cheers,
> Benoit
>
>
>