Re: [eigen] Addition to Eigen-unsupported Spline module

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


On 23.06.2014 16:15, Hauke Heibel wrote:
There is one potential issue which came up when I looked at the code.

The code introduces an Eigen wide "typedef Array<DenseIndex, 1, Dynamic>
IndexType".

This is similar to the typedefs provided by Array.h though only typedefs
for int exist and none for DenseIndex.

I think there will be other places, where arrays of indexes will be useful, most importantly, perhaps, when implementing
http://eigen.tuxfamily.org/bz/show_bug.cgi?id=329

At least for bug 329 I think it would be better to allow anything that has a size() and operator[] (both returning any type of integer) implemented, e.g., also allow std::vector and std::array (if available).
The same is true for the in-development Tensor-module.


Christoph

--
----------------------------------------------
Dipl.-Inf., Dipl.-Math. Christoph Hertzberg
Cartesium 0.049
Universität Bremen
Enrique-Schmidt-Straße 5
28359 Bremen

Tel: +49 (421) 218-64252
----------------------------------------------



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