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

Franka sim #110

Merged
merged 7 commits into from
Oct 12, 2020
Merged

Franka sim #110

merged 7 commits into from
Oct 12, 2020

Conversation

dinies
Copy link
Contributor

@dinies dinies commented Oct 8, 2020

Franka simulation example

added :

  • urdf model of Franka robot
  • meshes related to the model
  • example file franka.cpp to simulate the robot

notes
- for the time being, the robot stays still.
- there an additional blank line on the iiwa.cpp that I forgot to revert before pushing the changes

wscript Outdated Show resolved Hide resolved
@costashatz
Copy link
Member

@dinies thanks for this PR.

@jbmouret @dinies I updated the URDF to one with the hand. Also this one that I had has completely different masses than the one you had. We should use one URDF that we know it makes sense.

Do you want to create 2 URDFs? One with the hand and one without?

Note: Ignore the weird shading of the first 2 links. I am creating a different PR to handle this.

@costashatz costashatz added this to the RobotDART 1.0.0 milestone Oct 10, 2020
@jbmouret
Copy link
Collaborator

We need the gripper, so no need for a second URDF.

We can merge for now if you like. I still want to ask around for "the best URDF" according to our colleagues, but we can change the URDF later.

@costashatz
Copy link
Member

Merging...

@costashatz costashatz merged commit dabc362 into master Oct 12, 2020
@costashatz costashatz deleted the franka_sim branch October 12, 2020 19:40
costashatz added a commit that referenced this pull request Dec 8, 2021
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.

3 participants