Skip to content

Commit

Permalink
refactor(external cmd converter)!: add autoware_ prefix (#7361)
Browse files Browse the repository at this point in the history
* add prefix to the code

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* rename

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* fix

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* fix

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* fix

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

* Update .github/CODEOWNERS

---------

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: Takayuki Murooka <takayuki5168@gmail.com>
yuki-takagi-66 and takayuki5168 authored Jun 10, 2024
1 parent e57e9f1 commit 4f12fa5
Showing 13 changed files with 41 additions and 38 deletions.
3 changes: 1 addition & 2 deletions .github/CODEOWNERS
This CODEOWNERS file contains errors

CODEOWNERS errors

  • Unknown owner on line 3: make sure a user with the email address opensource@apex.ai exists and has write access to the repository
    …ommon/autoware_auto_common/** opensource@apex.ai satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 4: make sure a user with the email address ahmed.ebrahim@leodrive.ai exists and has write access to the repository
    …etails_overlay_rviz_plugin/** ahmed.ebrahim@leodrive.ai
  • Unknown owner on line 6: make sure a user with the email address opensource@apex.ai exists and has write access to the repository
    …are_perception_rviz_plugin/** opensource@apex.ai satoshi.tanaka@tier4.jp shunsuke.miura@tier4.jp taiki.tanaka@tier4.jp takeshi.miura@tier4.jp yoshi.ri@tier4.jp
  • Unknown owner on line 7: make sure a user with the email address taichi.higashide@tier4.jp exists and has write access to the repository
    …ommon/autoware_point_types/** taichi.higashide@tier4.jp
  • Unknown owner on line 8: make sure a user with the email address adam.dabrowski@robotec.ai exists and has write access to the repository
    common/autoware_testing/** adam.dabrowski@robotec.ai satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 14: make sure a user with the email address opensource@apex.ai exists and has write access to the repository
    common/fake_test_node/** opensource@apex.ai satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 29: make sure a user with the email address ali.boyali@tier4.jp exists and has write access to the repository
    common/signal_processing/** ali.boyali@tier4.jp fumiya.watanabe@tier4.jp kyoichi.sugahara@tier4.jp takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp
  • Unknown owner on line 34: make sure a user with the email address uken.ryu@tier4.jp exists and has write access to the repository
    …gin/** makoto.ybauta@tier4.jp uken.ryu@tier4.jp
  • Unknown owner on line 34: make sure a user with the email address makoto.ybauta@tier4.jp exists and has write access to the repository
    …r4_camera_view_rviz_plugin/** makoto.ybauta@tier4.jp uken.ryu@tier4.jp
  • Unknown owner on line 43: make sure a user with the email address christopherj.ho@gmail.com exists and has write access to the repository
    common/time_utils/** christopherj.ho@gmail.com shumpei.wakabayashi@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 46: make sure a user with the email address xinyu.wang@tier4.jp exists and has write access to the repository
    …y/** ambroise.vincent@arm.com xinyu.wang@tier4.jp
  • Unknown owner on line 48: make sure a user with the email address berkay@leodrive.ai exists and has write access to the repository
    …ntrol_performance_analysis/** berkay@leodrive.ai fumiya.watanabe@tier4.jp satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp taiki.tanaka@tier4.jp takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 57: make sure a user with the email address berkay@leodrive.ai exists and has write access to the repository
    …rol/predicted_path_checker/** berkay@leodrive.ai
  • Unknown owner on line 60: make sure a user with the email address masayuki.aino@proxima-ai-tech.com exists and has write access to the repository
    …rt_mpc_trajectory_follower/** masayuki.aino@proxima-ai-tech.com
  • Unknown owner on line 75: make sure a user with the email address zhe.shen@tier4.jp exists and has write access to the repository
    …er4.jp tomoya.kimura@tier4.jp zhe.shen@tier4.jp zulfaqar.azmi@tier4.jp
  • Unknown owner on line 113: make sure a user with the email address taichi.higashide@tier4.jp exists and has write access to the repository
    …r4.jp shintaro.tomie@tier4.jp taichi.higashide@tier4.jp
  • Unknown owner on line 118: make sure a user with the email address xinyu.wang@tier4.jp exists and has write access to the repository
    …m/** ambroise.vincent@arm.com xinyu.wang@tier4.jp
  • Unknown owner on line 119: make sure a user with the email address xinyu.wang@tier4.jp exists and has write access to the repository
    …s/** ambroise.vincent@arm.com xinyu.wang@tier4.jp
  • Unknown owner on line 121: make sure a user with the email address carl.liu@autocore.ai exists and has write access to the repository
    …tion/lidar_centerpoint_tvm/** carl.liu@autocore.ai xinyu.wang@tier4.jp
  • Unknown owner on line 121: make sure a user with the email address xinyu.wang@tier4.jp exists and has write access to the repository
    …t_tvm/** carl.liu@autocore.ai xinyu.wang@tier4.jp
  • Unknown owner on line 154: make sure a user with the email address zhe.shen@tier4.jp exists and has write access to the repository
    …er4.jp tomoya.kimura@tier4.jp zhe.shen@tier4.jp
  • Unknown owner on line 164: make sure a user with the email address ahmed.ebrahim@leodrive.ai exists and has write access to the repository
    …g_distance_time_calculator/** ahmed.ebrahim@leodrive.ai
  • Unknown owner on line 180: make sure a user with the email address ahmed.ebrahim@leodrive.ai exists and has write access to the repository
    …ty_no_drivable_lane_module/** ahmed.ebrahim@leodrive.ai fumiya.watanabe@tier4.jp satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 194: make sure a user with the email address berkay@leodrive.ai exists and has write access to the repository
    …ning/obstacle_stop_planner/** berkay@leodrive.ai bnk@leodrive.ai satoshi.ota@tier4.jp shumpei.wakabayashi@tier4.jp taiki.tanaka@tier4.jp tomoya.kimura@tier4.jp
  • Unknown owner on line 222: make sure a user with the email address nagy.tomas@tier4.jp exists and has write access to the repository
    …el/** maxime.clement@tier4.jp nagy.tomas@tier4.jp
  • Unknown owner on line 235: make sure a user with the email address uken.ryu@tier4.jp exists and has write access to the repository
    … shumpei.wakabayashi@tier4.jp uken.ryu@tier4.jp
  • Unknown owner on line 246: make sure a user with the email address berkay@leodrive.ai exists and has write access to the repository
    tools/reaction_analyzer/** berkay@leodrive.ai
Original file line number Diff line number Diff line change
@@ -244,9 +244,8 @@ system/system_monitor/** fumihito.ito@tier4.jp tetsuhiro.kawaguchi@tier4.jp
system/topic_state_monitor/** ryohsuke.mitsudome@tier4.jp
system/velodyne_monitor/** fumihito.ito@tier4.jp
tools/reaction_analyzer/** berkay@leodrive.ai
vehicle/autoware_steer_offset_estimator/** taiki.tanaka@tier4.jp
vehicle/autoware_accel_brake_map_calibrator/** taiki.tanaka@tier4.jp takeshi.miura@tier4.jp tomoya.kimura@tier4.jp
vehicle/external_cmd_converter/** takamasa.horibe@tier4.jp
vehicle/autoware_external_cmd_converter/** takamasa.horibe@tier4.jp
vehicle/autoware_raw_vehicle_cmd_converter/** makoto.kurihara@tier4.jp taiki.tanaka@tier4.jp takamasa.horibe@tier4.jp
vehicle/autoware_steer_offset_estimator/** taiki.tanaka@tier4.jp
vehicle/autoware_vehicle_info_utils/** shumpei.wakabayashi@tier4.jp taiki.tanaka@tier4.jp tomoya.kimura@tier4.jp
5 changes: 4 additions & 1 deletion launch/tier4_control_launch/launch/control.launch.py
Original file line number Diff line number Diff line change
@@ -290,7 +290,10 @@ def launch_setup(context, *args, **kwargs):
# external cmd converter
external_cmd_converter_loader = IncludeLaunchDescription(
PythonLaunchDescriptionSource(
[FindPackageShare("external_cmd_converter"), "/launch/external_cmd_converter.launch.py"]
[
FindPackageShare("autoware_external_cmd_converter"),
"/launch/external_cmd_converter.launch.py",
]
),
launch_arguments=[
("use_intra_process", LaunchConfiguration("use_intra_process")),
3 changes: 2 additions & 1 deletion launch/tier4_control_launch/package.xml
Original file line number Diff line number Diff line change
@@ -11,11 +11,12 @@
<buildtool_depend>ament_cmake_auto</buildtool_depend>
<buildtool_depend>autoware_cmake</buildtool_depend>

<exec_depend>autoware_external_cmd_converter</exec_depend>
<exec_depend>autoware_external_cmd_selector</exec_depend>
<exec_depend>autoware_lane_departure_checker</exec_depend>
<exec_depend>autoware_vehicle_cmd_gate</exec_depend>
<exec_depend>control_evaluator</exec_depend>
<exec_depend>external_cmd_converter</exec_depend>
<exec_depend>external_cmd_selector</exec_depend>
<exec_depend>shift_decider</exec_depend>
<exec_depend>trajectory_follower_node</exec_depend>

19 changes: 19 additions & 0 deletions vehicle/autoware_external_cmd_converter/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
cmake_minimum_required(VERSION 3.14)
project(autoware_external_cmd_converter)

find_package(autoware_cmake REQUIRED)
autoware_package()

ament_auto_add_library(autoware_external_cmd_converter SHARED
src/node.cpp
)

rclcpp_components_register_node(autoware_external_cmd_converter
PLUGIN "autoware::external_cmd_converter::ExternalCmdConverterNode"
EXECUTABLE external_cmd_converter_node
)

ament_auto_package(INSTALL_TO_SHARE
launch
config
)
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -12,8 +12,8 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#ifndef EXTERNAL_CMD_CONVERTER__NODE_HPP_
#define EXTERNAL_CMD_CONVERTER__NODE_HPP_
#ifndef AUTOWARE_EXTERNAL_CMD_CONVERTER__NODE_HPP_
#define AUTOWARE_EXTERNAL_CMD_CONVERTER__NODE_HPP_

#include "tier4_autoware_utils/ros/polling_subscriber.hpp"

@@ -32,7 +32,7 @@
#include <memory>
#include <string>

namespace external_cmd_converter
namespace autoware::external_cmd_converter
{
using GearCommand = autoware_vehicle_msgs::msg::GearCommand;
using autoware_control_msgs::msg::Control;
@@ -104,6 +104,6 @@ class ExternalCmdConverterNode : public rclcpp::Node
double get_shift_velocity_sign(const GearCommand & cmd);
};

} // namespace external_cmd_converter
} // namespace autoware::external_cmd_converter

#endif // EXTERNAL_CMD_CONVERTER__NODE_HPP_
#endif // AUTOWARE_EXTERNAL_CMD_CONVERTER__NODE_HPP_
Original file line number Diff line number Diff line change
@@ -105,8 +105,8 @@ def generate_launch_description():
]

component = ComposableNode(
package="external_cmd_converter",
plugin="external_cmd_converter::ExternalCmdConverterNode",
package="autoware_external_cmd_converter",
plugin="autoware::external_cmd_converter::ExternalCmdConverterNode",
name="external_cmd_converter",
remappings=[
_create_mapping_tuple("in/external_control_cmd"),
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
<?xml version="1.0"?>
<launch>
<arg name="external_cmd_converter_param" default="$(find-pkg-share external_cmd_converter)/config/external_cmd_converter.param.yaml"/>
<arg name="external_cmd_converter_param" default="$(find-pkg-share autoware_external_cmd_converter)/config/external_cmd_converter.param.yaml"/>
<!-- map file -->
<arg name="csv_path_accel_map" default="$(find-pkg-share autoware_raw_vehicle_cmd_converter)/data/default/accel_map.csv"/>
<arg name="csv_path_brake_map" default="$(find-pkg-share autoware_raw_vehicle_cmd_converter)/data/default/brake_map.csv"/>
@@ -24,7 +24,7 @@
<arg name="out/latest_external_control_cmd" default="/api/external/get/command/selected/control"/>

<!-- node -->
<node pkg="external_cmd_converter" exec="external_cmd_converter_node" name="external_cmd_converter" output="screen">
<node pkg="autoware_external_cmd_converter" exec="external_cmd_converter_node" name="external_cmd_converter" output="screen">
<param from="$(var external_cmd_converter_param)" allow_substs="true"/>
<remap from="in/external_control_cmd" to="$(var in/external_control_cmd)"/>
<remap from="in/shift_cmd" to="$(var in/shift_cmd)"/>
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<?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>external_cmd_converter</name>
<name>autoware_external_cmd_converter</name>
<version>0.1.0</version>
<description>The external_cmd_converter package</description>
<description>The autoware_external_cmd_converter package</description>
<maintainer email="takamasa.horibe@tier4.jp">Takamasa Horibe</maintainer>
<maintainer email="eiki.nagata.2@tier4.jp">Eiki Nagata</maintainer>
<license>Apache License 2.0</license>
Original file line number Diff line number Diff line change
@@ -12,14 +12,14 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#include "external_cmd_converter/node.hpp"
#include "autoware_external_cmd_converter/node.hpp"

#include <algorithm>
#include <memory>
#include <string>
#include <utility>

namespace external_cmd_converter
namespace autoware::external_cmd_converter
{
ExternalCmdConverterNode::ExternalCmdConverterNode(const rclcpp::NodeOptions & node_options)
: Node("external_cmd_converter", node_options)
@@ -226,7 +226,7 @@ bool ExternalCmdConverterNode::check_remote_topic_rate()

return true;
}
} // namespace external_cmd_converter
} // namespace autoware::external_cmd_converter

#include <rclcpp_components/register_node_macro.hpp>
RCLCPP_COMPONENTS_REGISTER_NODE(external_cmd_converter::ExternalCmdConverterNode)
RCLCPP_COMPONENTS_REGISTER_NODE(autoware::external_cmd_converter::ExternalCmdConverterNode)
19 changes: 0 additions & 19 deletions vehicle/external_cmd_converter/CMakeLists.txt

This file was deleted.

0 comments on commit 4f12fa5

Please sign in to comment.