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

Mass of UR5e are incorrect #186

Closed
olufskovnielsen opened this issue Aug 22, 2024 · 1 comment · Fixed by #187
Closed

Mass of UR5e are incorrect #186

olufskovnielsen opened this issue Aug 22, 2024 · 1 comment · Fixed by #187
Assignees
Labels
bug Something isn't working

Comments

@olufskovnielsen
Copy link
Collaborator

Masses of UR5e are incorrect https://github.com/UniversalRobots/Universal_Robots_ROS2_Description/blob/rolling/config/ur5e/physical_parameters.yaml
They should be based on this https://www.universal-robots.com/articles/ur/application-installation/dh-parameters-for-calculations-of-kinematics-and-dynamics
The current values are copied from UR5 CB3

@fmauch fmauch self-assigned this Sep 4, 2024
@fmauch
Copy link
Contributor

fmauch commented Sep 4, 2024

It looks like you are absolutely correct! Thanks for pointing that out.

Just for reference: This needs to be fixed for ROS 1, as well.

@fmauch fmauch added the bug Something isn't working label Sep 4, 2024
@fmauch fmauch linked a pull request Sep 4, 2024 that will close this issue
@VinDp VinDp closed this as completed in #187 Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants