-
Notifications
You must be signed in to change notification settings - Fork 630
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(autoware_pointcloud_preprocessor): rework ring passthrough filter parameters #8472
refactor(autoware_pointcloud_preprocessor): rework ring passthrough filter parameters #8472
Conversation
Signed-off-by: vividf <yihsiang.fang@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: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #8472 +/- ##
==========================================
- Coverage 25.23% 25.21% -0.02%
==========================================
Files 1325 1330 +5
Lines 98572 98715 +143
Branches 38088 38148 +60
==========================================
+ Hits 24874 24893 +19
- Misses 71156 71272 +116
- Partials 2542 2550 +8
*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.
Your schema file is not processed because none of config files match:
No configuration files found for schema sensing/autoware_pointcloud_preprocessor/schema/passthrough_filter_uint16_node.schema.json.
FYI, CI takes schema file name without suffix and compare it with all config files in ../config/ directory with name <schema_file_name>*.param.yaml. You could change schema or config name (may affect other packages).
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
@amadeuszsz |
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
@badai-nguyen @knzo25 @drwnz This PR needs a maintainer to approve it, thanks! |
sensing/autoware_pointcloud_preprocessor/config/passthrough_filter_uint16_node.param.yaml
Outdated
Show resolved
Hide resolved
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@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
…ilter parameters (autowarefoundation#8472) * feat: rework ring passthrough parameters Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: fix cmake Signed-off-by: vividf <yihsiang.fang@tier4.jp> * feat: add schema Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: fix readme Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: fix parameter file name Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: add boundary Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: fix default parameter Signed-off-by: vividf <yihsiang.fang@tier4.jp> * chore: fix default parameter in schema Signed-off-by: vividf <yihsiang.fang@tier4.jp> --------- Signed-off-by: vividf <yihsiang.fang@tier4.jp> Signed-off-by: emuemuJP <k.matsumoto.0807@gmail.com>
Description
This PR includes the following changes
nodelet
tonode
.A big thank you to @Ariiees for the PRs (#8297 and #7422), where we worked together on the entire pointcloud preprocessor parameters, launch file, and schema.
Related links
Parent Issue:
How was this PR tested?
ros2 launch autoware_pointcloud_preprocessor ring_passthrough_filter.launch.xml
Notes for reviewers
None.
Interface changes
None.
Effects on system behavior
None.