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

Modify RRT* range #17

Merged
merged 6 commits into from
Sep 13, 2023
Merged

Modify RRT* range #17

merged 6 commits into from
Sep 13, 2023

Conversation

amalnanavati
Copy link
Contributor

@amalnanavati amalnanavati commented Sep 7, 2023

Description

Motivated by ada_feeding#72.

Earlier, the range for RRT* was set to 0.0 which, internally in OMPL, ended up being set to 7.079055 for ADA (determined through the debug logs). This resulted in some plans that made needlessly large motions, like reaching across the user and overshooting in order to go from the resting configuration to the staging configuration.

Smaller range increases planning time, but results in more concise motions. range: 3.5 had better motions but some still overshot. range: 1.75 had good motions, but often failed to find a plan within 2-2.5 secs. range: 2.5 empirically seemed like a good compromise.

This PR also updates the moveit configuration to handle workspace bounds ada_feeding#82, by making the initial position the home position, and rotating MoveIt to have visibility inside the walls.

Further Improving Plan Quality

As an aside, here are some tuning steps we should consider doing in the future to further improve planning:

  • Consider changing the termination criteria to ExactSolution. If that empirically results in similar plans, then stick with that and bump up the planning time, so that the planner returns as soon as it finds a solution as opposed to continuing to optimize as long as it has time.
  • Tune the other parameters for OMPL's RRTStar. The ones around k-nn, heuristics, and informed sampling seem particularly relevant.
  • Consider using AnytimePathShortening (MoveIt2, OMPL) as opposed to RRTStar.

Copy link
Contributor

@egordon egordon left a comment

Choose a reason for hiding this comment

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

Okay for now

@egordon egordon merged commit 95727e2 into main Sep 13, 2023
@egordon egordon deleted the amaln/rrt_range branch September 13, 2023 01:22
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.

2 participants