Skip to content

Commit d05ba1e

Browse files
committed
PipelinePlanner: Reject approximate solutions
1 parent 5b6287a commit d05ba1e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

core/src/solvers/pipeline_planner.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ bool PipelinePlanner::plan(const planning_scene::PlanningSceneConstPtr& from,
181181
::planning_interface::MotionPlanResponse res;
182182
bool success = planner_->generatePlan(from, req, res);
183183
result = res.trajectory_;
184-
return success;
184+
return success && res.error_code_.val == moveit_msgs::msg::MoveItErrorCodes::SUCCESS;
185185
}
186186

187187
bool PipelinePlanner::plan(const planning_scene::PlanningSceneConstPtr& from, const moveit::core::LinkModel& link,
@@ -206,7 +206,7 @@ bool PipelinePlanner::plan(const planning_scene::PlanningSceneConstPtr& from, co
206206
::planning_interface::MotionPlanResponse res;
207207
bool success = planner_->generatePlan(from, req, res);
208208
result = res.trajectory_;
209-
return success;
209+
return success && res.error_code_.val == moveit_msgs::msg::MoveItErrorCodes::SUCCESS;
210210
}
211211
} // namespace solvers
212212
} // namespace task_constructor

0 commit comments

Comments
 (0)