ros-drivers / velodyne

ROS support for Velodyne 3D LIDARs
http://ros.org/wiki/velodyne
Other
646 stars 643 forks source link

Velodyne pcl (#335) #446

Closed wep21 closed 1 year ago

wep21 commented 2 years ago

ros2 porting of #335 A part of https://github.com/ros-drivers/velodyne/pull/434

wep21 commented 2 years ago

cc @clalancette @JWhitleyWork

JWhitleyWork commented 2 years ago

@wep21 Myself and @spuetz decided to leave this out of the ROS2 version as the PCL dependency is very heavy and that also depends on all of Boost, which is also very heavy. Is there a particular reason you need this to be in the driver?

wep21 commented 2 years ago

@JWhitleyWork There is no particular reason and this is a utility function for external use, so it is possible to proceed to sync master #434 without this PR.