velodyne icon indicating copy to clipboard operation
velodyne copied to clipboard

Containers and PCL point type for backwards compatibility

Open spuetz opened this issue 4 years ago • 12 comments

This is a possible solution for #340 for melodic.

  • It restores the pcl dependency
  • The missing point_type.h is added again.
  • Corresponding containers without a time field are added.
  • The assignment of the time value has been fixed for the organized cloud container, see organized_cloudXYZIRT.cc.

Things to do before merging / releasing

  • [ ] Someone using the driver with the point_type.h should test this.
  • [x] For Noetic, we should remove pcl and add a velodyne_pcl package (#335)
  • [x] Furthermore, we could merge the classes in convert.h and transform.h, since it more or less identical now. (#351)

spuetz avatar Jun 08 '20 17:06 spuetz

Item 3 in your list is handled by #344. Since this functionality is necessary to get a new release, I'll merge this first and have the author of #344 rebase on your changes.

JWhitleyWork avatar Jun 08 '20 19:06 JWhitleyWork

We should have this also in the master for noetic with the option of selecting the right container. But, of course without the point_type.h and the pcl dependencies. Further, we could handle the containers as plugins, using the pluginlib. Then people could use the driver and modify their container, meaning modifying what is in the cloud and how, e.g. how it is transformed, etc. Additionally, for noetic I would remove the config flag for organize_cloud since this is covered by using one of the organized containers.

spuetz avatar Jun 09 '20 08:06 spuetz

@spuetz Can you please address my concerns? After that, I'll see if I can find a package to test with.

JWhitleyWork avatar Jul 09 '20 16:07 JWhitleyWork

@spuetz ping for changes.

JWhitleyWork avatar Jul 17 '20 00:07 JWhitleyWork

@spuetz Any update here?

JWhitleyWork avatar Aug 28 '20 19:08 JWhitleyWork

Sorry for the late reply. I'll have time in two weeks to finish this.

spuetz avatar Sep 03 '20 07:09 spuetz

Thanks, @spuetz!

JWhitleyWork avatar Sep 03 '20 15:09 JWhitleyWork

@spuetz ping

JWhitleyWork avatar Nov 10 '20 00:11 JWhitleyWork

@spuetz Still two items remaining when you can make time.

JWhitleyWork avatar Dec 01 '20 18:12 JWhitleyWork

@spuetz Can we get this wrapped up?

JWhitleyWork avatar Aug 23 '21 18:08 JWhitleyWork

Any update on this? @spuetz @JWhitleyWork

hriihimaki avatar Oct 25 '21 19:10 hriihimaki

Rebased on current melodic-devel. Waiting for @spuetz to complete as I just don't have the background or time.

JWhitleyWork avatar Dec 04 '22 22:12 JWhitleyWork