Skip to content

Commit

Permalink
refactor(vehicle_velocity_converter)!: prefix package and namespace w…
Browse files Browse the repository at this point in the history
…ith autoware (autowarefoundation#8967)

* add autoware prefix

Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>

* fix conflict

Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>

---------

Signed-off-by: a-maumau <maumaumaumaumaumaumaumaumaumau@gmail.com>
Co-authored-by: Yamato Ando <yamato.ando@gmail.com>
  • Loading branch information
a-maumau and YamatoAndo authored Dec 6, 2024
1 parent 3231bbf commit 0bbdbe1
Show file tree
Hide file tree
Showing 11 changed files with 21 additions and 15 deletions.
2 changes: 1 addition & 1 deletion .github/_CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -211,7 +211,7 @@ sensing/autoware_radar_static_pointcloud_filter/** satoshi.tanaka@tier4.jp shuns
sensing/autoware_radar_threshold_filter/** satoshi.tanaka@tier4.jp shunsuke.miura@tier4.jp taekjin.lee@tier4.jp yoshi.ri@tier4.jp
sensing/autoware_radar_tracks_noise_filter/** satoshi.tanaka@tier4.jp shunsuke.miura@tier4.jp taekjin.lee@tier4.jp yoshi.ri@tier4.jp
sensing/livox/autoware_livox_tag_filter/** kenzo.lobos@tier4.jp ryohsuke.mitsudome@tier4.jp
sensing/vehicle_velocity_converter/** ryu.yamamoto@tier4.jp
sensing/autoware_vehicle_velocity_converter/** ryu.yamamoto@tier4.jp
simulator/autoware_carla_interface/** maxime.clement@tier4.jp mradityagio@gmail.com
simulator/dummy_perception_publisher/** yukihiro.saito@tier4.jp
simulator/fault_injection/** keisuke.shima@tier4.jp
Expand Down
4 changes: 2 additions & 2 deletions launch/tier4_simulator_launch/launch/simulator.launch.xml
Original file line number Diff line number Diff line change
Expand Up @@ -188,9 +188,9 @@
<push-ros-namespace namespace="sensing"/>
<arg name="input_vehicle_velocity_topic" default="/vehicle/status/velocity_status"/>
<arg name="output_twist_with_covariance" default="/sensing/vehicle_velocity_converter/twist_with_covariance"/>
<arg name="config_file" default="$(find-pkg-share vehicle_velocity_converter)/config/vehicle_velocity_converter.param.yaml"/>
<arg name="config_file" default="$(find-pkg-share autoware_vehicle_velocity_converter)/config/vehicle_velocity_converter.param.yaml"/>

<node pkg="vehicle_velocity_converter" exec="vehicle_velocity_converter_node" output="both">
<node pkg="autoware_vehicle_velocity_converter" exec="autoware_vehicle_velocity_converter_node" output="both">
<param from="$(var config_file)"/>
<remap from="velocity_status" to="$(var input_vehicle_velocity_topic)"/>
<remap from="twist_with_covariance" to="$(var output_twist_with_covariance)"/>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
cmake_minimum_required(VERSION 3.14)
project(vehicle_velocity_converter)
project(autoware_vehicle_velocity_converter)

find_package(autoware_cmake REQUIRED)
autoware_package()
Expand All @@ -9,7 +9,7 @@ ament_auto_add_library(${PROJECT_NAME} SHARED
)

rclcpp_components_register_node(${PROJECT_NAME}
PLUGIN "VehicleVelocityConverter"
PLUGIN "autoware::vehicle_velocity_converter::VehicleVelocityConverter"
EXECUTABLE ${PROJECT_NAME}_node
EXECUTOR SingleThreadedExecutor
)
Expand Down
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# vehicle_velocity_converter
# autoware_vehicle_velocity_converter

## Purpose

Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
<launch>
<arg name="input_vehicle_velocity_topic" default="velocity_status"/>
<arg name="output_twist_with_covariance" default="twist_with_covariance"/>
<arg name="config_file" default="$(find-pkg-share vehicle_velocity_converter)/config/vehicle_velocity_converter.param.yaml"/>
<arg name="config_file" default="$(find-pkg-share autoware_vehicle_velocity_converter)/config/vehicle_velocity_converter.param.yaml"/>

<node pkg="vehicle_velocity_converter" exec="vehicle_velocity_converter_node" output="both">
<node pkg="autoware_vehicle_velocity_converter" exec="autoware_vehicle_velocity_converter_node" output="both">
<param from="$(var config_file)"/>
<remap from="velocity_status" to="$(var input_vehicle_velocity_topic)"/>
<remap from="twist_with_covariance" to="$(var output_twist_with_covariance)"/>
Expand Down
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>vehicle_velocity_converter</name>
<name>autoware_vehicle_velocity_converter</name>
<version>0.38.0</version>
<description>The vehicle_velocity_converter package</description>
<description>The autoware_vehicle_velocity_converter package</description>
<maintainer email="ryu.yamamoto@tier4.jp">Ryu Yamamoto</maintainer>
<license>Apache License 2.0</license>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,10 +12,12 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#include "vehicle_velocity_converter/vehicle_velocity_converter.hpp"
#include "vehicle_velocity_converter.hpp"

#include <string>

namespace autoware::vehicle_velocity_converter
{
VehicleVelocityConverter::VehicleVelocityConverter(const rclcpp::NodeOptions & options)
: rclcpp::Node("vehicle_velocity_converter", options)
{
Expand Down Expand Up @@ -55,6 +57,7 @@ void VehicleVelocityConverter::callback_velocity_report(

twist_with_covariance_pub_->publish(twist_with_covariance_msg);
}
} // namespace autoware::vehicle_velocity_converter

#include <rclcpp_components/register_node_macro.hpp>
RCLCPP_COMPONENTS_REGISTER_NODE(VehicleVelocityConverter)
RCLCPP_COMPONENTS_REGISTER_NODE(autoware::vehicle_velocity_converter::VehicleVelocityConverter)
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,8 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#ifndef VEHICLE_VELOCITY_CONVERTER__VEHICLE_VELOCITY_CONVERTER_HPP_
#define VEHICLE_VELOCITY_CONVERTER__VEHICLE_VELOCITY_CONVERTER_HPP_
#ifndef VEHICLE_VELOCITY_CONVERTER_HPP_
#define VEHICLE_VELOCITY_CONVERTER_HPP_

#include <rclcpp/rclcpp.hpp>

Expand All @@ -25,6 +25,8 @@
#include <string>
#include <vector>

namespace autoware::vehicle_velocity_converter
{
class VehicleVelocityConverter : public rclcpp::Node
{
public:
Expand All @@ -43,5 +45,6 @@ class VehicleVelocityConverter : public rclcpp::Node
double stddev_wz_;
double speed_scale_factor_;
};
} // namespace autoware::vehicle_velocity_converter

#endif // VEHICLE_VELOCITY_CONVERTER__VEHICLE_VELOCITY_CONVERTER_HPP_
#endif // VEHICLE_VELOCITY_CONVERTER_HPP_

0 comments on commit 0bbdbe1

Please sign in to comment.