-
Notifications
You must be signed in to change notification settings - Fork 49
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(autoware_path_generator): function to smooth the route #227
base: main
Are you sure you want to change the base?
feat(autoware_path_generator): function to smooth the route #227
Conversation
Description: This commit is kind of feature porting from `autoware.universe` as follows * Import `PathWithLaneId DefaultFixedGoalPlanner::modifyPathForSmoothGoalConnection` from the following `autoware.universe` code https://github.com/autowarefoundation/autoware.universe/blob/a0816b7e3e35fbe822fefbb9c9a8132365608b49/planning/behavior_path_planner/autoware_behavior_path_goal_planner_module/src/default_fixed_goal_planner.cpp#L74-L104 * Also import all related functions from the `autoware.universe` side Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
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 #227 +/- ##
==========================================
- Coverage 78.75% 0.00% -78.76%
==========================================
Files 11 87 +76
Lines 193 8714 +8521
Branches 73 889 +816
==========================================
- Hits 152 0 -152
- Misses 11 8714 +8703
+ Partials 30 0 -30
☔ View full report in Codecov by Sentry. |
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
const double seg_dist = | ||
autoware::motion_utils::calcSignedArcLength(input.points, seg_idx, seg_idx + 1); |
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.
this is just calc_distance2d
?
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.
Thank you for this informative comment. I also guessed this is just a calculation for 2-dimensional distance between two points. If this is not my misunderstanding, the calcSignedArcLength()
function calculates the total path length by summing up the 2D distances between consecutive points, as shown in the following code:
Ref. to code:
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.
Perhaps we need to ensure which method should we use.
planning/autoware_path_generator/include/autoware/path_generator/utils.hpp
Outdated
Show resolved
Hide resolved
planning/autoware_path_generator/include/autoware/path_generator/utils.hpp
Show resolved
Hide resolved
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp> Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp> Co-authored-by: Kosuke Takeuchi <kosuke.tnp@gmail.com>
* Enhance error handlings * Remove unused variables * Simplify the code * Improve readability a little bit Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…sasakisasaki/autoware.core into feat-embed-smooth-path-as-alpha-quality Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
* This fix is for the following PR: autowarefoundation/autoware.core#227 Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
* This comment is wrote because of my misunderstanding Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Fixing error in some CI checks. |
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…sasakisasaki/autoware.core into feat-embed-smooth-path-as-alpha-quality Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Oh, it seems the test is failing. Let me investigate and fix it. Sorry. |
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.
Please add unit tests starting with only the simple cases.
@youtalk Thank you so much for your proposal! Yes, that must be a practical approach for the improvement of the code quality! I'll add some small, but would be effective tests. @kosuke55 In this PR's domain, I could detect the following hidden issue (expected package does not exist) thanks to the already existing tests. That's why I'm welcome adding the tests in this PR. Do you have any proposal what kind of tests would be effective? 👀
|
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
* Sorry, I was forgetting to do this on my local env. Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
…sasakisasaki/autoware.core into feat-embed-smooth-path-as-alpha-quality Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: Junya Sasaki <junya.sasaki@tier4.jp>
Description
This is kind of feature porting from
autoware.universe
as followsPathWithLaneId DefaultFixedGoalPlanner::modifyPathForSmoothGoalConnection
from theautoware.universe
side codeautoware.universe
sideQuality of Code
I'm improving the code quality so this PR is ready for review.The unit tests for the added feature has not been added yet.The refactoring plan is mentioned in this issue.
Perhaps we can work on the refactoring with a few PRs. Including this idea for refactoring, please feel free to share your idea for the improvement of code quality. Thank you!
How was this PR tested?
vcs import
at around 18:00 JST on 26th Feb. 2025behavior_planning_launch_xml.txt
tier4_planning_component_launch_xml.txt
path_generator_param_yaml.txt
Screencast_psim.webm
Status of Additional Tests
(ADDED) It seems the tests on the evaluator shows a failed scenario link to failed one. I'm now investigating the issue.
Notes for reviewers
Please do not enable auto merge as this PR needed to be merged with this launch side PR at the same time.
Please feel free to provide all the needed tests for merging this PR. I'm really happy for performing the tests 👍
Effects on system behavior
Needed to be investigated during the testing process.