-
Notifications
You must be signed in to change notification settings - Fork 88
fix(path_generator): merge waypoint groups with shared overlap interval #586
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
base: main
Are you sure you want to change the base?
fix(path_generator): merge waypoint groups with shared overlap interval #586
Conversation
Signed-off-by: mitukou1109 <[email protected]>
Signed-off-by: mitukou1109 <[email protected]>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #586 +/- ##
==========================================
- Coverage 47.78% 47.75% -0.04%
==========================================
Files 313 313
Lines 20088 20099 +11
Branches 8746 8747 +1
==========================================
- Hits 9599 9598 -1
- Misses 9668 9680 +12
Partials 821 821
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: mitukou1109 <[email protected]>
Description
This PR fixes waypoint grouping in case adjacent waypoint groups share their overlap interval (partly or entirely).
Before:

After:

Related links
Internal: https://star4.slack.com/archives/C0575HP7NJG/p1753246578053939?thread_ts=1753246357.263529&cid=C0575HP7NJG
How was this PR tested?
Psim
Screencast.from.2025.07.25.16.17.46.webm
Notes for reviewers
None.
Interface changes
ROS Parameter Changes
Additions and removals
waypoint_group.separation_threshold
double
1.0
Modifications
waypoint_group.interval_margin_ratio
double
10.0
waypoint.connection_gradient_from_centerline
double
10.0
Effects on system behavior
None.