search

moveit / moveit_ros

THIS REPO HAS MOVED TO https://github.com/ros-planning/moveit
70 stars 118 forks source link

Add nodehandle for specific params #707

Closed chengshy closed 8 years ago

chengshy commented 8 years ago

Add an new constructor for ocuupancy_map_monitor to read from specifc param files. Which could be very helpful if we want to maintain several octomap_monitors

v4hn commented 8 years ago

+1 indigo onward. There is no reason to hard-code the NodeHandle here. This seems to be a common problem in MoveIt! as we just merged #701 for the CurrentStateMonitor.

jonbinney commented 8 years ago

+1

v4hn commented 8 years ago

Thanks for the request! cherry-picked to indigo and kinetic

davetcoleman commented 8 years ago

@jonbinney thanks for the +1 - we need more reviewers!