François Pomerleau

Tests are failing on OSX ``` [ 72%] Linking CXX executable icp_customized Undefined symbols for architecture x86_64: "YAML::Load(std::__1::basic_istream&)", referenced from: PointMatcher::ICPChainBase::loadFromYaml(std::__1::basic_istream&) in libpointmatcher.a(ICP.cpp.o) PointMatcher::ICPChainBase::loadFromYaml(std::__1::basic_istream&) in libpointmatcher.a(ICP.cpp.o) PointMatcher::DataPointsFilters::DataPointsFilters(std::__1::basic_istream&) in libpointmatcher.a(DataPointsFilter.cpp.o) PointMatcher::DataPointsFilters::DataPointsFilters(std::__1::basic_istream&)...

Right now, those are the only supported formats: - csv (Comma Separated Values) - vtk (Visualization Toolkit Files) - ply (Polygon File Format) - pcd (Point Cloud Library Format) New...

ok to test

Seems to be a configuration on the Jenkins server. I'm not admin on that server, I'll have to find who's taking care of it at ETH. ``` Cloning into '/home/jenkins/workspace/pointmatcher/label/ubuntu-xenial/libnabo'......

@kubelvla is that PR ready for a merge?

Thanks for letting us know! That's indeed troublesome as we are using it also in some core applications. Indeed, if @MathLabu have an inside, it would be great.

@YoshuaNava , that's a great idea. Most of the time, we need visual cue to debug those things.

I'm glad to hear that. We are close to need this filter on a critical deployment and I was getting nervous about this bug.

Awesome @YoshuaNava ! I'm very happy that you took the time to investigate that. @simonpierredeschenes will have a look shortly to make sure we can close that issue relatively soon....

No, we didn't have time to test it yet. We will validate the fix most probably after the PR.