dart icon indicating copy to clipboard operation
dart copied to clipboard

Make VoxelGridShape as optional component

Open jslee02 opened this issue 7 years ago • 0 comments

https://github.com/dartsim/dart/pull/1076#pullrequestreview-134244152:

It would be nice if we could restructure that relationship somehow. For example, have a dart-shapes library that contains the default dependency-free shapes, and then dart-shapes-voxel could be a component that adds the VoxelGridShape class and has an octomap dependency.

We'd have to make drastic changes to the implementations of the collision detection wrappers, so that shapes can inform the collision detector about how to convert its data from the DART representation to the underlying collision detector's representation. That would have the additional benefit of allowing us to support user-defined shape types without the user needing to make any changes to the DART codebase.

jslee02 avatar Jul 08 '18 03:07 jslee02