search

tue-robotics / ed_navigation

ED interface to the base navigation modules
2 stars 4 forks source link

Move depth sensor integration to costmap_2d #1

Open reinzor opened 8 years ago

reinzor commented 8 years ago

Should be in the costmap_2d obstacle layer so that we can specify the occupied to default time as we do for the laser and the optional kinect. Also the depth sensor integration is not related to the world model.

https://github.com/tue-robotics/navigation/blob/hydro-devel/costmap_2d/plugins/obstacle_layer.cpp

Rayman commented 7 years ago

@reinzor, can this be closed?

MatthijsBurgh commented 7 years ago

I think this issue is related to: https://github.com/tue-robotics/tue_robocup/issues/117 and https://github.com/tue-robotics/navigation/commit/fa7dba58e711cb52c8efde306e9d67146c74246f. This is removed again in https://github.com/tue-robotics/navigation/commit/d7ff6f0ef61c4796123093289361bec764315c10.