Skip to content

Conversation

@sea-bass
Copy link
Contributor

@sea-bass sea-bass commented Dec 31, 2024

Description

If people want to use pick_ik, it can still be installed optionally. But it's a bit wasteful to have it be built all the time.

Checklist

  • Required by CI: Code is auto formatted using clang-format
  • While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

@stephanie-eng
Copy link
Contributor

Can we add a small reminder to this blurb in the pick_ik tutorial to install pick_ik before running the pre-configured example? We already have instructions in the tutorial on how to do it, but since it won't quite be "out-of-the box", wouldn't hurt to remind folks.
image

@stephanie-eng
Copy link
Contributor

If you don't want to do the above, I can probably just do it in a future PR later, I'm also fine with this as-is.

@sea-bass sea-bass merged commit 8134bc7 into main Dec 31, 2024
9 checks passed
@sea-bass sea-bass deleted the remove-pick-ik branch December 31, 2024 16:10
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.

4 participants