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

jaco_description lacks package.xml #13624

Closed
ggould-tri opened this issue Jul 1, 2020 · 3 comments
Closed

jaco_description lacks package.xml #13624

ggould-tri opened this issue Jul 1, 2020 · 3 comments

Comments

@ggould-tri
Copy link
Contributor

Most of the manipulation models subdirectories have package.xml files to allow ROS-style access via the package:// URI scheme.

ggould@tars:~/git/drake$ ls manipulation/models/*/package.xml
manipulation/models/allegro_hand_description/package.xml
manipulation/models/franka_description/package.xml
manipulation/models/iiwa_description/package.xml
manipulation/models/wsg_50_description/package.xml

Note that the jaco_description (and ycb) subdirs are not on that list.

This required a fairly ugly workaround in a downstream project.

FYI @sammy-tri @EricCousineau-TRI

@jwnimmer-tri
Copy link
Collaborator

jwnimmer-tri commented Jul 1, 2020

For the record, we have discussed removing all of those package.xml files in lieu of a single package.xml at the root of Drake (#10531). But until we make that change elsewhere, we should be consistent for the jaco.

@jwnimmer-tri
Copy link
Collaborator

Fixed by #14134 perhaps?

@EricCousineau-TRI
Copy link
Contributor

Oh yeah, oops!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants