Skip to content

Commit 2ad15b1

Browse files
chore(trajectory_validator): remove a trajectory_validator plugin (#12522)
* delete collision filter --------- Signed-off-by: yuki-takagi-66 <yuki.takagi@tier4.jp>
1 parent 8cf35f3 commit 2ad15b1

8 files changed

Lines changed: 10 additions & 406 deletions

File tree

planning/autoware_trajectory_validator/CMakeLists.txt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,6 @@ rclcpp_components_register_node(${trajectory_validator_lib_target}
3434

3535
ament_auto_add_library(autoware_trajectory_validator_plugins SHARED
3636
src/filters/safety/out_of_lane_filter.cpp
37-
src/filters/safety/collision_filter.cpp
3837
src/filters/safety/vehicle_constraint_filter.cpp
3938
src/filters/traffic_rule/traffic_light_filter.cpp
4039
)

planning/autoware_trajectory_validator/README.md

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@ Checks applied to each trajectory:
1212

1313
- Data validity: removes trajectories that contain NaNs, non‑finite numbers, inconsistent timestamps, or are too short.
1414
- Lane adherence: removes trajectories that will exit all lanelets within the configured look‑ahead time.
15-
- Collision risk: removes trajectories whose estimated time‑to‑collision with any predicted object falls below threshold in the look‑ahead time.
1615
- Vehicle constraint: removes trajectories that violates vehicle constraints, such as maximum speed, acceleration, and deceleration.
1716

1817
After these checks, the remaining trajectories, along with their original `generator_info`, are published.
@@ -31,18 +30,16 @@ After these checks, the remaining trajectories, along with their original `gener
3130

3231
### Parameters
3332

34-
| Parameter name | Type | Default | Description |
35-
| --------------------------------------- | ------------ | ------- | ----------------------------------------------------------------------------- |
36-
| `filter_names` | string array | [] | List of safety filter plugins to use (e.g., OutOfLaneFilter, CollisionFilter) |
37-
| `out_of_lane.time` | double | 3.0 | Look-ahead time [s] during which the trajectory must stay inside a lane |
38-
| `out_of_lane.min_value` | double | 0.0 | Minimum distance [m] from lane boundary |
39-
| `collision.time` | double | 3.0 | Look-ahead time [s] for collision search |
40-
| `collision.min_value` | double | 2.0 | Minimum acceptable time to collision [s] |
41-
| `vehicle_constraint.max_speed` | double | 16.7 | Maximum allowed speed [m/s] |
42-
| `vehicle_constraint.max_acceleration` | double | 5.0 | Maximum allowed acceleration [m/s^2] |
43-
| `vehicle_constraint.max_deceleration` | double | 5.0 | Maximum allowed deceleration; positive but represents deceleration [m/s^s] |
44-
| `vehicle_constraint.max_steering_angle` | double | 0.8 | Maximum allowed steering angle [rad] |
45-
| `vehicle_constraint.max_steering_rate` | double | 0.3 | Maximum allowed steering rate [rad/s] |
33+
| Parameter name | Type | Default | Description |
34+
| --------------------------------------- | ------------ | ------- | -------------------------------------------------------------------------- |
35+
| `filter_names` | string array | [] | List of safety filter plugins to use (e.g., OutOfLaneFilter) |
36+
| `out_of_lane.time` | double | 3.0 | Look-ahead time [s] during which the trajectory must stay inside a lane |
37+
| `out_of_lane.min_value` | double | 0.0 | Minimum distance [m] from lane boundary |
38+
| `vehicle_constraint.max_speed` | double | 16.7 | Maximum allowed speed [m/s] |
39+
| `vehicle_constraint.max_acceleration` | double | 5.0 | Maximum allowed acceleration [m/s^2] |
40+
| `vehicle_constraint.max_deceleration` | double | 5.0 | Maximum allowed deceleration; positive but represents deceleration [m/s^s] |
41+
| `vehicle_constraint.max_steering_angle` | double | 0.8 | Maximum allowed steering angle [rad] |
42+
| `vehicle_constraint.max_steering_rate` | double | 0.3 | Maximum allowed steering rate [rad/s] |
4643

4744
## Future Work
4845

planning/autoware_trajectory_validator/config/trajectory_validator.param.yaml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66
shadow_mode_filter_names:
77
[
88
"safety::OutOfLaneFilter",
9-
"safety::CollisionFilter",
109
"safety::VehicleConstraintFilter",
1110
"traffic_rule::TrafficLightFilter",
1211
]
@@ -15,10 +14,6 @@
1514
time: 2.0
1615
min_value: 0.0
1716

18-
collision:
19-
time: 3.0
20-
min_value: 2.0
21-
2217
vehicle_constraint:
2318
max_speed: 16.7 # m/s
2419
max_acceleration: 5.0 # m/s^2

planning/autoware_trajectory_validator/include/autoware/trajectory_validator/filters/safety/collision_filter.hpp

Lines changed: 0 additions & 73 deletions
This file was deleted.

planning/autoware_trajectory_validator/param/parameter_struct.yaml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -23,18 +23,6 @@ validator:
2323
description: Minimum distance to lane boundary
2424
read_only: false
2525

26-
collision:
27-
time:
28-
type: double
29-
default_value: 3.0
30-
description: The time range to check
31-
read_only: false
32-
min_value:
33-
type: double
34-
default_value: 2.0
35-
description: Minimum acceptable time to collision
36-
read_only: false
37-
3826
vehicle_constraint:
3927
max_speed:
4028
type: double

planning/autoware_trajectory_validator/plugins.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,12 +6,6 @@
66
Filters trajectories that go out of lane boundaries
77
</description>
88
</class>
9-
<class type="safety::CollisionFilter"
10-
base_class_type="autoware::trajectory_validator::plugin::ValidatorInterface">
11-
<description>
12-
Filters trajectories based on Time To Collision (TTC) with predicted objects
13-
</description>
14-
</class>
159
<class type="safety::VehicleConstraintFilter"
1610
base_class_type="autoware::trajectory_validator::plugin::ValidatorInterface">
1711
<description>

0 commit comments

Comments
 (0)