-
Notifications
You must be signed in to change notification settings - Fork 22
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_utils_pcl): split package #29
feat(autoware_utils_pcl): split package #29
Conversation
Signed-off-by: Takagi, Isamu <isamu.takagi@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: Takagi, Isamu <isamu.takagi@tier4.jp>
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
autoware_utils/include/autoware_utils/ros/transform_listener.hpp
Outdated
Show resolved
Hide resolved
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
@isamu-takagi overall it looks good, just some minor feedback about copyright notices. Thanks! |
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
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.
@isamu-takagi LGTM
Description
#34
Split out the part related to
PCL
fromautoware_utils
. Also the minimumtf2
part for it asautoware_utils_geometry
.How was this PR tested?
Check that the build of autoware core and universe passes
Notes for reviewers
None.
Effects on system behavior
None.