locusrobotics / robot_navigation

Spiritual successor to ros-planning/navigation.
444 stars 149 forks source link

Porting dlux_global_planner to ROS 2 Dashing #45

Open sloretz opened 5 years ago

sloretz commented 5 years ago

I'm looking into what it would take to port dlux_global_planner to ROS 2 Dashing. Hopefully you don't mind if I ask a few questions.

orduno commented 5 years ago

Should a ROS 2 port of dlux_global_planner use nav_core2::GlobalPlanner or nav2_tasks::ComputePathToPose...?

@sloretz Nav2 interfaces with planning and control through ROS2 actions. Dlux would need an action service for nav2_msgs::action::ComputePathToPose. Since we are using behavior trees (BT), nav2_tasks::ComputePathToPose defines a BT action node for getting a plan from an action call.

A few random things to consider:

crdelsey commented 5 years ago

Where should ports of these packages live?

@sloretz I'll leave that up to you and @DLu. If you decide you want to put this in navigation2, we'd be happy to add it and will support you as much as possible. My desire is to integrate this and navigation2 as much as possible, and where not possible, make it easy to swap back and forth between them.