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

chore: sync upstream #559

Merged
merged 6 commits into from
Jun 7, 2023
Merged

chore: sync upstream #559

merged 6 commits into from
Jun 7, 2023

Conversation

tier4-autoware-public-bot[bot]
Copy link

TakaHoribe and others added 6 commits June 6, 2023 14:01
…utowarefoundation#3901)

* remove unnecessary casting

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use return value

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* add warn function

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* update comment format

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor debug diag

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use return value convertToAutowareTrajectory

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* add warn_throttle

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* using namespace

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use at() for index access

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* replace local function with autoware_util

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor mpc_util

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use function

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* functionalize calculateMPC func

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use MPCWeight struct

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor member variable descriptions

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove each getWeight function

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* traj -> trajectory

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* replace vel + pose to odometry

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove unused TF

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* change arg type

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove model_type member

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor testing

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use lambda for declare parameter"

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* make struct for trajectory smoothing param

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* define log in mpc_lateral_controller,hpp

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* add comment on the parameter

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* update descriotion

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* update comment

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* separate steering_predictor

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* use back()

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* remove namespace for ackermannControl

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* move update_param to util

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* clean up constructor

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* minor clean up

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* clean up include

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* replace mpc interploation with common interpolation

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor dynamic velocity filtering

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor some arguments

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* refactor logger

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

* fix test for steering_predictor initialization

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>

---------

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
…n#3591)

Signed-off-by: Takamasa Horibe <horibe.takamasa@gmail.com>
…ndation#3842)

* feat(mission_planner): add guard for reroute

Signed-off-by: yutaka <purewater0901@gmail.com>

* feat(mission_planner): add modified goal implementation

Signed-off-by: yutaka <purewater0901@gmail.com>

* update

Signed-off-by: yutaka <purewater0901@gmail.com>

* update

Signed-off-by: yutaka <purewater0901@gmail.com>

* update

Signed-off-by: yutaka <purewater0901@gmail.com>

* update

Signed-off-by: yutaka <purewater0901@gmail.com>

* update

Signed-off-by: yutaka <purewater0901@gmail.com>

* remove modified goal

Signed-off-by: yutaka <purewater0901@gmail.com>

---------

Signed-off-by: yutaka <purewater0901@gmail.com>
Co-authored-by: Takagi, Isamu <43976882+isamu-takagi@users.noreply.github.com>
…towarefoundation#3906)

Signed-off-by: Muhammad Zulfaqar Azmi <zulfaqar.azmi@tier4.jp>
…n#3907)

Signed-off-by: tomoya.kimura <tomoya.kimura@tier4.jp>
fix(behavior_path_planner): clear stop reason

Signed-off-by: satoshi-ota <satoshi.ota928@gmail.com>
@codecov-commenter
Copy link

Codecov Report

Patch coverage: 44.70% and project coverage change: +1.17 🎉

Comparison is base (dc9282d) 12.41% compared to head (1375cf5) 13.58%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@              Coverage Diff               @@
##           tier4/main     #559      +/-   ##
==============================================
+ Coverage       12.41%   13.58%   +1.17%     
==============================================
  Files             112      249     +137     
  Lines           13387    24012   +10625     
  Branches         7240     9334    +2094     
==============================================
+ Hits             1662     3263    +1601     
- Misses           9599    17110    +7511     
- Partials         2126     3639    +1513     
Flag Coverage Δ
differential 13.58% <44.70%> (+1.17%) ⬆️
Impacted Files Coverage Δ
...e/object_detection/object_polygon_display_base.hpp 0.00% <ø> (ø)
.../include/mpc_lateral_controller/mpc_trajectory.hpp 50.00% <ø> (ø)
...pc_lateral_controller/qp_solver/qp_solver_osqp.hpp 0.00% <ø> (ø)
...l_controller/qp_solver/qp_solver_unconstr_fast.hpp 100.00% <ø> (ø)
...ntroller/src/qp_solver/qp_solver_unconstr_fast.cpp 83.33% <ø> (ø)
...anner/include/behavior_path_planner/parameters.hpp 50.00% <ø> (ø)
...th_planner/scene_module/scene_module_interface.hpp 30.37% <0.00%> (-0.20%) ⬇️
...or_path_planner/src/behavior_path_planner_node.cpp 18.69% <0.00%> (-0.05%) ⬇️
...g/behavior_path_planner/src/utils/safety_check.cpp 28.34% <0.00%> (ø)
...on_planner/src/mission_planner/mission_planner.cpp 0.00% <0.00%> (ø)
... and 15 more

... and 152 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.

@tier4-autoware-public-bot tier4-autoware-public-bot bot merged commit 7d45f48 into tier4/main Jun 7, 2023
@tier4-autoware-public-bot tier4-autoware-public-bot bot deleted the sync-upstream branch June 7, 2023 03:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants