More changed functions and add re-planning

Author: eholum-nasa

clr_pick_and_place_demo/src/run_demo.cpp

@@ -760,38 +760,37 @@ class RunDemoNode : public rclcpp::Node
     {
       move_group->setJointValueTarget(move_group->getNamedTargetValues(waypoint.preset_name));
     }
-    if (waypoint.use_jconfig)
+    else if (waypoint.use_jconfig)
     {
       move_group->setJointValueTarget(waypoint.config);
-      move_group->setNumPlanningAttempts(10);
     }
     else
     {
-      geometry_msgs::msg::Pose start_pose = move_group->getCurrentPose().pose;
       geometry_msgs::msg::Pose end_pose = waypoint.pose;
       if (waypoint.is_relative)
       {
+        geometry_msgs::msg::Pose start_pose = move_group->getCurrentPose().pose;
         end_pose = this->relative_to_global(start_pose, end_pose);
       }
-      move_group->setJointValueTarget(end_pose);
-      move_group->setNumPlanningAttempts(10);
+      move_group->setPoseTarget(end_pose);
     }
 
+    move_group->setNumPlanningAttempts(5);
     move_group->setMaxVelocityScalingFactor(scaling);
     move_group->setMaxAccelerationScalingFactor(scaling);
 
     moveit::planning_interface::MoveGroupInterface::Plan plan;
     moveit::core::MoveItErrorCode error_code = move_group->plan(plan);
 
-    if (error_code == moveit::core::MoveItErrorCode::SUCCESS)
+    if (error_code)
     {
       RCLCPP_INFO(LOGGER, "Successfully computed trajectory");
       trajectory = plan.trajectory;
       return true;
     }
     else
     {
-      RCLCPP_ERROR(LOGGER, "Planning failed with error code %s", error_code.message.c_str());
+      RCLCPP_ERROR(LOGGER, "Planning failed: %s", error_code.message.c_str());
       return false;
     }
   }