Skip to content

Commit 5f46662

Browse files
author
Dan Zimmerman
committed
Revert joint dynamics changes
1 parent b6efb97 commit 5f46662

10 files changed

+8
-193
lines changed

config/ur10/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur10e/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur16e/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur3/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur3e/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur5/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

config/ur5e/joint_dynamics_parameters.yaml

Lines changed: 0 additions & 24 deletions
This file was deleted.

urdf/inc/ur_common.xacro

Lines changed: 1 addition & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -41,15 +41,14 @@
4141
<mesh filename="${mesh}"/>
4242
</xacro:macro>
4343

44-
<xacro:macro name="read_model_data" params="joint_limits_parameters_file kinematics_parameters_file physical_parameters_file visual_parameters_file joint_dynamics_parameters_file force_abs_paths">
44+
<xacro:macro name="read_model_data" params="joint_limits_parameters_file kinematics_parameters_file physical_parameters_file visual_parameters_file force_abs_paths">
4545

4646
<xacro:property name="force_abs_paths" value="${force_abs_paths}" scope="parent"/>
4747
<!-- Read .yaml files from disk, load content into properties -->
4848
<xacro:property name="config_joint_limit_parameters" value="${xacro.load_yaml(joint_limits_parameters_file)}"/>
4949
<xacro:property name="config_kinematics_parameters" value="${xacro.load_yaml(kinematics_parameters_file)}"/>
5050
<xacro:property name="config_physical_parameters" value="${xacro.load_yaml(physical_parameters_file)}"/>
5151
<xacro:property name="config_visual_parameters" value="${xacro.load_yaml(visual_parameters_file)}"/>
52-
<xacro:property name="config_joint_dynamics" value="${xacro.load_yaml(joint_dynamics_parameters_file)}"/>
5352

5453
<!-- Extract subsections from yaml dictionaries -->
5554
<xacro:property name="sec_limits" value="${config_joint_limit_parameters['joint_limits']}"/>
@@ -58,7 +57,6 @@
5857
<xacro:property name="sec_inertia_parameters" value="${config_physical_parameters['inertia_parameters']}" />
5958
<xacro:property name="sec_mesh_files" value="${config_visual_parameters['mesh_files']}" scope="parent"/>
6059
<xacro:property name="sec_kinematics" value="${config_kinematics_parameters['kinematics']}" />
61-
<xacro:property name="sec_joint_dynamics" value="${config_joint_dynamics['joint_dynamics']}" />
6260

6361
<!-- JOINTS LIMIT PARAMETERS -->
6462
<xacro:property name="shoulder_pan_lower_limit" value="${sec_limits['shoulder_pan_joint']['min_position']}" scope="parent"/>
@@ -86,20 +84,6 @@
8684
<xacro:property name="wrist_3_velocity_limit" value="${sec_limits['wrist_3_joint']['max_velocity']}" scope="parent"/>
8785
<xacro:property name="wrist_3_effort_limit" value="${sec_limits['wrist_3_joint']['max_effort']}" scope="parent"/>
8886

89-
<!-- JOINT DYNAMICS PARAMETERS -->
90-
<xacro:property name="shoulder_pan_damping" value="${sec_joint_dynamics['shoulder_pan_joint']['damping']}" scope="parent"/>
91-
<xacro:property name="shoulder_pan_friction" value="${sec_joint_dynamics['shoulder_pan_joint']['friction']}" scope="parent"/>
92-
<xacro:property name="shoulder_lift_damping" value="${sec_joint_dynamics['shoulder_lift_joint']['damping']}" scope="parent"/>
93-
<xacro:property name="shoulder_lift_friction" value="${sec_joint_dynamics['shoulder_lift_joint']['friction']}" scope="parent"/>
94-
<xacro:property name="elbow_joint_damping" value="${sec_joint_dynamics['elbow_joint']['damping']}" scope="parent"/>
95-
<xacro:property name="elbow_joint_friction" value="${sec_joint_dynamics['elbow_joint']['friction']}" scope="parent"/>
96-
<xacro:property name="wrist_1_damping" value="${sec_joint_dynamics['wrist_1_joint']['damping']}" scope="parent"/>
97-
<xacro:property name="wrist_1_friction" value="${sec_joint_dynamics['wrist_1_joint']['friction']}" scope="parent"/>
98-
<xacro:property name="wrist_2_damping" value="${sec_joint_dynamics['wrist_2_joint']['damping']}" scope="parent"/>
99-
<xacro:property name="wrist_2_friction" value="${sec_joint_dynamics['wrist_2_joint']['friction']}" scope="parent"/>
100-
<xacro:property name="wrist_3_damping" value="${sec_joint_dynamics['wrist_3_joint']['damping']}" scope="parent"/>
101-
<xacro:property name="wrist_3_friction" value="${sec_joint_dynamics['wrist_3_joint']['friction']}" scope="parent"/>
102-
10387
<!-- DH PARAMETERS -->
10488
<xacro:property name="d1" value="${sec_dh_parameters['d1']}" scope="parent"/>
10589
<xacro:property name="a2" value="${sec_dh_parameters['a2']}" scope="parent"/>

