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(freespace_planning_algorithm): update freespace planner params #1080

Merged
merged 5 commits into from
Jul 29, 2024

Conversation

mkquda
Copy link
Contributor

@mkquda mkquda commented Jul 19, 2024

Description

Modified parameters for freespace planner (as well as goal planner and start planner) corresponding to the changes implemented in the following PR's:

(Note: merge with or after above PR's)

Tests performed

Tested on psim.

Effects on system behavior

more configuration options for A star planner:

  • can configure search method (backward or forward)
  • can tune cost for curvature
  • can tune cost for smoothness
  • can tune cost for distance to obstacles
  • can configure expansion distance value, and set to adaptive

Interface changes

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@github-actions github-actions bot added the component:map Map creation, storage, and loading. (auto-assigned) label Jul 19, 2024
@mkquda mkquda marked this pull request as ready for review July 22, 2024 09:37
Signed-off-by: mohammad alqudah <alqudah.mohammad@tier4.jp>
Signed-off-by: mohammad alqudah <alqudah.mohammad@tier4.jp>
Signed-off-by: mohammad alqudah <alqudah.mohammad@tier4.jp>
Signed-off-by: mohammad alqudah <alqudah.mohammad@tier4.jp>
Signed-off-by: mohammad alqudah <alqudah.mohammad@tier4.jp>
@mkquda mkquda merged commit bf6e342 into autowarefoundation:main Jul 29, 2024
13 checks passed
@mkquda mkquda deleted the update-freespace-planner-params branch July 29, 2024 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:map Map creation, storage, and loading. (auto-assigned) tag:run-build-and-test-differential
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants