Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(trajectory_follower_base): trajectory follower base add autoware prefix #7343

Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ control/predicted_path_checker/** berkay@leodrive.ai
control/pure_pursuit/** takamasa.horibe@tier4.jp
control/shift_decider/** takamasa.horibe@tier4.jp
control/autoware_smart_mpc_trajectory_follower/** masayuki.aino@proxima-ai-tech.com
control/trajectory_follower_base/** takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp
control/autoware_trajectory_follower_base/** takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp
control/trajectory_follower_node/** takamasa.horibe@tier4.jp takayuki.murooka@tier4.jp
control/autoware_vehicle_cmd_gate/** takamasa.horibe@tier4.jp tomoya.kimura@tier4.jp
evaluator/control_evaluator/** daniel.sanchez@tier4.jp takayuki.murooka@tier4.jp
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -19,8 +19,8 @@
#include "autoware_mpc_lateral_controller/mpc_trajectory.hpp"
#include "autoware_mpc_lateral_controller/mpc_utils.hpp"
#include "autoware_mpc_lateral_controller/steering_offset/steering_offset.hpp"
#include "autoware_trajectory_follower_base/lateral_controller_base.hpp"
#include "rclcpp/rclcpp.hpp"
#include "trajectory_follower_base/lateral_controller_base.hpp"

#include "autoware_control_msgs/msg/lateral.hpp"
#include "autoware_planning_msgs/msg/trajectory.hpp"
Expand Down
2 changes: 1 addition & 1 deletion control/autoware_mpc_lateral_controller/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

<depend>autoware_control_msgs</depend>
<depend>autoware_planning_msgs</depend>
<depend>autoware_trajectory_follower_base</depend>
<depend>autoware_vehicle_info_utils</depend>
<depend>autoware_vehicle_msgs</depend>
<depend>diagnostic_msgs</depend>
Expand All @@ -35,7 +36,6 @@
<depend>tf2_ros</depend>
<depend>tier4_autoware_utils</depend>
<depend>tier4_debug_msgs</depend>
<depend>trajectory_follower_base</depend>

<test_depend>ament_cmake_ros</test_depend>
<test_depend>ament_lint_auto</test_depend>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,14 +20,14 @@
#include "autoware_pid_longitudinal_controller/lowpass_filter.hpp"
#include "autoware_pid_longitudinal_controller/pid.hpp"
#include "autoware_pid_longitudinal_controller/smooth_stop.hpp"
#include "autoware_trajectory_follower_base/longitudinal_controller_base.hpp"
#include "autoware_vehicle_info_utils/vehicle_info_utils.hpp"
#include "diagnostic_updater/diagnostic_updater.hpp"
#include "rclcpp/rclcpp.hpp"
#include "tf2/utils.h"
#include "tf2_ros/buffer.h"
#include "tf2_ros/transform_listener.h"
#include "tier4_autoware_utils/ros/marker_helper.hpp"
#include "trajectory_follower_base/longitudinal_controller_base.hpp"

#include <Eigen/Core>
#include <Eigen/Geometry>
Expand Down
2 changes: 1 addition & 1 deletion control/autoware_pid_longitudinal_controller/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,7 @@
<depend>autoware_adapi_v1_msgs</depend>
<depend>autoware_control_msgs</depend>
<depend>autoware_planning_msgs</depend>
<depend>autoware_trajectory_follower_base</depend>
<depend>autoware_vehicle_info_utils</depend>
<depend>autoware_vehicle_msgs</depend>
<depend>diagnostic_msgs</depend>
Expand All @@ -36,7 +37,6 @@
<depend>tf2_ros</depend>
<depend>tier4_autoware_utils</depend>
<depend>tier4_debug_msgs</depend>
<depend>trajectory_follower_base</depend>

<test_depend>ament_cmake_ros</test_depend>
<test_depend>ament_lint_auto</test_depend>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
cmake_minimum_required(VERSION 3.14)
project(trajectory_follower_base)
project(autoware_trajectory_follower_base)

find_package(autoware_cmake REQUIRED)
autoware_package()
Expand Down
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 TRAJECTORY_FOLLOWER_BASE__INPUT_DATA_HPP_
#define TRAJECTORY_FOLLOWER_BASE__INPUT_DATA_HPP_
#ifndef AUTOWARE_TRAJECTORY_FOLLOWER_BASE__INPUT_DATA_HPP_
#define AUTOWARE_TRAJECTORY_FOLLOWER_BASE__INPUT_DATA_HPP_

#include "autoware_adapi_v1_msgs/msg/operation_mode_state.hpp"
#include "autoware_planning_msgs/msg/trajectory.hpp"
Expand All @@ -33,4 +33,4 @@ struct InputData
};
} // namespace autoware::motion::control::trajectory_follower

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

#ifndef TRAJECTORY_FOLLOWER_BASE__LATERAL_CONTROLLER_BASE_HPP_
#define TRAJECTORY_FOLLOWER_BASE__LATERAL_CONTROLLER_BASE_HPP_
#ifndef AUTOWARE_TRAJECTORY_FOLLOWER_BASE__LATERAL_CONTROLLER_BASE_HPP_
#define AUTOWARE_TRAJECTORY_FOLLOWER_BASE__LATERAL_CONTROLLER_BASE_HPP_

#include "autoware_trajectory_follower_base/input_data.hpp"
#include "autoware_trajectory_follower_base/sync_data.hpp"
#include "rclcpp/rclcpp.hpp"
#include "trajectory_follower_base/input_data.hpp"
#include "trajectory_follower_base/sync_data.hpp"

#include "autoware_control_msgs/msg/lateral.hpp"

Expand All @@ -44,4 +44,4 @@ class LateralControllerBase

} // namespace autoware::motion::control::trajectory_follower

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

#ifndef TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_
#define TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_
#ifndef AUTOWARE_TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_
#define AUTOWARE_TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_

#include "autoware_trajectory_follower_base/input_data.hpp"
#include "autoware_trajectory_follower_base/sync_data.hpp"
#include "rclcpp/rclcpp.hpp"
#include "trajectory_follower_base/input_data.hpp"
#include "trajectory_follower_base/sync_data.hpp"

#include "autoware_control_msgs/msg/longitudinal.hpp"

Expand Down Expand Up @@ -46,4 +46,4 @@ class LongitudinalControllerBase

} // namespace autoware::motion::control::trajectory_follower

#endif // TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_
#endif // AUTOWARE_TRAJECTORY_FOLLOWER_BASE__LONGITUDINAL_CONTROLLER_BASE_HPP_
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 TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_
#define TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_
#ifndef AUTOWARE_TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_
#define AUTOWARE_TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_

namespace autoware::motion::control::trajectory_follower
{
Expand All @@ -30,4 +30,4 @@ struct LongitudinalSyncData

} // namespace autoware::motion::control::trajectory_follower

#endif // TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_
#endif // AUTOWARE_TRAJECTORY_FOLLOWER_BASE__SYNC_DATA_HPP_
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 TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_
#define TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_
#ifndef AUTOWARE_TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_
#define AUTOWARE_TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_

////////////////////////////////////////////////////////////////////////////////
#if defined(__WIN32)
Expand All @@ -34,4 +34,4 @@
#error "Unsupported Build Configuration"
#endif

#endif // TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_
#endif // AUTOWARE_TRAJECTORY_FOLLOWER_BASE__VISIBILITY_CONTROL_HPP_
Original file line number Diff line number Diff line change
@@ -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>trajectory_follower_base</name>
<name>autoware_trajectory_follower_base</name>
<version>1.0.0</version>
<description>Library for generating lateral and longitudinal controls following a trajectory</description>

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

#include "trajectory_follower_base/lateral_controller_base.hpp"
#include "autoware_trajectory_follower_base/lateral_controller_base.hpp"

namespace autoware::motion::control::trajectory_follower
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
// See the License for the specific language governing permissions and
// limitations under the License.

#include "trajectory_follower_base/longitudinal_controller_base.hpp"
#include "autoware_trajectory_follower_base/longitudinal_controller_base.hpp"

namespace autoware::motion::control::trajectory_follower
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -30,12 +30,12 @@
#ifndef PURE_PURSUIT__PURE_PURSUIT_LATERAL_CONTROLLER_HPP_
#define PURE_PURSUIT__PURE_PURSUIT_LATERAL_CONTROLLER_HPP_

#include "autoware_trajectory_follower_base/lateral_controller_base.hpp"
#include "pure_pursuit/pure_pursuit.hpp"
#include "pure_pursuit/pure_pursuit_viz.hpp"
#include "rclcpp/rclcpp.hpp"
#include "tf2_ros/buffer.h"
#include "tf2_ros/transform_listener.h"
#include "trajectory_follower_base/lateral_controller_base.hpp"

#include <motion_utils/resample/resample.hpp>
#include <motion_utils/trajectory/conversion.hpp>
Expand Down
2 changes: 1 addition & 1 deletion control/pure_pursuit/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@

<depend>autoware_control_msgs</depend>
<depend>autoware_planning_msgs</depend>
<depend>autoware_trajectory_follower_base</depend>
<depend>autoware_vehicle_info_utils</depend>
<depend>boost</depend>
<depend>geometry_msgs</depend>
Expand All @@ -32,7 +33,6 @@
<depend>tf2_ros</depend>
<depend>tier4_autoware_utils</depend>
<depend>tier4_debug_msgs</depend>
<depend>trajectory_follower_base</depend>
<depend>visualization_msgs</depend>

<test_depend>ament_lint_auto</test_depend>
Expand Down
4 changes: 2 additions & 2 deletions control/trajectory_follower_node/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,13 +6,13 @@ Generate control commands to follow a given Trajectory.

## Design

This is a node of the functionalities implemented in the controller class derived from [trajectory_follower_base](../trajectory_follower_base/README.md#trajectory-follower) package. It has instances of those functionalities, gives them input data to perform calculations, and publishes control commands.
This is a node of the functionalities implemented in the controller class derived from [autoware_trajectory_follower_base](../autoware_trajectory_follower_base/README.md#trajectory-follower) package. It has instances of those functionalities, gives them input data to perform calculations, and publishes control commands.

By default, the controller instance with the `Controller` class as follows is used.

```plantuml
@startuml
package trajectory_follower_base {
package autoware_trajectory_follower_base {
abstract class LateralControllerBase {
longitudinal_sync_data_

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@
#ifndef TRAJECTORY_FOLLOWER_NODE__CONTROLLER_NODE_HPP_
#define TRAJECTORY_FOLLOWER_NODE__CONTROLLER_NODE_HPP_

#include "autoware_trajectory_follower_base/lateral_controller_base.hpp"
#include "autoware_trajectory_follower_base/longitudinal_controller_base.hpp"
#include "autoware_vehicle_info_utils/vehicle_info_utils.hpp"
#include "rclcpp/rclcpp.hpp"
#include "tf2/utils.h"
Expand All @@ -23,8 +25,6 @@
#include "tier4_autoware_utils/ros/logger_level_configure.hpp"
#include "tier4_autoware_utils/ros/polling_subscriber.hpp"
#include "tier4_autoware_utils/system/stop_watch.hpp"
#include "trajectory_follower_base/lateral_controller_base.hpp"
#include "trajectory_follower_base/longitudinal_controller_base.hpp"
#include "trajectory_follower_node/visibility_control.hpp"

#include <Eigen/Core>
Expand Down
2 changes: 1 addition & 1 deletion control/trajectory_follower_node/package.xml
Original file line number Diff line number Diff line change
Expand Up @@ -24,14 +24,14 @@
<depend>autoware_mpc_lateral_controller</depend>
<depend>autoware_pid_longitudinal_controller</depend>
<depend>autoware_planning_msgs</depend>
<depend>autoware_trajectory_follower_base</depend>
<depend>autoware_vehicle_info_utils</depend>
<depend>autoware_vehicle_msgs</depend>
<depend>motion_utils</depend>
<depend>pure_pursuit</depend>
<depend>rclcpp</depend>
<depend>rclcpp_components</depend>
<depend>tier4_autoware_utils</depend>
<depend>trajectory_follower_base</depend>
<depend>visualization_msgs</depend>

<exec_depend>ros2launch</exec_depend>
Expand Down
Loading