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

feat: add RRT* planner #504

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

villekuosmanen
Copy link
Contributor

Motivation and Context

RRT* planner for planning optimal trajectories.

Tested in my own robot agent that's based on Habitat Lab. Should work here as well but haven't tested. Needs to be integrated into the agents, possibly as an optional param only used for real robots with a fallback to RRTConnect of failed plans, as it feels too inefficient to use in sim.

Changelog

Add RRT* planner.

How Has This Been Tested

Tested as is within my robot agent that is based on the same planning system, but not specifically tested here.

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • I have made changes to existing documentation where needed.
  • I have added tests that show that the PR is functional.
  • I ran tests/hw_manual_test.py on hardware and observed that robot behavior looks normal.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jun 8, 2024
@villekuosmanen villekuosmanen changed the title feat: add rtt-star planner feat: add RRT* planner Jun 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants