-
Notifications
You must be signed in to change notification settings - Fork 673
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(pointcloud_preprocessor): publish excluded points in ring_outlier_filter for debug purpose #6483
feat(pointcloud_preprocessor): publish excluded points in ring_outlier_filter for debug purpose #6483
Conversation
940da42
to
f602ad3
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #6483 +/- ##
==========================================
- Coverage 14.63% 14.63% -0.01%
==========================================
Files 1900 1900
Lines 130645 130678 +33
Branches 38381 38381
==========================================
Hits 19123 19123
- Misses 90064 90097 +33
Partials 21458 21458
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Sentry. |
f602ad3
to
5640819
Compare
sensing/pointcloud_preprocessor/src/outlier_filter/ring_outlier_filter_nodelet.cpp
Outdated
Show resolved
Hide resolved
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
Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
501c2d1
to
2ae8c9f
Compare
Updated with a rebase because this PR is merged: |
0410e92
into
autowarefoundation:main
…r_filter for debug purpose (#6483) * publish excluded points for debug purpose Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * improve performance of extraction method Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * change variable name to avoid spell miss checker error --------- Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> Signed-off-by: Kotaro Yoshimoto <pythagora.yoshimoto@gmail.com>
…r_filter for debug purpose (autowarefoundation#6483) * publish excluded points for debug purpose Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * improve performance of extraction method Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp> * change variable name to avoid spell miss checker error --------- Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>
Description
Publish excluded points in ring_outlier_filter for visualization.
This feature is disabled by default.
The white points have been excluded by the ring_outlier_filter, whereas the colorful points represent the pointcloud after the ring_outlier_filter.
Tests performed
running logging_simulator with enabling sensing moudle.
How to check this feature
necessary change for autoware.rviz
sample rosbag for TIER IV developer
TIERIV INTERNAL LINK
Basically I expect this feature works with any rosbag file though.
Effects on system behavior
Not applicable.
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.