Skip to content

Commit

Permalink
fix hand controller and missing parameter.
Browse files Browse the repository at this point in the history
Removed unimplemented controller instantiation in launcher.
These will have to be reimplemented and fine-tuned.
Added missing use_full_hand_model parameter for Vizzy simulation launcher.
  • Loading branch information
joao-borrego committed Apr 17, 2018
1 parent 6f9609e commit d9c0f50
Show file tree
Hide file tree
Showing 6 changed files with 44 additions and 25 deletions.
51 changes: 27 additions & 24 deletions vizzy_control/config/vizzy_control.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,30 +15,33 @@ vizzy:
pid: {p: 100.0, i: 0.0, d: 0.0}

# Right Hand Position Controllers
r_min_phal_1_position_controller:
type: velocity_controllers/JointPositionController
joint: r_min_phal_1_joint
pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

r_med_phal_1_position_controller:
type: velocity_controllers/JointPositionController
joint: r_med_phal_1_joint
pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

r_ind_phal_1_position_controller:
type: velocity_controllers/JointPositionController
joint: r_ind_phal_1_joint
pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

r_thumb_phal_1_position_controller:
type: velocity_controllers/JointPositionController
joint: r_thumb_phal_1_joint
pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

r_thumb_phal_2_position_controller:
type: velocity_controllers/JointPositionController
joint: r_thumb_phal_2_joint
pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}
#
# TODO: Implement controller after hand update
#
# r_min_phal_1_position_controller:
# type: velocity_controllers/JointPositionController
# joint: r_min_phal_1_joint
# pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

# r_med_phal_1_position_controller:
# type: velocity_controllers/JointPositionController
# joint: r_med_phal_1_joint
# pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

# r_ind_phal_1_position_controller:
# type: velocity_controllers/JointPositionController
# joint: r_ind_phal_1_joint
# pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

# r_thumb_phal_1_position_controller:
# type: velocity_controllers/JointPositionController
# joint: r_thumb_phal_1_joint
# pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

# r_thumb_phal_2_position_controller:
# type: velocity_controllers/JointPositionController
# joint: r_thumb_phal_2_joint
# pid: {p: 100.0, i: 0.0, d: 0.0, i_clamp: 0.0}

# Head Position Controllers
neck_pan_position_controller:
Expand Down
8 changes: 7 additions & 1 deletion vizzy_control/launch/vizzy_control.launch
Original file line number Diff line number Diff line change
Expand Up @@ -53,12 +53,18 @@
l_eye_position_controller
r_eye_position_controller
left_arm_trajectory_controller
right_arm_trajectory_controller
right_arm_trajectory_controller"/>

<!--
TODO: Implement controller after hand update
-->
<!--
r_min_phal_1_position_controller
r_med_phal_1_position_controller
r_ind_phal_1_position_controller
r_thumb_phal_1_position_controller
r_thumb_phal_2_position_controller"/>
-->

</group>

Expand Down
2 changes: 2 additions & 0 deletions vizzy_description/robots/vizzy.urdf.xacro
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,8 @@
<!-- Load Yarp plugins for real robot execution -->
<xacro:arg name="use_yarp" default="false"/>

<!-- Load full inertial and collision model or a simplified model -->
<xacro:arg name="use_full_gazebo_model" default="false"/>
<!-- Load a fully detailed hand model with properly modeled links and joints -->
<xacro:arg name="use_full_hand_model" default="false"/>
<!-- Disable laser sensors -->
Expand Down
3 changes: 3 additions & 0 deletions vizzy_gazebo/launch/gazebo.launch
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@
<arg name="use_sim_time" default="true"/>
<!-- Load full inertial and collision model or a simplified model -->
<arg name="use_full_gazebo_model" default="false"/>
<!-- Load a fully detailed hand model with properly modeled links and joints -->
<arg name="use_full_hand_model" default="false"/>

<!-- Map -->
<arg name="world" default="empty.world"/>
Expand Down Expand Up @@ -46,6 +48,7 @@
<arg name="use_yarp" value="$(arg use_yarp)"/>
<arg name="urdf_file" value="$(arg urdf_file)"/>
<arg name="use_full_gazebo_model" value="$(arg use_full_gazebo_model)"/>
<arg name="use_full_hand_model" value="$(arg use_full_hand_model)"/>
<arg name="disable_laser" value="$(arg disable_laser)"/>
<arg name="disable_3d_sensor" value="$(arg disable_3d_sensor)"/>
</include>
Expand Down
2 changes: 2 additions & 0 deletions vizzy_gazebo/launch/vizzy_spawn.launch
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,7 @@
<arg name="urdf_file" default="vizzy.urdf.xacro"/>
<arg name="use_yarp" default="false"/>
<arg name="use_full_gazebo_model" default="false"/>
<arg name="use_full_hand_model" default="false"/>

<arg name="disable_laser" default="false"/>
<arg name="disable_3d_sensor" default="false"/>
Expand All @@ -14,6 +15,7 @@
<param name="robot_description" command="$(find xacro)/xacro --inorder $(find vizzy_description)/robots/$(arg urdf_file)
use_yarp:=$(arg use_yarp)
use_full_gazebo_model:=$(arg use_full_gazebo_model)
use_full_hand_model:=$(arg use_full_hand_model)
disable_laser:=$(arg disable_laser)
disable_3d_sensor:=$(arg disable_3d_sensor)"/>

Expand Down
3 changes: 3 additions & 0 deletions vizzy_launch/launch/vizzy_simulation.launch
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,8 @@
<arg name="use_sim_time" default="true"/>
<!-- Load full inertial and collision model or a simplified model -->
<arg name="use_full_gazebo_model" default="false"/>
<!-- Load a fully detailed hand model with properly modeled links and joints -->
<arg name="use_full_hand_model" default="false"/>

<!-- rviz -->
<arg name="rviz" default="true"/>
Expand Down Expand Up @@ -88,6 +90,7 @@
<arg name="use_sim_time" value="$(arg use_sim_time)"/>
<arg name="world" value="$(arg world)"/>
<arg name="use_full_gazebo_model" value="$(arg use_full_gazebo_model)"/>
<arg name="use_full_hand_model" value="$(arg use_full_hand_model)"/>
<arg name="disable_laser" value="$(arg disable_laser)"/>
<arg name="disable_3d_sensor" value="$(arg disable_3d_sensor)"/>
</include>
Expand Down

0 comments on commit d9c0f50

Please sign in to comment.