fix(behavior_velocity): fix detection area ,stop line,traffic lights stop line insertion ,occlusion spot

[taikitanaka3]

Signed-off-by: tanaka3 ttatcoder@outlook.jp

Related Issue(required)

#504

Description(required)

Solve problem
when blinspot walkway crosswalk is enabled at the same time duplicated stop line is inserted, then behavior container dies.
In this case longitudinalOffsetToSegment in calcSignedArcLength died with same stop point inserted.

fix-2022-03-14_13.19.07.mp4

this PR fix with util

• walkway
• no stopping area
• crosswalk

this PR wont refactor

• blindspot
• intersection

anther fix is necessary for

• detection area
• stop line
• traffic light
• virtual traffic light

this PR is

• going to fix edge case with minimal change
• not going to change whole module with refactoring

Review Procedure(required)

launch scenario sim and see there is no issue.

fix-2022-03-10_19.09.03.mp4

Related PR(optional)

Pre-Review Checklist for the PR Author

PR Author should check the checkboxes below when creating the PR.

• Read commit-guidelines
• Assign PR to reviewer

If you are adding new package following items are required:

• Documentation with description of the package is available
• A sample launch file and parameter file are available if the package contains executable nodes

Checklist for the PR Reviewer

Reviewers should check the checkboxes below before approval.

• Commits are properly organized and messages are according to the guideline
• PR title describes the changes

Post-Review Checklist for the PR Author

PR Author should check the checkboxes below before merging.

• All open points are addressed and tracked via issues or tickets

CI Checks

• Build and test for PR / build-and-test-pr: Required to pass before the merge.
• Build and test for PR / clang-tidy-pr: NOT required to pass before the merge. It is up to the reviewer(s). Found false positives? See the [guidelines][clang-tidy-guidelines].
• Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

[codecov]

Codecov Report

Merging #505 (f931220) into main (a03ab1f) will decrease coverage by 4.69%.
The diff coverage is 42.10%.

@@            Coverage Diff            @@
##             main    #505      +/-   ##
=========================================
- Coverage   10.93%   6.24%   -4.70%     
=========================================
  Files         705      70     -635     
  Lines       49780    4983   -44797     
  Branches     7085     697    -6388     
=========================================
- Hits         5442     311    -5131     
+ Misses      39976    4527   -35449     
+ Partials     4362     145    -4217     

Impacted Files	Coverage Δ
...vior_velocity_planner/include/utilization/util.hpp	8.00% <ø> (-4.00%)	⬇️
...city_planner/src/scene_module/blind_spot/scene.cpp	0.00% <0.00%> (ø)
...locity_planner/src/scene_module/crosswalk/util.cpp	0.00% <0.00%> (ø)
..._planner/src/scene_module/detection_area/scene.cpp	0.00% <0.00%> (ø)
...module/no_stopping_area/scene_no_stopping_area.cpp	0.00% <0.00%> (ø)
...ocity_planner/src/scene_module/stop_line/scene.cpp	0.00% <0.00%> (ø)
...y_planner/src/scene_module/traffic_light/scene.cpp	0.00% <0.00%> (ø)
...r/src/scene_module/virtual_traffic_light/scene.cpp	0.00% <0.00%> (ø)
..._module/occlusion_spot/risk_predictive_braking.cpp	50.70% <50.00%> (-6.06%)	⬇️
...behavior_velocity_planner/src/utilization/util.cpp	37.19% <81.25%> (+4.76%)	⬆️
... and 644 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 a03ab1f...f931220. Read the comment docs.

[taikitanaka3]

• release note & enabled auto merge