search

LCAS / topological_navigation

The topological navigation framework
Apache License 2.0
31 stars 36 forks source link

Fixed move_base rosparam missing for topological navigation #65

Closed pulver22 closed 1 year ago

pulver22 commented 3 years ago

Hi, the current code is not setting the correct move_base rosparam required for navigating between waypoints. The navigation does work if the robot uses DWAPlannerROS as planner but it doesn't if, for example, we are using TrajectoryPlannerROS. I checked that this has been fixed in the Rasberry code but not in the current package.

ayu135 commented 3 years ago

We are planning on fixing this with the upcoming toponav2 related changes, it will change things in a major way on how the execute policy and nav actions are handled internally. I will hold off on merging this for now till the toponav 2 changes are merged as that should fix this problem.

marc-hanheide commented 1 year ago

I judge this to be deprecarted due to the move to toponav2. I'll close. Please reopen @pulver22 if still needed.