urdf/ur.urdf.xacro

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
<xacro:arg name="kinematics_params" default="$(find ur_description)/config/$(arg ur_type)/default_kinematics.yaml"/>
2020
<xacro:arg name="physical_params" default="$(find ur_description)/config/$(arg ur_type)/physical_parameters.yaml"/>
2121
<xacro:arg name="visual_params" default="$(find ur_description)/config/$(arg ur_type)/visual_parameters.yaml"/>
22-
<xacro:arg name="joint_dynamics_params" default="$(find ur_description)/config/$(arg ur_type)/joint_dynamics_parameters.yaml"/>
22+
2323
<xacro:arg name="transmission_hw_interface" default=""/>
2424
<xacro:arg name="safety_limits" default="false"/>
2525
<xacro:arg name="safety_pos_margin" default="0.15"/>
@@ -71,7 +71,6 @@
7171
kinematics_parameters_file="$(arg kinematics_params)"
7272
physical_parameters_file="$(arg physical_params)"
7373
visual_parameters_file="$(arg visual_params)"
74-
joint_dynamics_parameters_file="$(arg joint_dynamics_params)"
7574
force_abs_paths="${sim_gazebo or sim_ignition}"/>
7675

7776
<!-- arm instance -->

urdf/ur_macro.xacro

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@
218218
<xacro:if value="${safety_limits}">
219219
<safety_controller soft_lower_limit="${shoulder_pan_lower_limit + safety_pos_margin}" soft_upper_limit="${shoulder_pan_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
220220
</xacro:if>
221-
<dynamics damping="${shoulder_pan_damping}" friction="${shoulder_pan_friction}"/>
221+
<dynamics damping="0" friction="0"/>
222222
</joint>
223223
<joint name="${prefix}shoulder_lift_joint" type="revolute">
224224
<parent link="${prefix}shoulder_link" />
@@ -230,7 +230,7 @@
230230
<xacro:if value="${safety_limits}">
231231
<safety_controller soft_lower_limit="${shoulder_lift_lower_limit + safety_pos_margin}" soft_upper_limit="${shoulder_lift_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
232232
</xacro:if>
233-
<dynamics damping="${shoulder_lift_damping}" friction="${shoulder_lift_friction}"/>
233+
<dynamics damping="0" friction="0"/>
234234
</joint>
235235
<joint name="${prefix}elbow_joint" type="revolute">
236236
<parent link="${prefix}upper_arm_link" />
@@ -242,7 +242,7 @@
242242
<xacro:if value="${safety_limits}">
243243
<safety_controller soft_lower_limit="${elbow_joint_lower_limit + safety_pos_margin}" soft_upper_limit="${elbow_joint_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
244244
</xacro:if>
245-
<dynamics damping="${elbow_joint_damping}" friction="${elbow_joint_friction}"/>
245+
<dynamics damping="0" friction="0"/>
246246
</joint>
247247
<joint name="${prefix}wrist_1_joint" type="revolute">
248248
<parent link="${prefix}forearm_link" />
@@ -254,7 +254,7 @@
254254
<xacro:if value="${safety_limits}">
255255
<safety_controller soft_lower_limit="${wrist_1_lower_limit + safety_pos_margin}" soft_upper_limit="${wrist_1_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
256256
</xacro:if>
257-
<dynamics damping="${wrist_1_damping}" friction="${wrist_1_friction}"/>
257+
<dynamics damping="0" friction="0"/>
258258
</joint>
259259
<joint name="${prefix}wrist_2_joint" type="revolute">
260260
<parent link="${prefix}wrist_1_link" />
@@ -266,7 +266,7 @@
266266
<xacro:if value="${safety_limits}">
267267
<safety_controller soft_lower_limit="${wrist_2_lower_limit + safety_pos_margin}" soft_upper_limit="${wrist_2_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
268268
</xacro:if>
269-
<dynamics damping="${wrist_2_damping}" friction="${wrist_2_friction}"/>
269+
<dynamics damping="0" friction="0"/>
270270
</joint>
271271
<joint name="${prefix}wrist_3_joint" type="revolute">
272272
<parent link="${prefix}wrist_2_link" />
@@ -278,7 +278,7 @@
278278
<xacro:if value="${safety_limits}">
279279
<safety_controller soft_lower_limit="${wrist_3_lower_limit + safety_pos_margin}" soft_upper_limit="${wrist_3_upper_limit - safety_pos_margin}" k_position="${safety_k_position}" k_velocity="0.0"/>
280280
</xacro:if>
281-
<dynamics damping="${wrist_3_damping}" friction="${wrist_3_friction}"/>
281+
<dynamics damping="0" friction="0"/>
282282
</joint>
283283

284284
<link name="${prefix}ft_frame"/>

0 commit comments

Comments
 (0)