-
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
refactor(lidar_centerpoint)!: fix namespace and directory structure #8049
refactor(lidar_centerpoint)!: fix namespace and directory structure #8049
Conversation
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8049 +/- ##
========================================
Coverage 29.13% 29.13%
========================================
Files 1609 1635 +26
Lines 118141 118835 +694
Branches 50848 51057 +209
========================================
+ Hits 34419 34622 +203
- Misses 74494 74965 +471
- Partials 9228 9248 +20
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@MasatoSaeki |
@knzo25 |
@MasatoSaeki
|
Question: Since the package changed name, maybe Optional: Transfusion already addressed this, but the namespace for the previously called |
@knzo25
It is ok. I confirmed that to mot's code owner.
We need to handle this. So I will do later.
Either is good for me. If you recommend
It is true. Please wait a moment while I handle this. |
Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp>
I changed namespace from |
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
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
…utowarefoundation#8049) * add prefix in lidar_centerpoint Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add .gitignore Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change include package name in image_projection_based fusion Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change in codeowner Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * delete package Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * style(pre-commit): autofix * solve conflict too Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix include file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix typo in launch file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add prefix in README Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix bugs by conflict Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * change namespace from to Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp>
…utowarefoundation#8049) * add prefix in lidar_centerpoint Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add .gitignore Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change include package name in image_projection_based fusion Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change in codeowner Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * delete package Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * style(pre-commit): autofix * solve conflict too Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix include file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix typo in launch file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add prefix in README Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix bugs by conflict Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * change namespace from to Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp>
…utowarefoundation#8049) * add prefix in lidar_centerpoint Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add .gitignore Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change include package name in image_projection_based fusion Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change in codeowner Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * delete package Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * style(pre-commit): autofix * solve conflict too Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix include file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix typo in launch file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add prefix in README Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix bugs by conflict Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * change namespace from to Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp>
…utowarefoundation#8049) * add prefix in lidar_centerpoint Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add .gitignore Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change include package name in image_projection_based fusion Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * change in codeowner Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * delete package Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * style(pre-commit): autofix * solve conflict too Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix include file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix typo in launch file Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * add prefix in README Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * fix bugs by conflict Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix * change namespace from to Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> * style(pre-commit): autofix --------- Signed-off-by: MasatoSaeki <masato.saeki@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com> Co-authored-by: Kenzo Lobos Tsunekawa <kenzo.lobos@tier4.jp>
Description
This PR puts headers in the
autoware
namespace.Additional works
Related links
Part of: autowarefoundation/autoware#4569
How was this PR tested?
Tested in a local recompute environment how to test.
And also tested
colcon test
. results is below.autoware_lidar_centerpoint
image_projection_based_fusion
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.