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

[Hauke Heibel <hauke.heibel@xxxxxxxxx>]

This seed crashes on my system 1403534440 with a 64 bit build and Visual Studio 2012. I have no time to debug this right now. I hope you can reproduce it on your side.

Regards,

Hauke

On Mon, Jun 23, 2014 at 4:15 PM, Hauke Heibel <hauke.heibel@xxxxxxxxx> 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 see the following potential solutions:

a) add a Spline namespace and move the typedef and other Spline related code there

b) add EIGEN_MAKE_ARRAY_TYPEDEFS_ALL_SIZES(DenseIndex, Idx) (or something similar)

c) be explicit and write it all out in the few places where it is used

I am not sure whether to use b) or c). At the moment I am slightly in favor of c) though that might change at any time.

Jeff already mentioned on the pull request that unfortunately the IndexType did not properly fit to the SplineTraits since it is independent of the Spline type.

Regards,

Hauke

On Mon, Jun 23, 2014 at 4:00 PM, Jeff J <complexzeros@xxxxxxxxx> wrote:

If anyone is interested in taking a look, I submitted a pull request for an addition I made to perform Spline interpolation to data points with derivative constraints. This is particularly useful for shape design, which is what I'm interested in.

Best,

Jeff