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

[WUD-538] Add spot_arm URDF file #528

Closed
wants to merge 1 commit into from

Conversation

tcappellari-bdai
Copy link
Collaborator

Change Overview

a URDF (instead of SRDF or xacro) file is needed to compute the arm jacobian for ticket WUD-538. This PR adds it to spot_description.

Testing Done

N/A

Please create a checklist of tests you plan to do and check off the ones that have been completed successfully. Ensure that ROS 2 tests use domain_coordinator to prevent port conflicts. Further guidance for testing can be found on the ros utilities wiki.

Copy link
Collaborator

possibly stupid question but shouldn't you be able to use https://github.com/bdaiinstitute/spot_ros2/blob/main/spot_description/urdf/standalone_arm.urdf.xacro? That's what teleop uses for its arm related pinnochio stuff

@tcappellari-bdai
Copy link
Collaborator Author

possibly stupid question but shouldn't you be able to use https://github.com/bdaiinstitute/spot_ros2/blob/main/spot_description/urdf/standalone_arm.urdf.xacro? That's what teleop uses for its arm related pinnochio stuff

Is there a way to convert it into an acceptable file for pinocchio? It kept saying it wasn't in proper URDF format and I couldn't figure out how to get it to play nice

Copy link
Collaborator

it should be possible, I'll try to find the code from teleop and message you!

Copy link
Collaborator

@amessing-bdai amessing-bdai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Blocking this until we get an answer about can we use the existing urdf or modify it. Having two of these will be confusing.

@tcappellari-bdai
Copy link
Collaborator Author

ya nevermind i was being dumb lol. figured it out

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