Skip to content

mujoco: Support RemoveModelFromWorld feature#892

Draft
iche033 wants to merge 1 commit intoiche033/mujoco_get_contactsfrom
iche033/mujoco_remove_model
Draft

mujoco: Support RemoveModelFromWorld feature#892
iche033 wants to merge 1 commit intoiche033/mujoco_get_contactsfrom
iche033/mujoco_remove_model

Conversation

@iche033
Copy link
Contributor

@iche033 iche033 commented Mar 9, 2026

🎉 New feature

Addresses #299
Depends on:

Summary

Added the RemoveModelFromWorld feature for mujoco plugin.

  • Implemented various RemoveModel* functions
  • Fixed a bug in SDFFeatures that calls GetNextEntity twice when constructing a model
  • Nested models are not supported yet.

Test it

ctest -R UNIT_mujoco_SDFFeatures_TEST

Checklist

  • Signed all commits for DCO
  • Added a screen capture or video to the PR description that demonstrates the feature
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • Updated Bazel files (if adding new files). Created an issue otherwise.
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers
  • Was GenAI used to generate this PR? If so, make sure to add "Generated-by" to your commits. (See this policy for more info.)

Generated-by: Gemini 3

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by and Generated-by messages.

Backports: If this is a backport, please use Rebase and Merge instead.

Signed-off-by: Ian Chen <iche@intrinsic.ai>
@iche033 iche033 requested review from azeey and scpeters as code owners March 9, 2026 23:17
@iche033 iche033 marked this pull request as draft March 9, 2026 23:17
@iche033 iche033 added the mujoco label Mar 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: Inbox

Development

Successfully merging this pull request may close these issues.

1 participant