Skip to content

Commit 1d745ec

Browse files
committed
fixing path longer on approach
Signed-off-by: Pradheep <padhupradheep@gmail.com>
1 parent 3f31112 commit 1d745ec

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

‎nav2_behavior_tree/plugins/decorator/path_longer_on_approach.cpp‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,9 @@ bool PathLongerOnApproach::isPathUpdated(
3434
nav_msgs::msg::Path & new_path,
3535
nav_msgs::msg::Path & old_path)
3636
{
37-
return new_path != old_path && old_path.poses.size() != 0 &&
38-
new_path.poses.size() != 0 &&
39-
old_path.poses.back().pose == new_path.poses.back().pose;
37+
return old_path.poses.size() != 0 &&
38+
new_path.poses.size() != 0 && new_path.poses.size() > old_path.poses.size() &&
39+
old_path.poses.back().pose.position == new_path.poses.back().pose.position;
4040
}
4141

4242
bool PathLongerOnApproach::isRobotInGoalProximity(
@@ -64,7 +64,7 @@ inline BT::NodeStatus PathLongerOnApproach::tick()
6464

6565
if (first_time_ == false) {
6666
if (old_path_.poses.empty() || new_path_.poses.empty() ||
67-
old_path_.poses.back() != new_path_.poses.back())
67+
old_path_.poses.back().pose != new_path_.poses.back().pose)
6868
{
6969
first_time_ = true;
7070
}

0 commit comments

Comments
 (0)