Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(behavior_velocity_crosswalk_module): clean up the code #4157

Merged
merged 6 commits into from
Jul 6, 2023

Conversation

takayuki5168
Copy link
Contributor

@takayuki5168 takayuki5168 commented Jul 4, 2023

Description

clean up the code of behavior_velocity_crosswalk_module

Tests performed

planning simulator
scenario simulator: https://evaluation.tier4.jp/evaluation/reports/3b6abef0-995f-5e5a-8cf2-83cfd5ec5289?project_id=prd_jt (no degradation)

Effects on system behavior

nothing

Pre-review checklist for the PR author

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

In-review checklist for the PR reviewers

The PR reviewers must check the checkboxes below before approval.

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.

@github-actions github-actions bot added component:planning Route planning, decision-making, and navigation. (auto-assigned) component:common Common packages from the autoware-common repository. (auto-assigned) labels Jul 4, 2023
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
@takayuki5168 takayuki5168 marked this pull request as ready for review July 5, 2023 13:53
@codecov
Copy link

codecov bot commented Jul 5, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.32 ⚠️

Comparison is base (c9280db) 15.23% compared to head (c724ae8) 14.91%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4157      +/-   ##
==========================================
- Coverage   15.23%   14.91%   -0.32%     
==========================================
  Files        1468     1516      +48     
  Lines      101783   103930    +2147     
  Branches    31405    31396       -9     
==========================================
+ Hits        15503    15506       +3     
- Misses      69417    71563    +2146     
+ Partials    16863    16861       -2     
Flag Coverage Δ *Carryforward flag
differential 17.54% <0.00%> (?)
total 15.24% <ø> (+0.01%) ⬆️ Carriedforward from c9280db

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

Impacted Files Coverage Δ
...e/tier4_autoware_utils/geometry/boost_geometry.hpp 50.00% <ø> (ø)
..._velocity_crosswalk_module/src/scene_crosswalk.cpp 0.00% <0.00%> (ø)
...or_velocity_crosswalk_module/src/scene_walkway.cpp 0.00% <0.00%> (ø)
...ng/behavior_velocity_crosswalk_module/src/util.cpp 0.00% <0.00%> (ø)

... and 49 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@takayuki5168 takayuki5168 merged commit d905e38 into autowarefoundation:main Jul 6, 2023
@takayuki5168 takayuki5168 deleted the refactor/crosswalk branch July 6, 2023 00:56
takayuki5168 added a commit to tier4/autoware.universe that referenced this pull request Jul 19, 2023
…warefoundation#4157)

* refactor

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

* use tier4_autoware_utils's boost types

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

* return Point instead of Point2d

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

* remove unnecessary lambda function

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

* add Line2d in tier4_autoware_utils

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

* remove unnecessary member variable

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

---------

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
takayuki5168 added a commit to tier4/autoware.universe that referenced this pull request Aug 9, 2023
…warefoundation#4157)

* refactor

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

* use tier4_autoware_utils's boost types

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

* return Point instead of Point2d

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

* remove unnecessary lambda function

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

* add Line2d in tier4_autoware_utils

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

* remove unnecessary member variable

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

---------

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:common Common packages from the autoware-common repository. (auto-assigned) component:planning Route planning, decision-making, and navigation. (auto-assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants