Skip to content

Commit

Permalink
[RotationShimController] rotate also on short paths
Browse files Browse the repository at this point in the history
When the path is shorter than ´forward_sampling_distance´ the
rotatitonShimController would directly give control to the primary
controller to navigate to the goal. This would lead to the exact
behavior that was tried to be fixed by the rotationShimController: "The
result is an awkward, stuttering, or whipping around behavior" [1]. It
often resulted in navigation failure.

In this case, the controller should try to rotate towards the last point
of the path, so that the primary controller can have a better starting
orientation.

[1] https://navigation.ros.org/tuning/index.html#rotate-in-place-behavior
  • Loading branch information
gennartan committed May 2, 2024
1 parent 1f980a1 commit d8ae3c1
Showing 1 changed file with 1 addition and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -195,10 +195,7 @@ geometry_msgs::msg::PoseStamped RotationShimController::getSampledPathPt()
}
}

throw nav2_core::ControllerException(
std::string(
"Unable to find a sampling point at least %0.2f from the robot,"
"passing off to primary controller plugin.", forward_sampling_distance_));
return current_path_.poses.back();
}

geometry_msgs::msg::Pose
Expand Down

0 comments on commit d8ae3c1

Please sign in to comment.