KDTreeEigenMatrixAdaptor for the case when points of the state space are located in columns

[AlekseyMuraviev]

KDTreeEigenMatrixAdaptor works with data in which "Each row in the matrix represents a point in the state space", i.e. for three-dimensional points, the data should be stored in the Eigen::MatrixX3d.

In some cases, the points are conveniently stored in columns - in the Eigen::Matrix3Xd.
For example, here implemented an adapter for this case:

...
struct KDTreeAdaptor {
    ...
    KDTreeAdaptor(const MatrixType &mat, const int leaf_max_size = 10) : m_data_matrix(mat) {
        const size_t dims = mat.rows();
        ...

Can you add this option to the library?

[jlblancoc]

Ok, makes sense... Feel free of proposing a pull request, for example, adding an optional argument to the template to define the data row/column order. El 10 mar. 2017 9:38 a. m., "AlekseyMuraviev" <notifications@github.com> escribió:

…

KDTreeEigenMatrixAdaptor works with data in which "*Each row in the matrix represents a point in the state space*", i.e. for three-dimensional points, the data should be stored in the Eigen::MatrixX3d. In some cases, the points are conveniently stored in columns - in the Eigen::Matrix3Xd. For example, here <https://github.com/OpenGP/sparseicp/blob/master/ICP.h> implemented an adapter for this case: ... struct KDTreeAdaptor { ... KDTreeAdaptor(const MatrixType &mat, const int leaf_max_size = 10) : m_data_matrix(mat) { const size_t dims = mat.rows(); ... Can you add this option to the library? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#37>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AFPj2r6HB8fjBIFZF4Jqa5FQrFwAqRGdks5rkQwggaJpZM4MZHIE> .