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.

Sign up for GitHub

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

Jump to bottom

[following] add lethal cost threshold config #200

Merged
merged 2 commits into from
Jun 1, 2024
Merged

[following] add lethal cost threshold config #200

merged 2 commits into from
Jun 1, 2024

Conversation

yambati03
Copy link
Contributor

@yambati03 yambati03 commented May 31, 2024 •
edited
Loading

Description

This PR adds a tunable lethal_cost_threshold to the trajectory following code. This makes re-planning a lot better!

Self Checklist

  • I have formatted my code using ament_uncrustify --reformat
  • I have tested that the new behavior works

@yambati03 yambati03 changed the title Add lethal cost threshold config [following] add lethal cost threshold config May 31, 2024
@davidcalderon03 davidcalderon03 merged commit e977673 into master Jun 1, 2024
3 of 5 checks passed
@davidcalderon03 davidcalderon03 deleted the fix/lethal_cost_threshold branch June 1, 2024 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidcalderon03 davidcalderon03 davidcalderon03 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yambati03 @davidcalderon03