elevation_mapping icon indicating copy to clipboard operation
elevation_mapping copied to clipboard

Feature/filtered elevation layer

Open chisarie opened this issue 7 years ago • 7 comments

A plugin for [costmap_2d]](http://wiki.ros.org/costmap_2d). It takes the elevation map as input and generates a layer of obstacles in the local costmap_2d.

chisarie avatar Nov 05 '18 08:11 chisarie

I don't think elevation_layer is a good name for this package. Since it's a plugin for costmap_2d, this should be somehow clear. What are the names of other costmap_2d plugins?

pfankhauser avatar Dec 04 '18 13:12 pfankhauser

I don't think elevation_layer is a good name for this package. Since it's a plugin for costmap_2d, this should be somehow clear. What are the names of other costmap_2d plugins?

The other names are inflation_layer, obstacle_layer, static_layer, voxel_layer. https://github.com/ANYbotics/navigation_ros/tree/master/costmap_2d/plugins

chisarie avatar Dec 04 '18 15:12 chisarie

@pfankhauser Hi, addressed the comments you wrote, i hope i didn't miss anything.

chisarie avatar Dec 10 '18 08:12 chisarie

@harmishhk @hogabrie @pfankhauser Since we will not use costmap_2d, this is not needed anymore, right? Or Peter, do you want it merged anyway? Otherwise it can be closed.

chisarie avatar Mar 11 '19 13:03 chisarie

We will not use costmap_2d in the future, but we are using at the moment, and I also think this could be useful for community if we merge. Moreover, the functionality of filtering at specific height will be needed also with grid_map based cost-map that we are going to use in the future.

harmishhk avatar Mar 11 '19 14:03 harmishhk

@harmishhk and @pfankhauser what should we do about this PR? Is @chisarie working on the changes?

maximilianwulf avatar Apr 23 '19 06:04 maximilianwulf

@harmishhk and @pfankhauser what should we do about this PR? Is @chisarie working on the changes?

@chisarie is not working on this any more. There are some changes needed for this PR before we can merge it, as long as nobody needs it, I suggest to leave it in this branch for later reference.

pfankhauser avatar Apr 23 '19 07:04 pfankhauser