fix(behavior_path_planner, motion_velocity_smoother): supress unnecessary output

[tkimura4]

Description

supress unnecessary output

1. motion_velocity_smoother (L2)
   When the vehicle speed is low, apply function returns false and the following messages are output repeteadly.
   Fail to solve optimization.

autoware.universe/planning/motion_velocity_smoother/src/smoother/l2_pseudo_jerk_smoother.cpp

Lines 52 to 56 in 007487a

	if (std::fabs(input.front().longitudinal_velocity_mps) < 0.1) {
	RCLCPP_DEBUG(logger_, "closest v_max < 0.1. assume vehicle stopped. return.");
	return false;
	}

autoware.universe/planning/motion_velocity_smoother/src/motion_velocity_smoother_node.cpp

Line 632 in 007487a

RCLCPP_WARN(get_logger(), "Fail to solve optimization.");

However, in this case, the optimization has not failed and there is no need to output the avove message.
So, I fixed not to output the above message.

2. turn_signal_decider in behavior_path_planner
   When convertToFrenetCoordinate3d returns false, the following message are output repeadly.
   failed to convert vehicle pose into frenet coordinate

(This function does the conversion of self-position to Frenet coordinates.)

However, if the self-position exceeds the goal even a little, the processing of this function will always fail.
So, I changed the output type of this function from ERROR to DEBUG.

autoware.universe/planning/behavior_path_planner/src/turn_signal_decider.cpp

Lines 60 to 63 in 007487a

	if (!util::convertToFrenetCoordinate3d(path, current_pose.position, &vehicle_pose_frenet)) {
	RCLCPP_ERROR_THROTTLE(
	logger_, clock, 5000, "failed to convert vehicle pose into frenet coordinate");
	return std::make_pair(turn_signal, distance);

Pre-review checklist for the PR author

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

Reviewers must check the checkboxes below before approval.

• The PR follows the pull request guidelines.

Post-review checklist for the PR author

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 the write access can merge the PR.

[codecov]

Codecov Report

Merging #579 (88de003) into main (007487a) will decrease coverage by 10.91%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main    #579       +/-   ##
==========================================
- Coverage   10.91%   0.00%   -10.92%     
==========================================
  Files         706      44      -662     
  Lines       49835    6656    -43179     
  Branches     7116       0     -7116     
==========================================
- Hits         5438       0     -5438     
+ Misses      40016    6656    -33360     
+ Partials     4381       0     -4381     

Impacted Files	Coverage Δ
.../behavior_path_planner/src/turn_signal_decider.cpp	0.00% <0.00%> (ø)
..._smoother/src/smoother/l2_pseudo_jerk_smoother.cpp	0.00% <0.00%> (ø)
...nclude/lidar_apollo_instance_segmentation/util.hpp
map/lanelet2_extension/lib/road_marking.cpp
.../src/distortion_corrector/distortion_corrector.cpp
...n/include/traffic_light_roi_visualizer/nodelet.hpp
common/signal_processing/src/lowpass_filter_1d.cpp
common/autoware_auto_common/test/gtest_main.cpp
...nodes/costmap_generator/costmap_generator_node.cpp
..._planner/src/scene_module/detection_area/scene.cpp
... and 654 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 007487a...88de003. Read the comment docs.

[TakaHoribe]

LGTM. (This msg is not critical or not even error.)