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

Fix masses of robot links #689

Open
wants to merge 2 commits into
base: noetic-devel
Choose a base branch
from

Conversation

VinDp
Copy link
Collaborator

@VinDp VinDp commented Oct 30, 2024

Changed the masses to match the actual specifications, as done for ROS2 in #187.

@VinDp VinDp requested a review from fmauch October 30, 2024 15:09
@VinDp
Copy link
Collaborator Author

VinDp commented Nov 27, 2024

I also added the changes from #195 to be on pair with the ROS 2 version.
While doing so I also noticed that there were some parts that could maybe be cleaned up, like in physical_parameters.yaml of the ur30. Should I include that in this PR or is it better to leave it for a different one?

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