-
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
refactor(obstacle_cruise_planner): clean up a part of the code #3006
refactor(obstacle_cruise_planner): clean up a part of the code #3006
Conversation
5bd64b3
to
415b544
Compare
415b544
to
8e7f688
Compare
8e7f688
to
9c0b021
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #3006 +/- ##
==========================================
- Coverage 12.40% 12.39% -0.01%
==========================================
Files 1369 1369
Lines 96209 96222 +13
Branches 27729 27719 -10
==========================================
- Hits 11931 11924 -7
- Misses 71612 71671 +59
+ Partials 12666 12627 -39
*This pull request uses carry forward flags. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
9c0b021
to
8a938c2
Compare
8a938c2
to
b97d905
Compare
…ation#3006) Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp> Signed-off-by: Kenji Miyake <kenji.miyake@tier4.jp>
b97d905
to
f291f92
Compare
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
6240085
to
eabf295
Compare
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
eabf295
to
c43f614
Compare
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
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
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Description
This PR includes
Define
Obstacle
struct again so that the pointcloud can be converted to this struct.Obstacle
(TargetObstacle
) struct strongly depends on the algorithm rather than just the abstraction of the input.TargetObstacle
toObstacle
struct which is the abstraction of the input for objects and pointcloudStop/Cruise/SlowDown Obstacle
struct which is obstacle data for each algorithm.Refactor obstacle filtering method
Rename some parameters
use std::optional instead of boost::optional
Implement slow down function which is currently empty.
TODO
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.