Fix pclomp::GICP for humble#82
Conversation
Signed-off-by: Amadeusz Szymko <amadeusz.szymko.2@tier4.jp>
|
@MaximSmolskiy thank you for this PR! In 84d5c81 I added Regarding #73:
@SakodaShintaro could you please take a look on this PR? |
|
(cc. @YamatoAndo ) Thank you for the pull request. Please let me confirm the background of this PR. Currently, Also, this autowarefoundation/autoware#5390 Is it necessary to make this change in this repository instead of |
|
@SakodaShintaro Can we do both? 🙏🏻 This PR does not break Humble distro. We use this repo in https://github.com/tier4/CalibrationTools/ |
SakodaShintaro
left a comment
There was a problem hiding this comment.
These changes look good to me.
In the future, could you consider switching the dependency of CalibrationTools from ndt_omp to autoware_core?
|
@SakodaShintaro I can't answer for now. First we will check if we can make CalibrationTools autoware*-agnostic. If so, we might prefer to use standalone Thank you for your review! 🙇🏻♂️ |
Fix #81
Ported important changes from PointCloudLibrary/pcl#3854
After these changes:
ROS_DISTRO=humble