Apply suggestions from code review

Co-authored-by: Sebastian Castro <[email protected]>
moveit · Sep 29, 2023 · a899bed · a899bed
1 parent ecac45a
commit a899bed
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/moveit_ros/moveit_servo/config/servo_parameters.yaml b/moveit_ros/moveit_servo/config/servo_parameters.yaml
@@ -49,7 +49,7 @@ servo:
   active_subgroup: {
     type: string,
     default_value: "",
-    description: "This parameter can be used to switch online to actuating a sub-joint group of the move group with servo. \
+    description: "This parameter can be used to switch online to actuating a subgroup of the move group. \
     If it is empty, the full move group is actuated."
   }
 

diff --git a/moveit_ros/moveit_servo/src/utils/command.cpp b/moveit_ros/moveit_servo/src/utils/command.cpp
@@ -275,7 +275,7 @@ JointDeltaResult jointDeltaFromIK(const Eigen::VectorXd& cartesian_position_delt
 
   if (!servo_params.active_subgroup.empty())
   {
-    // Create full delta vector and add delta theta's at actuated joints
+    // Create full delta vector and add delta theta values only at actuated joints
     const auto& move_group_joint_names =
         robot_state->getJointModelGroup(servo_params.move_group_name)->getActiveJointModelNames();
     const auto& subgroup_joint_names =