feat(tier4_autoware_utils): add time argument in appendMarkerArray

[takayuki5168]

Signed-off-by: Takayuki Murooka takayuki5168@gmail.com

Description

No behavior changes of autoware
add time argument in appendMarkerArray

In order to make following appendMarkerArray in behavior velocity planner common

autoware.universe/planning/behavior_velocity_planner/src/scene_module/detection_area/debug.cpp

Lines 160 to 162 in 1622892

	appendMarkerArray(
	createCorrespondenceMarkerArray(detection_area_reg_elem_, current_time), current_time,
	&wall_marker);

autoware.universe/planning/behavior_velocity_planner/include/utilization/marker_helper.hpp

Lines 92 to 101 in 1622892

	inline void appendMarkerArray(
	const visualization_msgs::msg::MarkerArray & additional_marker_array,
	const builtin_interfaces::msg::Time current_time,
	visualization_msgs::msg::MarkerArray * marker_array)
	{
	for (const auto & marker : additional_marker_array.markers) {
	marker_array->markers.push_back(marker);
	marker_array->markers.back().header.stamp = current_time;
	}
	}

Pre-review checklist for the PR author

The PR author must check the checkboxes below when creating the PR.

• I've confirmed the contribution guidelines.
• The PR follows the pull request guidelines.

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

The PR author must check the checkboxes below before merging.

• There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

[TakaHoribe]

Is this function not used anywhere?
If the build passes, then ok.

[takayuki5168]

@TakaHoribe
Sorry, I added a default argument some minutes ago.
73d60ec

This function is used everywhere in planning. With an ↑ argument, the build will pass.

[taikitanaka3]

LGTM

[codecov]

Codecov Report

Merging #1415 (73d60ec) into main (123a472) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1415      +/-   ##
==========================================
- Coverage   10.08%   10.03%   -0.05%     
==========================================
  Files        1117     1117              
  Lines       77287    77626     +339     
  Branches    17852    17860       +8     
==========================================
  Hits         7793     7793              
- Misses      62145    62484     +339     
  Partials     7349     7349              

Flag	Coverage Δ		*Carryforward flag
differential	10.68% <ø> (?)
total	10.06% <ø> (ø)		Carriedforward from 123a472

*This pull request uses carry forward flags. Click here to find out more.

Impacted Files	Coverage Δ
...or_path_planner/src/behavior_path_planner_node.cpp	0.22% <0.00%> (-0.01%)	⬇️
planning/obstacle_cruise_planner/src/node.cpp	0.00% <0.00%> (ø)
...lanning/obstacle_stop_planner/src/debug_marker.cpp	0.00% <0.00%> (ø)
...ning/obstacle_cruise_planner/src/polygon_utils.cpp	0.00% <0.00%> (ø)
...ing/surround_obstacle_checker/src/debug_marker.cpp	0.00% <0.00%> (ø)
.../obstacle_cruise_planner/src/planner_interface.cpp	0.00% <0.00%> (ø)
...acle_avoidance_planner/src/debug_visualization.cpp	0.00% <0.00%> (ø)
...elocity_planner/src/scene_module/run_out/debug.cpp	0.00% <0.00%> (ø)
...ocity_planner/src/scene_module/crosswalk/debug.cpp	0.00% <0.00%> (ø)
...lanner/src/pid_based_planner/pid_based_planner.cpp	0.00% <0.00%> (ø)
... and 3 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 123a472...73d60ec. Read the comment docs.