change from publish_noise_points to publish_outlier_pointcloud

Signed-off-by: kyoichi-sugahara <kyoichi.sugahara@tier4.jp>

Author: kyoichi-sugahara

sensing/pointcloud_preprocessor/docs/ring-outlier-filter.md

@@ -22,14 +22,14 @@ This implementation inherits `pointcloud_preprocessor::Filter` class, please ref
 
 ### Core Parameters
 
-| Name                      | Type    | Default Value | Description                                                                                             |
-| ------------------------- | ------- | ------------- | ------------------------------------------------------------------------------------------------------- |
-| `distance_ratio`          | double  | 1.03          |                                                                                                         |
-| `object_length_threshold` | double  | 0.1           |                                                                                                         |
-| `num_points_threshold`    | int     | 4             |                                                                                                         |
-| `max_rings_num`           | uint_16 | 128           |                                                                                                         |
-| `max_points_num_per_ring` | size_t  | 4000          | Set this value large enough such that `HFoV / resolution < max_points_num_per_ring`                     |
-| `publish_noise_points`    | bool    | false         | Flag to publish point cloud noise. Due to performance concerns, please set to false during experiments. |
+| Name                         | Type    | Default Value | Description                                                                                             |
+| ---------------------------- | ------- | ------------- | ------------------------------------------------------------------------------------------------------- |
+| `distance_ratio`             | double  | 1.03          |                                                                                                         |
+| `object_length_threshold`    | double  | 0.1           |                                                                                                         |
+| `num_points_threshold`       | int     | 4             |                                                                                                         |
+| `max_rings_num`              | uint_16 | 128           |                                                                                                         |
+| `max_points_num_per_ring`    | size_t  | 4000          | Set this value large enough such that `HFoV / resolution < max_points_num_per_ring`                     |
+| `publish_outlier_pointcloud` | bool    | false         | Flag to publish point cloud noise. Due to performance concerns, please set to false during experiments. |
 
 ## Assumptions / Known limits
 

sensing/pointcloud_preprocessor/include/pointcloud_preprocessor/outlier_filter/ring_outlier_filter_nodelet.hpp

@@ -51,7 +51,7 @@ class RingOutlierFilterComponent : public pointcloud_preprocessor::Filter
   int num_points_threshold_;
   uint16_t max_rings_num_;
   size_t max_points_num_per_ring_;
-  bool publish_noise_points_;
+  bool publish_outlier_pointcloud_;
 
   /** \brief Parameter service callback result : needed to be hold */
   OnSetParametersCallbackHandle::SharedPtr set_param_res_;

sensing/pointcloud_preprocessor/src/outlier_filter/ring_outlier_filter_nodelet.cpp

@@ -47,7 +47,8 @@ RingOutlierFilterComponent::RingOutlierFilterComponent(const rclcpp::NodeOptions
     max_rings_num_ = static_cast<uint16_t>(declare_parameter("max_rings_num", 128));
     max_points_num_per_ring_ =
       static_cast<size_t>(declare_parameter("max_points_num_per_ring", 4000));
-    publish_noise_points_ = static_cast<bool>(declare_parameter("publish_noise_points", false));
+    publish_outlier_pointcloud_ =
+      static_cast<bool>(declare_parameter("publish_outlier_pointcloud", false));
   }
 
   using std::placeholders::_1;
@@ -74,7 +75,7 @@ void RingOutlierFilterComponent::faster_filter(
   // Set up the noise points cloud, if noise points are to be published.
   PointCloud2 noise_points;
   size_t noise_points_size = 0;
-  if (publish_noise_points_) {
+  if (publish_outlier_pointcloud_) {
     noise_points.point_step = sizeof(PointXYZI);
     noise_points.data.resize(noise_points.point_step * input->width);
   }
@@ -159,7 +160,7 @@ void RingOutlierFilterComponent::faster_filter(
 
           output_size += output.point_step;
         }
-      } else if (publish_noise_points_) {
+      } else if (publish_outlier_pointcloud_) {
         for (int i = walk_first_idx; i <= walk_last_idx; i++) {
           auto noise_ptr = reinterpret_cast<PointXYZI *>(&noise_points.data[noise_points_size]);
           auto input_ptr =
@@ -208,7 +209,7 @@ void RingOutlierFilterComponent::faster_filter(
 
         output_size += output.point_step;
       }
-    } else if (publish_noise_points_) {
+    } else if (publish_outlier_pointcloud_) {
       for (int i = walk_first_idx; i < walk_last_idx; i++) {
         auto noise_ptr = reinterpret_cast<PointXYZI *>(&noise_points.data[noise_points_size]);
         auto input_ptr = reinterpret_cast<const PointXYZI *>(&input->data[indices[i]]);
@@ -231,7 +232,7 @@ void RingOutlierFilterComponent::faster_filter(
 
   setUpPointCloudFormat(input, output, output_size, /*num_fields=*/4);
 
-  if (publish_noise_points_) {
+  if (publish_outlier_pointcloud_) {
     setUpPointCloudFormat(input, noise_points, noise_points_size, /*num_fields=*/4);
     noise_points_publisher_->publish(noise_points);
   }
@@ -281,8 +282,9 @@ rcl_interfaces::msg::SetParametersResult RingOutlierFilterComponent::paramCallba
   if (get_param(p, "num_points_threshold", num_points_threshold_)) {
     RCLCPP_DEBUG(get_logger(), "Setting new num_points_threshold to: %d.", num_points_threshold_);
   }
-  if (get_param(p, "publish_noise_points", publish_noise_points_)) {
-    RCLCPP_DEBUG(get_logger(), "Setting new publish_noise_points to: %d.", publish_noise_points_);
+  if (get_param(p, "publish_outlier_pointcloud", publish_outlier_pointcloud_)) {
+    RCLCPP_DEBUG(
+      get_logger(), "Setting new publish_outlier_pointcloud to: %d.", publish_outlier_pointcloud_);
   }
 
   rcl_interfaces::msg::SetParametersResult result;