refactor: traffic light arbiter/autoware prefix (#8181)

* refactor(traffic_light_arbiter): apply `autoware` namespace to traffic_light_arbiter

Signed-off-by: Manato HIRABAYASHI <manato.hirabayashi@tier4.jp>

* refactor(traffic_light_arbiter): update the package name in CODEWONER

Signed-off-by: Manato HIRABAYASHI <manato.hirabayashi@tier4.jp>

* style(pre-commit): autofix

---------

Signed-off-by: Manato HIRABAYASHI <manato.hirabayashi@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

Author: manato

Diff for: .github/CODEOWNERS

@@ -136,7 +136,7 @@ perception/autoware_shape_estimation/** yoshi.ri@tier4.jp yukihiro.saito@tier4.j
 perception/autoware_simple_object_merger/** satoshi.tanaka@tier4.jp shunsuke.miura@tier4.jp yoshi.ri@tier4.jp
 perception/autoware_tensorrt_classifier/** kotaro.uetake@tier4.jp shunsuke.miura@tier4.jp
 perception/autoware_tensorrt_yolox/** dan.umeda@tier4.jp manato.hirabayashi@tier4.jp
-perception/traffic_light_arbiter/** kenzo.lobos@tier4.jp shunsuke.miura@tier4.jp
+perception/autoware_traffic_light_arbiter/** kenzo.lobos@tier4.jp shunsuke.miura@tier4.jp
 perception/autoware_traffic_light_classifier/** shunsuke.miura@tier4.jp tao.zhong@tier4.jp yukihiro.saito@tier4.jp
 perception/autoware_traffic_light_fine_detector/** shintaro.tomie@tier4.jp shunsuke.miura@tier4.jp tao.zhong@tier4.jp
 perception/autoware_traffic_light_map_based_detector/** shunsuke.miura@tier4.jp tao.zhong@tier4.jp yukihiro.saito@tier4.jp

Diff for: launch/tier4_perception_launch/launch/traffic_light_recognition/traffic_light.launch.xml

@@ -150,7 +150,7 @@
 
   <!-- V2X fusion -->
   <group>
-    <include file="$(find-pkg-share traffic_light_arbiter)/launch/traffic_light_arbiter.launch.xml">
+    <include file="$(find-pkg-share autoware_traffic_light_arbiter)/launch/traffic_light_arbiter.launch.xml">
       <arg name="perception_traffic_signals" value="$(var internal/traffic_signals)"/>
       <arg name="external_traffic_signals" value="$(var external/traffic_signals)"/>
       <arg name="output_traffic_signals" value="$(var judged/traffic_signals)"/>

Diff for: launch/tier4_perception_launch/package.xml

@@ -39,10 +39,16 @@
   <exec_depend>autoware_raindrop_cluster_filter</exec_depend>
   <exec_depend>autoware_shape_estimation</exec_depend>
   <exec_depend>autoware_tracking_object_merger</exec_depend>
+  <exec_depend>autoware_traffic_light_arbiter</exec_depend>
+  <exec_depend>autoware_traffic_light_classifier</exec_depend>
+  <exec_depend>autoware_traffic_light_fine_detector</exec_depend>
+  <exec_depend>autoware_traffic_light_map_based_detector</exec_depend>
+  <exec_depend>autoware_traffic_light_multi_camera_fusion</exec_depend>
+  <exec_depend>autoware_traffic_light_occlusion_predictor</exec_depend>
+  <exec_depend>autoware_traffic_light_visualization</exec_depend>
   <exec_depend>image_transport_decompressor</exec_depend>
   <exec_depend>pointcloud_to_laserscan</exec_depend>
   <exec_depend>topic_tools</exec_depend>
-  <exec_depend>traffic_light_arbiter</exec_depend>
 
   <test_depend>ament_lint_auto</test_depend>
   <test_depend>autoware_lint_common</test_depend>

Diff for: perception/traffic_light_arbiter/CMakeLists.txt renamed to perception/autoware_traffic_light_arbiter/CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.14)
-project(traffic_light_arbiter)
+project(autoware_traffic_light_arbiter)
 
 find_package(autoware_cmake REQUIRED)
 autoware_package()
@@ -10,7 +10,7 @@ ament_auto_add_library(${PROJECT_NAME} SHARED
 )
 
 rclcpp_components_register_nodes(${PROJECT_NAME}
-  TrafficLightArbiter
+  "autoware::TrafficLightArbiter"
 )
 
 ament_auto_package(INSTALL_TO_SHARE launch config)

Diff for: perception/traffic_light_arbiter/README.md renamed to perception/autoware_traffic_light_arbiter/README.md

@@ -12,8 +12,8 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_
-#define TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_
+#ifndef AUTOWARE__TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_
+#define AUTOWARE__TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_
 
 #include <autoware_lanelet2_extension/utility/query.hpp>
 #include <rclcpp/rclcpp.hpp>
@@ -100,4 +100,4 @@ class SignalMatchValidator
   bool isPedestrianSignal(const lanelet::Id & signal_id);
 };
 
-#endif  // TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_
+#endif  // AUTOWARE__TRAFFIC_LIGHT_ARBITER__SIGNAL_MATCH_VALIDATOR_HPP_

Diff for: perception/traffic_light_arbiter/config/traffic_light_arbiter.param.yaml renamed to perception/autoware_traffic_light_arbiter/config/traffic_light_arbiter.param.yaml

@@ -12,11 +12,11 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#ifndef TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_
-#define TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_
+#ifndef AUTOWARE__TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_
+#define AUTOWARE__TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_
 
+#include <autoware/traffic_light_arbiter/signal_match_validator.hpp>
 #include <rclcpp/rclcpp.hpp>
-#include <traffic_light_arbiter/signal_match_validator.hpp>
 
 #include <autoware_map_msgs/msg/lanelet_map_bin.hpp>
 #include <autoware_perception_msgs/msg/traffic_light_group_array.hpp>
@@ -26,6 +26,8 @@
 #include <memory>
 #include <unordered_set>
 
+namespace autoware
+{
 class TrafficLightArbiter : public rclcpp::Node
 {
 public:
@@ -58,5 +60,6 @@ class TrafficLightArbiter : public rclcpp::Node
   TrafficSignalArray latest_external_msg_;
   std::unique_ptr<SignalMatchValidator> signal_match_validator_;
 };
+}  // namespace autoware
 
-#endif  // TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_
+#endif  // AUTOWARE__TRAFFIC_LIGHT_ARBITER__TRAFFIC_LIGHT_ARBITER_HPP_

Diff for: perception/traffic_light_arbiter/include/traffic_light_arbiter/signal_match_validator.hpp renamed to perception/autoware_traffic_light_arbiter/include/autoware/traffic_light_arbiter/signal_match_validator.hpp

@@ -2,11 +2,11 @@
   <arg name="perception_traffic_signals" default="/internal/traffic_signals"/>
   <arg name="external_traffic_signals" default="/external/traffic_signals"/>
   <arg name="output_traffic_signals" default="/traffic_light_arbiter/traffic_signals"/>
-  <arg name="traffic_light_arbiter_param_path" default="$(find-pkg-share traffic_light_arbiter)/config/traffic_light_arbiter.param.yaml"/>
+  <arg name="traffic_light_arbiter_param_path" default="$(find-pkg-share autoware_traffic_light_arbiter)/config/traffic_light_arbiter.param.yaml"/>
 
   <push-ros-namespace namespace="traffic_light_arbiter"/>
   <node_container pkg="rclcpp_components" exec="component_container" name="container" namespace="">
-    <composable_node pkg="traffic_light_arbiter" plugin="TrafficLightArbiter" name="arbiter">
+    <composable_node pkg="autoware_traffic_light_arbiter" plugin="autoware::TrafficLightArbiter" name="arbiter">
       <remap from="~/sub/vector_map" to="/map/vector_map"/>
       <remap from="~/sub/perception_traffic_signals" to="$(var perception_traffic_signals)"/>
       <remap from="~/sub/external_traffic_signals" to="$(var external_traffic_signals)"/>

Diff for: perception/traffic_light_arbiter/include/traffic_light_arbiter/traffic_light_arbiter.hpp renamed to perception/autoware_traffic_light_arbiter/include/autoware/traffic_light_arbiter/traffic_light_arbiter.hpp

@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <?xml-model href="http://download.ros.org/schema/package_format3.xsd" schematypens="http://www.w3.org/2001/XMLSchema"?>
 <package format="3">
-  <name>traffic_light_arbiter</name>
+  <name>autoware_traffic_light_arbiter</name>
   <version>0.1.0</version>
   <description>The traffic_light_arbiter package</description>
   <maintainer email="kenzo.lobos@tier4.jp">Kenzo Lobos-Tsunekawa</maintainer>

Diff for: perception/traffic_light_arbiter/launch/traffic_light_arbiter.launch.xml renamed to perception/autoware_traffic_light_arbiter/launch/traffic_light_arbiter.launch.xml

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include "traffic_light_arbiter/signal_match_validator.hpp"
+#include "autoware/traffic_light_arbiter/signal_match_validator.hpp"
 
 namespace util
 {

Diff for: perception/traffic_light_arbiter/package.xml renamed to perception/autoware_traffic_light_arbiter/package.xml

@@ -12,10 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <autoware/traffic_light_arbiter/traffic_light_arbiter.hpp>
 #include <autoware_lanelet2_extension/utility/message_conversion.hpp>
 #include <autoware_lanelet2_extension/utility/query.hpp>
 #include <rclcpp/time.hpp>
-#include <traffic_light_arbiter/traffic_light_arbiter.hpp>
 
 #include <lanelet2_core/LaneletMap.h>
 #include <lanelet2_core/primitives/BasicRegulatoryElements.h>
@@ -65,6 +65,8 @@ std::vector<TrafficLightConstPtr> filter_pedestrian_signals(const LaneletMapCons
 
 }  // namespace lanelet
 
+namespace autoware
+{
 TrafficLightArbiter::TrafficLightArbiter(const rclcpp::NodeOptions & options)
 : Node("traffic_light_arbiter", options)
 {
@@ -228,6 +230,7 @@ void TrafficLightArbiter::arbitrateAndPublish(const builtin_interfaces::msg::Tim
 
   pub_->publish(output_signals_msg);
 }
+}  // namespace autoware
 
 #include <rclcpp_components/register_node_macro.hpp>
-RCLCPP_COMPONENTS_REGISTER_NODE(TrafficLightArbiter)
+RCLCPP_COMPONENTS_REGISTER_NODE(autoware::TrafficLightArbiter)