error compiling MRPT from svn [EIGEN_YES_I_KNOW_SPARSE_MODULE_IS_NOT_STABLE_YET]

[bellonemauro]

error compiling MRPT from svn [EIGEN_YES_I_KNOW_SPARSE_MODULE_IS_NOT_STABLE_YET]

[mlopezantequera]

same thing here

[bellonemauro]

As Josè Luis Blanco just told me, in cmake there is a config flag "use Embedeed Eigen" just check it and it should work for now. I've just compiled.
Then, I can suggest to who will fix this to consider the possibility of setting such flag to 1 as default, in order to avoid others the same problem (maybe just for know),
Mauro

[jlblancoc]

Hi guys,

I told @bellonemauro to upload this so @MarianoJT88 can take a look at the usage of CSparseMatrix in his class, and then:

1. If it's not actually needed, just remove the #include of sparse matrices in the DifOdo headers.

2. or, if they are really needed (and in the future for anything that requires Eigen/Sparse), please take a look at how the random field maps handle that:

*Header: *
https://github.com/jlblancoc/mrpt/blob/b4e802ca975eb163f31387a6eb8fa3753612299d/libs/maps/include/mrpt/slam/CRandomFieldGridMap2D.h#L23

Source
https://github.com/jlblancoc/mrpt/blob/b4e802ca975eb163f31387a6eb8fa3753612299d/libs/maps/src/maps/CRandomFieldGridMap2D.cpp#L255

Loook for every usage of EIGEN_VERSION_AT_LEAST(). I know it's tiring, but at least we assure that MRPT builds in all platforms...

Hope it helps! :-)
@MarianoJT88 : when you upload the fix, feel free to close this issue please!

@bellonemauro : It's better to leave that decision to the user, or to the automatic scripts that build mrpt for each Ubuntu distribution, so we can use the embedded version only when really necessary.

[MarianoJT88]

Class "CDifodo" has already been modified to work without Eigen/sparse.
Therefore I close this issue which should be corrected now.