fix(pointcloud_preprocessor): separate targets for base class

[wep21]

Description

closes #1009

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[wep21]

@lchojnack Could you try this PR for #1009? I can also reproduce the error with following commands.

ros2 run rclcpp_components component_container_mt
ros2 component load /ComponentManager compare_map_segmentation compare_map_segmentation::CompareElevationMapFilterComponent
ros2 component load /ComponentManager pointcloud_preprocessor pointcloud_preprocessor::VoxelGridOutlierFilterComponent

[codecov]

Codecov Report

Base: 10.40% // Head: 10.40% // No change to project coverage 👍

Coverage data is based on head (38ebd01) compared to base (342a959).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1982   +/-   ##
=======================================
  Coverage   10.40%   10.40%           
=======================================
  Files        1159     1159           
  Lines       82172    82172           
  Branches    19290    19290           
=======================================
  Hits         8554     8554           
  Misses      64348    64348           
  Partials     9270     9270           

Flag	Coverage Δ		*Carryforward flag
differential	0.00% <ø> (?)
total	10.39% <0.00%> (ø)		Carriedforward from 342a959

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files	Coverage Δ
...ssing/include/signal_processing/lowpass_filter.hpp	100.00% <ø> (ø)
...nal_processing/test/src/lowpass_filter_1d_test.cpp	35.71% <0.00%> (ø)
...signal_processing/test/src/lowpass_filter_test.cpp	25.71% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[lhakim85]

@lchojnack Could you try this PR for #1009? I can also reproduce the error with following commands.

I try this PR but get this error

Any idea @wep21

[lchojnack]

@lchojnack Could you try this PR for #1009? I can also reproduce the error with following commands.

@wep21 I've tested the PR and it's working. I don't see the issue anymore.

[mitsudome-r]

@lhakim85 Could you provide more details about your environment? (compiler, arm or amd64, etc)
Our build check on CI is passing, and we will be merging if we can't reproduce your issue on our machines.

[lhakim85]

@mitsudome-r here the detail
Setup A
Ubuntu 20.04.5
gcc 9.4.0
Intel i7-8700
GPU RTX 3060
Source installation
Amd64

Setup B
Ubuntu 20.04.5
gcc 9.4.0
Intel i9-10900k
GPU RTX 2070
Source installation
Amd64

Both setup fail to compile with same error

[lchojnack]

LGTM

[shmpwk]

It works for me.

Ubuntu20.04
11th Gen Intel® Core™ i7-11800H
GPU RTX 2520
gcc 9.4.0

[yukkysaito]

LGTM