Skip to content

Commit 7686e5a

Browse files
authored
feat: apply splitting of autoware_utils_geometry (#10270)
* fix build error Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp> * merge namespace Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp> --------- Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
1 parent b3d15d9 commit 7686e5a

File tree

5 files changed

+41
-40
lines changed

5 files changed

+41
-40
lines changed

planning/autoware_path_optimizer/include/autoware/path_optimizer/utils/geometry_utils.hpp

+7-2
Original file line numberDiff line numberDiff line change
@@ -36,14 +36,19 @@
3636
#include <string>
3737
#include <vector>
3838

39-
namespace autoware_utils
39+
namespace autoware_utils_geometry
4040
{
41+
4142
template <>
4243
geometry_msgs::msg::Point get_point(const autoware::path_optimizer::ReferencePoint & p);
4344

4445
template <>
4546
geometry_msgs::msg::Pose get_pose(const autoware::path_optimizer::ReferencePoint & p);
46-
} // namespace autoware_utils
47+
48+
template <>
49+
double get_longitudinal_velocity(const autoware::path_optimizer::ReferencePoint & p);
50+
51+
} // namespace autoware_utils_geometry
4752

4853
namespace autoware::path_optimizer
4954
{

planning/autoware_path_optimizer/include/autoware/path_optimizer/utils/trajectory_utils.hpp

+2-19
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
#include "autoware/motion_utils/trajectory/trajectory.hpp"
2222
#include "autoware/path_optimizer/common_structs.hpp"
2323
#include "autoware/path_optimizer/type_alias.hpp"
24+
#include "autoware/path_optimizer/utils/geometry_utils.hpp"
2425
#include "autoware_utils/geometry/geometry.hpp"
2526

2627
#include <Eigen/Core>
@@ -36,18 +37,6 @@
3637
#include <string>
3738
#include <vector>
3839

39-
namespace autoware_utils
40-
{
41-
template <>
42-
geometry_msgs::msg::Point get_point(const autoware::path_optimizer::ReferencePoint & p);
43-
44-
template <>
45-
geometry_msgs::msg::Pose get_pose(const autoware::path_optimizer::ReferencePoint & p);
46-
47-
template <>
48-
double get_longitudinal_velocity(const autoware::path_optimizer::ReferencePoint & p);
49-
} // namespace autoware_utils
50-
5140
namespace autoware::path_optimizer
5241
{
5342
namespace trajectory_utils
@@ -107,13 +96,7 @@ TrajectoryPoint convertToTrajectoryPoint(const T & point)
10796
}
10897

10998
template <>
110-
inline TrajectoryPoint convertToTrajectoryPoint(const ReferencePoint & ref_point)
111-
{
112-
TrajectoryPoint traj_point;
113-
traj_point.pose = autoware_utils::get_pose(ref_point);
114-
traj_point.longitudinal_velocity_mps = autoware_utils::get_longitudinal_velocity(ref_point);
115-
return traj_point;
116-
}
99+
TrajectoryPoint convertToTrajectoryPoint(const ReferencePoint & ref_point);
117100

118101
template <typename T>
119102
std::vector<TrajectoryPoint> convertToTrajectoryPoints(const std::vector<T> & points)

planning/autoware_path_optimizer/src/utils/geometry_utils.cpp

+23
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,29 @@
3636
#include <stack>
3737
#include <vector>
3838

39+
namespace autoware_utils_geometry
40+
{
41+
42+
template <>
43+
geometry_msgs::msg::Point get_point(const autoware::path_optimizer::ReferencePoint & p)
44+
{
45+
return p.pose.position;
46+
}
47+
48+
template <>
49+
geometry_msgs::msg::Pose get_pose(const autoware::path_optimizer::ReferencePoint & p)
50+
{
51+
return p.pose;
52+
}
53+
54+
template <>
55+
double get_longitudinal_velocity(const autoware::path_optimizer::ReferencePoint & p)
56+
{
57+
return p.longitudinal_velocity_mps;
58+
}
59+
60+
} // namespace autoware_utils_geometry
61+
3962
namespace autoware::path_optimizer
4063
{
4164
namespace bg = boost::geometry;

planning/autoware_path_optimizer/src/utils/trajectory_utils.cpp

+7-17
Original file line numberDiff line numberDiff line change
@@ -33,31 +33,21 @@
3333
#include <stack>
3434
#include <vector>
3535

36-
namespace autoware_utils
37-
{
38-
template <>
39-
geometry_msgs::msg::Point get_point(const autoware::path_optimizer::ReferencePoint & p)
36+
namespace autoware::path_optimizer
4037
{
41-
return p.pose.position;
42-
}
4338

44-
template <>
45-
geometry_msgs::msg::Pose get_pose(const autoware::path_optimizer::ReferencePoint & p)
39+
namespace trajectory_utils
4640
{
47-
return p.pose;
48-
}
4941

5042
template <>
51-
double get_longitudinal_velocity(const autoware::path_optimizer::ReferencePoint & p)
43+
TrajectoryPoint convertToTrajectoryPoint(const ReferencePoint & ref_point)
5244
{
53-
return p.longitudinal_velocity_mps;
45+
TrajectoryPoint traj_point;
46+
traj_point.pose = autoware_utils::get_pose(ref_point);
47+
traj_point.longitudinal_velocity_mps = autoware_utils::get_longitudinal_velocity(ref_point);
48+
return traj_point;
5449
}
55-
} // namespace autoware_utils
5650

57-
namespace autoware::path_optimizer
58-
{
59-
namespace trajectory_utils
60-
{
6151
ReferencePoint convertToReferencePoint(const TrajectoryPoint & traj_point)
6252
{
6353
ReferencePoint ref_point;

planning/behavior_velocity_planner/autoware_behavior_velocity_intersection_module/src/scene_intersection_prepare_data.cpp

+2-2
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
#include <utility>
4040
#include <vector>
4141

42-
namespace autoware_utils
42+
namespace autoware_utils_geometry
4343
{
4444

4545
template <>
@@ -52,7 +52,7 @@ inline geometry_msgs::msg::Point get_point(const lanelet::ConstPoint3d & p)
5252
return point;
5353
}
5454

55-
} // namespace autoware_utils
55+
} // namespace autoware_utils_geometry
5656

5757
namespace
5858
{

0 commit comments

Comments
 (0)