Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes hand real/gazebo discrepacy #155

Conversation

rickstaa
Copy link
Contributor

This commit makes sure that the hand joint control behaviour between the real and simulated robot is equal. Previously the 'franka_finger_joint2' mimics the 'franka_finger_joint1' in the real robot but not in the simulation.

I think having the same behaviour in both the simulation and on the real robot makes sense since it otherwise might causes errors in third party packages controlling the robot.

This commit makes sure that the hand joint control behavoir between the
real and simulated robot is equal. Previously the 'franka_finger_joint2'
mimics the 'franka_finger_joint1' in the real robot but not in the
simulation.
@rickstaa rickstaa force-pushed the fixes_hand_gazebo_real_discrepancy branch from 7b53cd4 to bdc0c31 Compare August 23, 2021 12:30
@rickstaa
Copy link
Contributor Author

This appears to break the panda_moveit_config package. I will investigate if this change can be applied while using MoveIt.

@rickstaa
Copy link
Contributor Author

rickstaa commented Aug 25, 2021

Closed, since adding the mimic tag to the gazebo urdf is currently not supported (see ros-simulation/gazebo_ros_pkgs#251).

@rickstaa rickstaa closed this Aug 25, 2021
@rickstaa rickstaa deleted the fixes_hand_gazebo_real_discrepancy branch November 9, 2021 14:46
gollth added a commit that referenced this pull request Nov 23, 2021
* commit 'b7a8cb1bcbca55fedbca0d215ce304f30cd9cb7b':
  increase grasp counter to fix going into holding state at low speeds
  CHANGE: Refactor finger collision models to macro
  ADD: Changelog entry
  FIX: Tune parameter for fingers to grasp successful in franka_gazebo
  FIX: Also apply forces in correct direction for "opening" grasps
  CHANGE: Finger collision meshes to primitive boxes
  FIX: Add slight offset between fingers in initial positions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant