-
Notifications
You must be signed in to change notification settings - Fork 669
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(lidar_centerpoint): fix constexpr related bugs #7686
Conversation
Signed-off-by: Ryuta Kambe <[email protected]>
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 #7686 +/- ##
==========================================
- Coverage 14.84% 2.63% -12.21%
==========================================
Files 1999 165 -1834
Lines 139163 10118 -129045
Branches 43716 1579 -42137
==========================================
- Hits 20661 267 -20394
+ Misses 95731 9761 -85970
+ Partials 22771 90 -22681
☔ 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.
I would have sworn that stl's math functions were compatible with constexpr 😢
LGTM
@knzo25 |
…n#7686) Signed-off-by: Ryuta Kambe <[email protected]> Signed-off-by: Simon Eisenmann <[email protected]>
…n#7686) Signed-off-by: Ryuta Kambe <[email protected]>
…n#7686) Signed-off-by: Ryuta Kambe <[email protected]> Signed-off-by: palas21 <[email protected]>
…n#7686) Signed-off-by: Ryuta Kambe <[email protected]>
Signed-off-by: Ryuta Kambe <[email protected]>
…n#7686) Signed-off-by: Ryuta Kambe <[email protected]>
Description
These are fixes based on clang-tidy CRITICAL warnings ( = clang compile error)
For example,
Related links
Parent Issue:
How was this PR tested?
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.