Re: [eigen] Eigen Types as Parameters for Functions |

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

*To*: eigen@xxxxxxxxxxxxxxxxxxx*Subject*: Re: [eigen] Eigen Types as Parameters for Functions*From*: Gael Guennebaud <gael.guennebaud@xxxxxxxxx>*Date*: Mon, 27 Feb 2012 19:34:50 +0100*Authentication-results*: mr.google.com; spf=pass (google.com: domain of gael.guennebaud@xxxxxxxxx designates 10.42.162.194 as permitted sender) smtp.mail=gael.guennebaud@xxxxxxxxx; dkim=pass header.i=gael.guennebaud@xxxxxxxxx*Dkim-signature*: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :content-type:content-transfer-encoding; bh=JpZLIb15d6Ky8EurAVP/dgm+DNLf5YwJAEFt7rptokc=; b=woOQqdowehezQZ1kYyXHeKWeAVjEJYz4spIXQJtXGWyP5XvnLqsnvVM7xWfOyjzRBP fsNFKBr3UP8JWdqKb5daE+s3THKahORPbQPlrt1BWTcNXW5sbgiKouccvgpqH7u+8qeD E2ozMaZX5aPP6NwFcQ5j6yfHBt1ohTUduAaFY=

Alright, I thought I had already replied to this thread but I haven't so here I go. My general idea to offer the ability to write non template function while still offering some flexibility on the input is really simple: simply add constructors to Map<> that can take Eigen's objects such as Matrix, Block, Map, etc. Then it is up to user to specify the level of flexibility he wants for his arguments: dynamic/fixed sizes, outer-stride? inner-stride? matrix/vector? For instance: void foo(const Map<MatrixXd,0,OuterStride<> >& arg); should accept any MatrixXd, Matrix4d, Block, Map, etc. There are some limitations though: - No template is possible at all, in particular it is not possible to templatize the scalar type, but this is fine to me since if the user can tolerate some template parameter then why not allowing everything. - The storage order has to be fixed too. I understand a runtime storage order might be convenient, but this is really not the Eigen philosophy, so this limitation will have to stay. However, I have question regarding the handling of non compatible expressions. For instance what should we do with: foo(A+B); // this is a pure expression without direct access or MatrixXcf C; foo(C.real()); // C.real() has an inner-stride, but foo does not accept that If we stick with Map<> I think that there is only one possible solution: compilation error. Then it is up to the user to explicitly evaluate it. Another approach would be to add a variant of Map tailored for this use case that would allow to silently evaluate the argument into a temporary. This is what happen if the user declare his argument as a const MatrixXd&. Any opinion on which approach we should follow? cheers, gael On Thu, Jan 26, 2012 at 3:12 PM, Christoph Hertzberg <chtz@xxxxxxxxxxxxxxxxxxxxxxxx> wrote: > On 26.01.2012 14:50, Benoit Jacob wrote: >> >> 2012/1/26 Christoph Hertzberg<chtz@xxxxxxxxxxxxxxxxxxxxxxxx>: >>> >>> On 26.01.2012 14:33, Benoit Jacob wrote: >>>> >>>> >>>> 2012/1/26 Christian Seiler<christian@xxxxxxxx>: >>>> >>>>> - subclass of MatrixBase, i.e. can be used inside the function like a >>>>> Matrix object >>>>> - has private members containing the above-mentioned bits of >>>>> information (pointer to first element, dimension, ...) >>>>> - can be implicitly converetd to from any compatible MatrixBase (i.e. >>>>> foo(mat) and foo(mat.block(...)) can be used directly) >>>> >>>> >>>> >>>> Just note that this is only possible for those matrix expressions >>>> whose coefficients are stored in memory with a simple enough layout >>>> that can be described in terms of strides. That is what we call a >>>> "Direct Access" matrix expression; the test is ExpressionType::Flags& >>>> DirectAccessBit. >>> >>> >>> >>> Yes, and I would say this would be the main application, if you have >>> complicated-enough functions (i.e. where you don't really bother about >>> evaluating expressions to a temporary if required). >>> And more importantly, when having parameters that are actually output >>> parameters, you are more or less stuck to direct access types anyway -- >>> and >>> this is a very important point, where currently const-correctness is >>> ignored >>> (almost) entirely. >>> >>> >>>> With that said, yes, we have been considering doing this, that's bug 58: >>>> http://eigen.tuxfamily.org/bz/show_bug.cgi?id=58 >>>> >>>> If someone wants to do this and needs mentoring, I should find time to >>>> do at least that mentoring. >>> >>> >>> >>> I'd be interested at least to join the "specification committee" ;) >> >> >> Really, I haven't thought about this in a long time so your thoughts >> on this subject are probably a lot fresher than mine. From the top of >> my head, the starting point is to look at MapBase (which is inherited >> both by Map and by Block<direct access type>) and ask: how much of the >> templated stuff there can we replace by runtime variables (data >> members), and try to make all direct-access types inherit the >> resulting base class (in particular, Matrix should inherit it). > > > Yes, my thoughts were about the same. Actually, the only difference between > a Map/Block and a Matrix should be that the Matrix "owns" its memory, i.e.. > has to deallocate it on destruction (with a special case of fixed size > matrices having static memory). > > Furthermore, if one allows strides in the Matrix-class, some specialized > types, such as AlignedVector3 would become just a typedef. Currently, if > someone needs an aligned vector7, he'd have to to the same thing again -- > also it might be interesting to store an aligned 3x3 matrix as the top 3 > rows of a 4x3 matrix allowing a much more efficient matrix-vector product.. > > > Christoph > > > > -- > ---------------------------------------------- > Dipl.-Inf. Christoph Hertzberg > Cartesium 0.051 > Universität Bremen > Enrique-Schmidt-Straße 5 > 28359 Bremen > > Tel: (+49) 421-218-64252 > ---------------------------------------------- > >

**Follow-Ups**:**Re: [eigen] Eigen Types as Parameters for Functions***From:*Gael Guennebaud

**Messages sorted by:**[ date | thread ]- Prev by Date:
**Re: [eigen] Status of AVX support** - Next by Date:
**Re: [eigen] Eigen Types as Parameters for Functions** - Previous by thread:
**[eigen] slow sparse dense product** - Next by thread:
**Re: [eigen] Eigen Types as Parameters for Functions**

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