refactor(object_range_splitter)!: fix namespace and directory structu…

…re (autowarefoundation#7644) * refactor: refactor object_range_splitter library and node Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: refactor object_range_splitter library and node Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: fix the node name Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: rename include path Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: remove unused dependency on sensor_msgs Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> * refactor: move header file to src Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp> --------- Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
tby-udel · Jul 14, 2024 · 4c53b39 · 4c53b39
1 parent 7dbf68d
commit 4c53b39
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 15 deletions.
diff --git a/perception/object_range_splitter/CMakeLists.txt b/perception/object_range_splitter/CMakeLists.txt
@@ -4,12 +4,12 @@ project(object_range_splitter)
 find_package(autoware_cmake REQUIRED)
 autoware_package()
 
-ament_auto_add_library(object_range_splitter SHARED
-  src/node.cpp
+ament_auto_add_library(${PROJECT_NAME} SHARED
+  src/object_range_splitter_node.cpp
 )
 
-rclcpp_components_register_node(object_range_splitter
-  PLUGIN "object_range_splitter::ObjectRangeSplitterNode"
+rclcpp_components_register_node(${PROJECT_NAME}
+  PLUGIN "autoware::object_range_splitter::ObjectRangeSplitterNode"
   EXECUTABLE object_range_splitter_node
 )
 

diff --git a/perception/object_range_splitter/package.xml b/perception/object_range_splitter/package.xml
@@ -14,7 +14,6 @@
   <depend>autoware_perception_msgs</depend>
   <depend>rclcpp</depend>
   <depend>rclcpp_components</depend>
-  <depend>sensor_msgs</depend>
 
   <test_depend>ament_lint_auto</test_depend>
   <test_depend>autoware_lint_common</test_depend>

diff --git a/...eption/object_range_splitter/src/node.cpp → ...litter/src/object_range_splitter_node.cpp b/...eption/object_range_splitter/src/node.cpp → ...litter/src/object_range_splitter_node.cpp
@@ -12,9 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include "object_range_splitter/node.hpp"
+#include "object_range_splitter_node.hpp"
 
-namespace object_range_splitter
+namespace autoware::object_range_splitter
 {
 ObjectRangeSplitterNode::ObjectRangeSplitterNode(const rclcpp::NodeOptions & node_options)
 : Node("object_range_splitter_node", node_options)
@@ -59,7 +59,7 @@ void ObjectRangeSplitterNode::objectCallback(
   long_range_object_pub_->publish(output_long_range_object_msg);
   short_range_object_pub_->publish(output_short_range_object_msg);
 }
-}  // namespace object_range_splitter
+}  // namespace autoware::object_range_splitter
 
 #include <rclcpp_components/register_node_macro.hpp>
-RCLCPP_COMPONENTS_REGISTER_NODE(object_range_splitter::ObjectRangeSplitterNode)
+RCLCPP_COMPONENTS_REGISTER_NODE(autoware::object_range_splitter::ObjectRangeSplitterNode)
diff --git a/...er/include/object_range_splitter/node.hpp → ...litter/src/object_range_splitter_node.hpp b/...er/include/object_range_splitter/node.hpp → ...litter/src/object_range_splitter_node.hpp
@@ -12,16 +12,16 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef OBJECT_RANGE_SPLITTER__NODE_HPP_
-#define OBJECT_RANGE_SPLITTER__NODE_HPP_
+#ifndef OBJECT_RANGE_SPLITTER_NODE_HPP_
+#define OBJECT_RANGE_SPLITTER_NODE_HPP_
 
 #include <rclcpp/rclcpp.hpp>
 
-#include <autoware_perception_msgs/msg/detected_objects.hpp>
+#include "autoware_perception_msgs/msg/detected_objects.hpp"
 
 #include <memory>
 
-namespace object_range_splitter
+namespace autoware::object_range_splitter
 {
 class ObjectRangeSplitterNode : public rclcpp::Node
 {
@@ -42,6 +42,6 @@ class ObjectRangeSplitterNode : public rclcpp::Node
   float spilt_range_;
 };
 
-}  // namespace object_range_splitter
+}  // namespace autoware::object_range_splitter
 
-#endif  // OBJECT_RANGE_SPLITTER__NODE_HPP_
+#endif  // OBJECT_RANGE_SPLITTER_NODE_HPP_