Potentially upstream the improvements to PCL? about ndt_omp HOT 9 OPEN

Hi @SteveMacenski @kunaltyagi ,
I believe there is no major concerns about upstreaming this module to PCL.
Some minor concerns are:

• namespace and class names should be changed
• the code should be updated to follow the recent PCL registration interface convention

I would like to update the code and make a PR on the PCL repository.

from ndt_omp.

Current status: One refactor PR is ongoing: PointCloudLibrary/pcl#4180 (waiting on my review, perhaps by this weekend)

One has been merged, and NDT OMP PR hasn't been created yet (IIRC)

from ndt_omp.

A PR would be welcome.

Please bear in mind a PR before GSoC begins or towards end of GSoC would be preferred. (Reasons being time and a GSoC project on new API to integrate SSE and non SSE code together)

from ndt_omp.

Awesome! That sounds really great. Thanks for that help, it will go a long way.

from ndt_omp.

Any movement here?

We're looking at implementing a NDT-MCL algorithm for Navigation2 to replace AMCL and we'd love to make use of this speed up!

from ndt_omp.

(we're also open for any help / collaboration on that front)

from ndt_omp.

Yes, we have been working on refactoring of the original NDT implementation. After it approved, I'll open a PR for NDT OMP with all the corrections made at PointCloudLibrary/pcl#4180.

from ndt_omp.

@koide3 what's the status of this? Looks like that PR is in so it should be ready?

from ndt_omp.

@SteveMacenski
I made a PR for the multi-threaded NDT, and probably it will be merged after a few review steps PointCloudLibrary/pcl#4277
I'll inform you here once it's merged

from ndt_omp.