Skip to content

[FEATURE] Batched path plan #1316

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

Merged
merged 52 commits into from
Jul 15, 2025
Merged

Conversation

Kashu7100
Copy link
Collaborator

@Kashu7100 Kashu7100 commented Jun 25, 2025

Description

Add batched (sampling based) path planning algorithms (RRT and RRTConnect) and refactored the path planners.

Related Issue

Resolves Genesis/issues/1076

Motivation and Context

We can now perform path planning in batched envs.

How Has This Been / Can This Be Tested?

Added new example:
python examples/rigid/grasp_bottle_batch.py -v -a rrt_connect
python examples/rigid/grasp_bottle_batch.py -v -a rrt

Screenshots (if appropriate):

Video with 100 parallel envs:

grasp_bottle.mp4

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@Kashu7100 Kashu7100 requested a review from duburcqa July 9, 2025 17:19
@Kashu7100
Copy link
Collaborator Author

Seems test_inverse_kinematics is failing, which is not the part of this PR.

@duburcqa
Copy link
Collaborator

Seems test_inverse_kinematics is failing, which is not the part of this PR.

Actually our IK test is very unit, and also relies on path planning. So your changes are having any impact on this test!

YilingQiao
YilingQiao previously approved these changes Jul 15, 2025
@Kashu7100
Copy link
Collaborator Author

updated the unit test for batching, addressed all comments.

@YilingQiao YilingQiao enabled auto-merge (squash) July 15, 2025 18:02
@YilingQiao
Copy link
Collaborator

Thank you @duburcqa for the reviews! As all comments are addressed, I think it's good to merge once CI passes.

@YilingQiao YilingQiao merged commit a7483c7 into Genesis-Embodied-AI:main Jul 15, 2025
13 checks passed
ACMLCZH pushed a commit to ACMLCZH/Genesis that referenced this pull request Jul 15, 2025
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