Skip to content

Commit 6a12bb7

Browse files
committed
bug fixes for real2sim and sim2real
1 parent 6c15f67 commit 6a12bb7

File tree

3 files changed

+20
-12
lines changed

3 files changed

+20
-12
lines changed

README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,3 +3,8 @@
33
The robot model here is based on the real2sim project: https://github.com/simpler-env/SimplerEnv
44

55

6+
## Changes Made:
7+
8+
- Added a .srdf file for SAPIEN to parse to avoid self collisions for better/faster simulation
9+
- Removed some collision meshes that were unnecessary (e.g. inside another collision mesh)
10+
- Removed mimic joint tags as they are not used and are modelled by the ManiSkill defined controllers

wx250s.srdf

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
<robot name="wx250s">
2+
<disable_collisions link1="base_link" link2="upper_arm_link" reason="Default"/>
3+
<disable_collisions link1="gripper_link" link2="left_finger_link" reason="Default"/>
4+
<disable_collisions link1="gripper_link" link2="right_finger_link" reason="Default"/>
5+
6+
<disable_collisions link1="gripper_bar_link" link2="gripper_link" reason="Default"/>
7+
<disable_collisions link1="gripper_bar_link" link2="left_finger_link" reason="Default"/>
8+
<disable_collisions link1="gripper_bar_link" link2="right_finger_link" reason="Default"/>
9+
10+
<disable_collisions link1="left_finger_link" link2="right_finger_link" reason="Default"/>
11+
</robot>

wx250s.urdf

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -252,28 +252,20 @@
252252
<parent link="ee_arm_link"/>
253253
<child link="gripper_prop_link"/>
254254
</joint>
255-
<!-- <transmission name="trans_gripper">
256-
<type>transmission_interface/SimpleTransmission</type>
257-
<joint name="gripper">
258-
<hardwareInterface>hardware_interface/EffortJointInterface</hardwareInterface>
259-
</joint>
260-
<actuator name="gripper_motor">
261-
<mechanicalReduction>1</mechanicalReduction>
262-
</actuator>
263-
</transmission> -->
264255
<link name="gripper_prop_link">
265256
<visual>
266257
<origin rpy="0 0 1.5707963267948966" xyz="-0.0685 0 0"/>
267258
<geometry>
268259
<mesh filename="meshes_wx250s/WXSA-250-M-8-Gripper-Prop.obj" scale="0.001 0.001 0.001"/>
269260
</geometry>
270261
</visual>
271-
<collision>
262+
<!-- stao: No need to model this collision, it is inside gripper_link -->
263+
<!-- <collision>
272264
<origin rpy="0 0 1.5707963267948966" xyz="-0.0685 0 0"/>
273265
<geometry>
274266
<mesh filename="meshes_wx250s/WXSA-250-M-8-Gripper-Prop.stl" scale="0.001 0.001 0.001"/>
275267
</geometry>
276-
</collision>
268+
</collision> -->
277269
<inertial>
278270
<origin rpy="0 0 1.5707963267948966" xyz="0.0008460000 -0.0000016817 0.0000420000"/>
279271
<mass value="0.00434"/>
@@ -360,7 +352,7 @@
360352
<origin rpy="0 0 0" xyz="0 0 0"/>
361353
<parent link="fingers_link"/>
362354
<child link="right_finger_link"/>
363-
<mimic joint="left_finger" multiplier="1" offset="0"/>
355+
<!-- <mimic joint="left_finger" multiplier="1" offset="0"/> -->
364356
</joint>
365357
<!-- <transmission name="trans_right_finger">
366358
<type>transmission_interface/SimpleTransmission</type>

0 commit comments

Comments
 (0)