-
Notifications
You must be signed in to change notification settings - Fork 682
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
docs(behavior_path_planner): add documentation for avoidance by lane change module #4087
docs(behavior_path_planner): add documentation for avoidance by lane change module #4087
Conversation
Documentation URL: https://autowarefoundation.github.io/autoware.universe/pr-4087/ |
…change module Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
7817db2
to
2b4d81f
Compare
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #4087 +/- ##
=======================================
Coverage 15.29% 15.29%
=======================================
Files 1461 1461
Lines 101215 101215
Branches 31215 31215
=======================================
Hits 15482 15482
Misses 68897 68897
Partials 16836 16836
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
| Name | Unit | Type | Description | Default value | | ||
| :------------------------------------------------- | ---- | ------ | ---------------------------------------------------------------------------------------------------------------------------------------- | ------------- | | ||
| execute_object_num | [-] | int | Number of avoidance target objects on ego driving lane is greater than this value, this module will be launched. | 1 | | ||
| execute_object_longitudinal_margin | [m] | double | [maybe unused] Only when distance between the ego and avoidance target object is longer than this value, this module will be launched. | 0.0 | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[maybe unused] <- Is this intentional?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this intentional?
Yes. But I'll remove this parameter.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (If the above one is no problem)
Description
Add documentation for avoidance by lane change module.
Tests performed
Nothing.
Effects on system behavior
Nothing.
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.
After all checkboxes are checked, anyone who has write access can merge the PR.