Markus Vieth

I can soon start working on making nanoflann a full replacement for FLANN in PCL, so that FLANN is optional and no PCL functionality is lost when FLANN is not...

> The error is `#error: error STL1003: Unexpected compiler, expected C++ compiler.`, which is thrown in file `yvals_core.h`. This file is provided by Visual Studio and contains: > > ```...

For your information: I started working on making nanoflann a full replacement for FLANN, but there is still a lot more work needed: https://github.com/PointCloudLibrary/pcl/compare/master...mvieth:pcl:nanoflann_default

@larshg I am thinking about adding a new function: ```c++ template Search * autoSelectMethod(const typename pcl::PointCloud::ConstPtr& cloud, bool sorted_results, std::uint32_t purpose=0); ``` which would select the fastest search method for...

> The current base class Registration seems to use an ICP algorithm by default. The algorithm is roughly divided into correspondence estimation and transformation estimation. Some rough alignment algorithms do...

Hi, your desire to customize the centroid computation is totally understandable. Looking at [the way that `CentroidPoint` is currently used in `VoxelGrid`](https://github.com/PointCloudLibrary/pcl/blob/master/filters/include/pcl/filters/impl/voxel_grid.hpp#L421), I guess it would be necessary to add...

> Your alternative idea sounds feasible, indeed. But wouldn't the signature of this function would become a bit clunky? Because you'll have to pass in `input_`, `index_vector`, `first_index` and `last_index`,...

Thank you for your pull request. Before you work further on this, let's discuss first for which parts of the code it makes the most sense to put it into...

> move (h|H)as_* traits into common/include/pcl/(field_traits.h|impl/field_traits.hpp) (alternative 2) Sounds good. In my opinion, the division into .h and .hpp files do not have to be strictly followed for new files,...

@larshg @UnaNancyOwen Can you confirm? I am not familiar enough with Windows and the dll/lib system