-
Notifications
You must be signed in to change notification settings - Fork 668
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
fix(landmark_based_localizer): fix to for moving the definition code of landmarks #5803
fix(landmark_based_localizer): fix to for moving the definition code of landmarks #5803
Conversation
Signed-off-by: Shintaro Sakoda <[email protected]>
I think this PR is basically OK. 👌 But, I will wait until the PR to autoware_common is reviewed before approving this. |
Signed-off-by: Shintaro Sakoda <[email protected]>
Signed-off-by: Shintaro Sakoda <[email protected]>
@KYabuuchi |
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.
@SakodaShintaro 🙆♂️
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #5803 +/- ##
=======================================
Coverage 15.24% 15.24%
=======================================
Files 1750 1750
Lines 120650 120634 -16
Branches 36738 36723 -15
=======================================
Hits 18392 18392
+ Misses 81631 81614 -17
- Partials 20627 20628 +1
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
51a8af8
into
autowarefoundation:main
…of landmarks (autowarefoundation#5803) * Fixed to use lanelet extension landmark Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed to build Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed link Signed-off-by: Shintaro Sakoda <[email protected]> --------- Signed-off-by: Shintaro Sakoda <[email protected]>
…of landmarks (autowarefoundation#5803) * Fixed to use lanelet extension landmark Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed to build Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed link Signed-off-by: Shintaro Sakoda <[email protected]> --------- Signed-off-by: Shintaro Sakoda <[email protected]>
…of landmarks (autowarefoundation#5803) * Fixed to use lanelet extension landmark Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed to build Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed link Signed-off-by: Shintaro Sakoda <[email protected]> --------- Signed-off-by: Shintaro Sakoda <[email protected]>
…of landmarks (autowarefoundation#5803) * Fixed to use lanelet extension landmark Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed to build Signed-off-by: Shintaro Sakoda <[email protected]> * Fixed link Signed-off-by: Shintaro Sakoda <[email protected]> --------- Signed-off-by: Shintaro Sakoda <[email protected]>
Description
Since the landmark definition is being moved to
autoware_common
, this pull request is a corresponding change.autowarefoundation/autoware_common#219
Tests performed
It has been confirmed that the
logging_simulator
runs with the same accuracy as before on AWSIM data with GT.Effects on system behavior
There are no effects on system behavior.
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.