-
Notifications
You must be signed in to change notification settings - Fork 683
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(ndt_scan_matcher): apply interface change of ndt_omp #8124
fix(ndt_scan_matcher): apply interface change of ndt_omp #8124
Conversation
Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
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.
Looks Good To Me
After the corresponding ndt_omp repository has been modified, I will submit the approve.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8124 +/- ##
==========================================
+ Coverage 23.74% 23.78% +0.03%
==========================================
Files 1379 1380 +1
Lines 101359 101345 -14
Branches 38524 38522 -2
==========================================
+ Hits 24072 24108 +36
+ Misses 74835 74772 -63
- Partials 2452 2465 +13
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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
34c1b8b
into
autowarefoundation:main
…ndation#8124) match the func name changed in ndt_omp Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> Co-authored-by: SakodaShintaro <shintaro.sakoda@tier4.jp>
…ndation#8124) match the func name changed in ndt_omp Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com> Co-authored-by: SakodaShintaro <shintaro.sakoda@tier4.jp> Signed-off-by: xtk8532704 <1041084556@qq.com>
Description
This PR is to apply the interface change of tier4/ndt_omp#57
This PR needs to be merged with the tier4/ndt_omp#57
Change:
estimate_xy_covariance_by_Laplace_approximation
toestimate_xy_covariance_by_laplace_approximation
Related links
Parent Issue:
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.