258 behavior velocity occlusion spot stability #12
Conversation
Signed-off-by: tanaka3 <ttatcoder@outlook.jp>
…n area length Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>
Signed-off-by: taikitanaka3 <taiki.tanaka@tier4.jp>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -97,6 +98,7 @@ struct PlannerData | |||
| // additional parameters | |||
| double max_stop_acceleration_threshold; | |||
| double max_stop_jerk_threshold; | |||
| double system_delay; | |||
| double delay_response_time; | |||
| double stop_line_extend_length; | |||
There was a problem hiding this comment.
constructor does not initialize these fields: stop_line_extend_length, accel_lowpass_gain_
| double stop_line_extend_length; | |
| double stop_line_extend_length{}; |
| } | ||
|
|
||
| template <> | ||
| inline void doTransform( |
There was a problem hiding this comment.
function template specialization tf2::doTransform<geometry_msgs::msg::Point_<std::allocator<void> > > has a primary template declaration with different parameter names
| grid_map::GridMap & grid_map, const GridParam & param, const bool is_show_debug_window, | ||
| const bool filter_occupancy_grid) | ||
| const bool filter_occupancy_grid, const bool use_object_footprints, | ||
| const bool use_object_ray_casts) | ||
| { | ||
| OccupancyGrid occupancy_grid = *occupancy_grid_ptr; | ||
| cv::Mat cv_image( | ||
| occupancy_grid.info.width, occupancy_grid.info.height, CV_8UC1, |
There was a problem hiding this comment.
use of a signed integer operand with a binary bitwise operator
| } | ||
| } | ||
| debug_point = points; | ||
| return parked_vehicles; | ||
| return; |
There was a problem hiding this comment.
redundant return statement at the end of a function with a void return type
| return; | |
| @@ -62,6 +62,7 @@ class OcclusionSpotModule : public SceneModuleInterface | |||
| PlannerParam param_; | |||
| tier4_autoware_utils::StopWatch<std::chrono::milliseconds> stop_watch_; | |||
| std::vector<lanelet::BasicPolygon2d> partition_lanelets_; | |||
| double prev_detection_area_length_; | |||
There was a problem hiding this comment.
constructor does not initialize these fields: prev_detection_area_length_
| double prev_detection_area_length_; | |
| double prev_detection_area_length_{}; |
Note: Confirm the contribution guidelines before submitting a pull request.
Click the
Previewtab and select a PR template:Do NOT send a PR with this description.