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

[examples] Move punyo files into drake_models #21231

Merged

Conversation

jwnimmer-tri
Copy link
Collaborator

@jwnimmer-tri jwnimmer-tri commented Apr 1, 2024

Ditch the "models" subdir (moving spatula up one level).

Remove unwanted install rules for these files.

Fix image naming for consistency.

[workspace] Upgrade drake_models to latest commit.

Twin PR: RobotLocomotion/models#53

Towards #13942.


This change is Reviewable

@jwnimmer-tri
Copy link
Collaborator Author

+@DamrongGuoy for feature review, please.

Copy link
Contributor

@DamrongGuoy DamrongGuoy left a comment

Choose a reason for hiding this comment

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

:lgtm:

CC: @joemasterjohn

Reviewed 9 of 9 files at r1, all commit messages.
Reviewable status: 1 unresolved discussion, needs platform reviewer assigned, needs at least two assigned reviewers


examples/hydroelastic/spatula_slip_control/BUILD.bazel line 26 at r1 (raw file):

    data = [
        ":models",
        "//manipulation/models/wsg_50_description:models",

BTW, we don't need this line anymore.

Copy link
Collaborator Author

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

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

+@rpoyner-tri for platform review per schedule (tomorrow), please.

Reviewable status: LGTM missing from assignee rpoyner-tri(platform)

@rpoyner-tri rpoyner-tri assigned ggould-tri and unassigned rpoyner-tri Apr 1, 2024
Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

-@rpoyner-tri +@ggould-tri for platform review per schedule (tomorrow), please.

Reviewable status: LGTM missing from assignee ggould-tri(platform)

Copy link
Contributor

@ggould-tri ggould-tri left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 8 of 9 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignees ggould-tri(platform),DamrongGuoy

Copy link
Collaborator Author

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 unresolved discussion


tools/workspace/drake_models/repository.bzl line 9 at r2 (raw file):

        name = name,
        repository = "RobotLocomotion/models",
        commit = "b2f1a5de046d0b0d27dd5b8f09020addc9a19280",

Working

Needs real sha

@jwnimmer-tri jwnimmer-tri force-pushed the wsg_50_hydro_bubble branch from 9f88e34 to 20e0a99 Compare April 2, 2024 14:18
Copy link
Collaborator Author

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

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

Reviewable status: :shipit: complete! all discussions resolved, LGTM from assignees ggould-tri(platform),DamrongGuoy


tools/workspace/drake_models/repository.bzl line 9 at r2 (raw file):

Previously, jwnimmer-tri (Jeremy Nimmer) wrote…

Working

Needs real sha

Done

Ditch the "models" subdir (moving spatula up one level).

Remove unwanted install rules for these files.

Fix image naming for consistency.

[workspace] Upgrade drake_models to latest commit.
@jwnimmer-tri jwnimmer-tri force-pushed the wsg_50_hydro_bubble branch from 20e0a99 to a1e9f43 Compare April 2, 2024 14:19
@jwnimmer-tri jwnimmer-tri merged commit 897b0dd into RobotLocomotion:master Apr 2, 2024
3 of 9 checks passed
@jwnimmer-tri jwnimmer-tri deleted the wsg_50_hydro_bubble branch April 2, 2024 14:26
RussTedrake pushed a commit to RussTedrake/drake that referenced this pull request Dec 15, 2024
Ditch the "models" subdir (moving spatula up one level).

Remove unwanted install rules for these files.

Fix image naming for consistency.

[workspace] Upgrade drake_models to latest commit.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority: medium release notes: fix This pull request contains fixes (no new features)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants