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

test(planning_test_utils): improve-planning-interface-test #443

Merged

Conversation

kyoichi-sugahara
Copy link

@kyoichi-sugahara kyoichi-sugahara commented May 12, 2023

Description

merge PR from
autowarefoundation#3401
autowarefoundation#3404
autowarefoundation#3661
autowarefoundation#3695
autowarefoundation#3721

Tests performed

run colcon test with following command

colcon test --event-handlers console_cohesion+ --packages-select freespace_planner behavior_path_planner obstacle_avoidance_planner behavior_velocity_planner planning_validator motion_velocity_smoother obstacle_cruise_planner obstacle_stop_planner obstacle_velocity_limiter scenario_selector
Summary: 10 packages finished [19.6s]

All tests have passed.

Effects on system behavior

Not applicable.

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.

…ion#3404)

* add planning_interface_test_manager class

Signed-off-by: kyoichi-sugahara <[email protected]>

* add counter function

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param in default_motion_velocity_smoother,param.yaml

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully launch target node

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully build the test

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for test_motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add abnormal trajectory test

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary part

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* style(pre-commit): autofix

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add declaration

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor callback functions for standardization

Signed-off-by: kyoichi-sugahara <[email protected]>

* refacotoring

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactored

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor functions into a single template function

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* relete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "delete unnecessary definition"

This reverts commit 6cd13f8.

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete module dependent part

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary arg

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply motion_velocity_smoother change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle stop planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot obstacle_cruise_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot planning_vaildator

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for freespace_planner_node

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle_stop_planner with slow down

Signed-off-by: kyoichi-sugahara <[email protected]>

* add part of interface test for freespace

Signed-off-by: kyoichi-sugahara <[email protected]>

* change package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply change of package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp commit for debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* add planning interface test manager for scenario selector

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* update parameter

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp for behavior_path_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test free space planner module

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for behavior path planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map detection area

Signed-off-by: kyoichi-sugahara <[email protected]>

* add no stopping area

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* for print debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor(behavior_path_planner): remove unnecessary functions (autowarefoundation#3271)

* refactor(behavior_path_planner): remove unnecessary functions

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

---------

Signed-off-by: yutaka <[email protected]>

* Revert "temp"

This reverts commit b82805e.

Signed-off-by: kyoichi-sugahara <[email protected]>

* suppress build error (-Werror=pedantic)

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for behavior_velocity_planner

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* Resolve differences outside of parameter files

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* upload cloud map

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix typo

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle cruise planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle_stop_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* revert parameter change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "add obstacle_avoidance_planner"

This reverts commit efb5d50.

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* feat(planning_test_utils): update test manager for all planning modules (autowarefoundation#3364)

* add planning_interface_test_manager class

Signed-off-by: kyoichi-sugahara <[email protected]>

* add counter function

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param in default_motion_velocity_smoother,param.yaml

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully launch target node

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully build the test

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for test_motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add abnormal trajectory test

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary part

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* style(pre-commit): autofix

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add declaration

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor callback functions for standardization

Signed-off-by: kyoichi-sugahara <[email protected]>

* refacotoring

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactored

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor functions into a single template function

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* relete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "delete unnecessary definition"

This reverts commit 6cd13f8.

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete module dependent part

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary arg

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply motion_velocity_smoother change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle stop planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot obstacle_cruise_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot planning_vaildator

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for freespace_planner_node

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle_stop_planner with slow down

Signed-off-by: kyoichi-sugahara <[email protected]>

* add part of interface test for freespace

Signed-off-by: kyoichi-sugahara <[email protected]>

* change package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply change of package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp commit for debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* add planning interface test manager for scenario selector

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* update parameter

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp for behavior_path_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test free space planner module

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for behavior path planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map detection area

Signed-off-by: kyoichi-sugahara <[email protected]>

* add no stopping area

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* for print debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor(behavior_path_planner): remove unnecessary functions (autowarefoundation#3271)

* refactor(behavior_path_planner): remove unnecessary functions

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

---------

Signed-off-by: yutaka <[email protected]>

* Revert "temp"

This reverts commit b82805e.

Signed-off-by: kyoichi-sugahara <[email protected]>

* suppress build error (-Werror=pedantic)

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for behavior_velocity_planner

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* Resolve differences outside of parameter files

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* upload cloud map

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix typo

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle cruise planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle_stop_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* revert parameter change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "add obstacle_avoidance_planner"

This reverts commit efb5d50.

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify CMake

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

---------

Signed-off-by: kyoichi-sugahara <[email protected]>
Signed-off-by: yutaka <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Yutaka Shimizu <[email protected]>
Signed-off-by: kyoichi-sugahara <[email protected]>

* fix config name

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary files

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix node name in commnet

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

---------

Signed-off-by: kyoichi-sugahara <[email protected]>
Signed-off-by: yutaka <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Yutaka Shimizu <[email protected]>
@kyoichi-sugahara kyoichi-sugahara changed the title test(obstacle_avoidance_planner): add interface test (#3404) test(planning_test_utils): improve-planning-interface-test May 12, 2023
kyoichi-sugahara and others added 10 commits May 13, 2023 00:59
…on#3401)

* add planning_interface_test_manager class

Signed-off-by: kyoichi-sugahara <[email protected]>

* add counter function

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param in default_motion_velocity_smoother,param.yaml

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully launch target node

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully build the test

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for test_motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add abnormal trajectory test

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary part

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* style(pre-commit): autofix

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add declaration

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor callback functions for standardization

Signed-off-by: kyoichi-sugahara <[email protected]>

* refacotoring

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactored

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor functions into a single template function

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* relete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "delete unnecessary definition"

This reverts commit 6cd13f8.

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete module dependent part

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary arg

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply motion_velocity_smoother change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle stop planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot obstacle_cruise_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot planning_vaildator

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for freespace_planner_node

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle_stop_planner with slow down

Signed-off-by: kyoichi-sugahara <[email protected]>

* add part of interface test for freespace

Signed-off-by: kyoichi-sugahara <[email protected]>

* change package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply change of package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp commit for debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* add planning interface test manager for scenario selector

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* update parameter

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp for behavior_path_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test free space planner module

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for behavior path planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map detection area

Signed-off-by: kyoichi-sugahara <[email protected]>

* add no stopping area

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* for print debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor(behavior_path_planner): remove unnecessary functions (autowarefoundation#3271)

* refactor(behavior_path_planner): remove unnecessary functions

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

---------

Signed-off-by: yutaka <[email protected]>

* Revert "temp"

This reverts commit b82805e.

Signed-off-by: kyoichi-sugahara <[email protected]>

* suppress build error (-Werror=pedantic)

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for behavior_velocity_planner

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* Resolve differences outside of parameter files

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* upload cloud map

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix typo

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle cruise planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle_stop_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* revert parameter change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "add obstacle_avoidance_planner"

This reverts commit efb5d50.

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify CMake

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* test success

Signed-off-by: kyoichi-sugahara <[email protected]>

* execption test is added

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary debug code

Signed-off-by: kyoichi-sugahara <[email protected]>

* add prototype declaration

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary comments

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* feat(planning_test_utils): update test manager for all planning modules (autowarefoundation#3364)

* add planning_interface_test_manager class

Signed-off-by: kyoichi-sugahara <[email protected]>

* add counter function

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param in default_motion_velocity_smoother,param.yaml

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully launch target node

Signed-off-by: kyoichi-sugahara <[email protected]>

* successfully build the test

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for test_motion_velocity_smoother

Signed-off-by: kyoichi-sugahara <[email protected]>

* add abnormal trajectory test

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary part

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* style(pre-commit): autofix

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add declaration

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor callback functions for standardization

Signed-off-by: kyoichi-sugahara <[email protected]>

* refacotoring

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactored

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor functions into a single template function

Signed-off-by: kyoichi-sugahara <[email protected]>

* Refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* style(pre-commit): autofix

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* relete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "delete unnecessary definition"

This reverts commit 6cd13f8.

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary definition

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete module dependent part

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary arg

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply motion_velocity_smoother change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle stop planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* run pre-commit

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot obstacle_cruise_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test fot planning_vaildator

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for freespace_planner_node

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for obstacle_stop_planner with slow down

Signed-off-by: kyoichi-sugahara <[email protected]>

* add part of interface test for freespace

Signed-off-by: kyoichi-sugahara <[email protected]>

* change package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* apply change of package name

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp commit for debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* add planning interface test manager for scenario selector

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* update parameter

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp for behavior_path_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test free space planner module

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* add test for behavior path planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass freespace test

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map

Signed-off-by: kyoichi-sugahara <[email protected]>

* update map detection area

Signed-off-by: kyoichi-sugahara <[email protected]>

* add no stopping area

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* for print debug

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor(behavior_path_planner): remove unnecessary functions (autowarefoundation#3271)

* refactor(behavior_path_planner): remove unnecessary functions

Signed-off-by: yutaka <[email protected]>

* update

Signed-off-by: yutaka <[email protected]>

---------

Signed-off-by: yutaka <[email protected]>

* Revert "temp"

This reverts commit b82805e.

Signed-off-by: kyoichi-sugahara <[email protected]>

* suppress build error (-Werror=pedantic)

Signed-off-by: kyoichi-sugahara <[email protected]>

* add interface test for behavior_velocity_planner

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* add param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* pass test

Signed-off-by: kyoichi-sugahara <[email protected]>

* build success

Signed-off-by: kyoichi-sugahara <[email protected]>

* update param

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete param file

Signed-off-by: kyoichi-sugahara <[email protected]>

* Resolve differences outside of parameter files

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* upload cloud map

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix typo

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor utils

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle cruise planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify test for obstacle_stop_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* revert parameter change

Signed-off-by: kyoichi-sugahara <[email protected]>

* add obstacle_avoidance_planner

Signed-off-by: kyoichi-sugahara <[email protected]>

* Revert "add obstacle_avoidance_planner"

This reverts commit efb5d50.

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* change file name

Signed-off-by: kyoichi-sugahara <[email protected]>

* modify CMake

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

* refactor

Signed-off-by: kyoichi-sugahara <[email protected]>

---------

Signed-off-by: kyoichi-sugahara <[email protected]>
Signed-off-by: yutaka <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Yutaka Shimizu <[email protected]>
Signed-off-by: kyoichi-sugahara <[email protected]>

* fix config name

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary files

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* temp

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* update test_manager

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix node name in commnet

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* fix build error

Signed-off-by: kyoichi-sugahara <[email protected]>

* delete unnecessary publisher from test_node

Signed-off-by: kyoichi-sugahara <[email protected]>

* set flag for launch scene module

Signed-off-by: kyoichi-sugahara <[email protected]>

* add ego pose test with comment out

Signed-off-by: kyoichi-sugahara <[email protected]>

* add username in TODO

Signed-off-by: kyoichi-sugahara <[email protected]>

* enable off-track test

Signed-off-by: Takamasa Horibe <[email protected]>

---------

Signed-off-by: kyoichi-sugahara <[email protected]>
Signed-off-by: yutaka <[email protected]>
Signed-off-by: Takamasa Horibe <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: Yutaka Shimizu <[email protected]>
Co-authored-by: Takamasa Horibe <[email protected]>
…timation via non initialized eigen vector (autowarefoundation#3651) (autowarefoundation#423)

add initialize for anchor point vector

Signed-off-by: yoshiri <[email protected]>
Co-authored-by: Yoshi Ri <[email protected]>
…warefoundation#3685)

* fix(lane_change): safety check not working when cancel disabled

Signed-off-by: Muhammad Zulfaqar <[email protected]>

* remove header

Signed-off-by: Muhammad Zulfaqar <[email protected]>

* override updateState for new and old arch.

Signed-off-by: Muhammad Zulfaqar <[email protected]>

* Revert "override updateState for new and old arch."

This reverts commit 4c509e3.

---------

Signed-off-by: Muhammad Zulfaqar <[email protected]>
…towarefoundation#3694)

* fix(lane_change): not remove unsafe candidate in new architecture

Signed-off-by: Muhammad Zulfaqar <[email protected]>

* By pass current_state_ check in isExecutionRequested

Signed-off-by: Muhammad Zulfaqar <[email protected]>

* Compile with old architecture

Signed-off-by: Muhammad Zulfaqar <[email protected]>

---------

Signed-off-by: Muhammad Zulfaqar <[email protected]>
…n appeared/disappeared or approved/disapproved (autowarefoundation#3664)

Signed-off-by: Mamoru Sobue <[email protected]>
autowarefoundation#481)

fix(behavior_path_planner): handle overlap lane in isEgoOutOfRoute() (autowarefoundation#3715)

Signed-off-by: Fumiya Watanabe <[email protected]>
Co-authored-by: Fumiya Watanabe <[email protected]>
…utowarefoundation#3695)

* test(planning_test_utils): add ASSART_NO_THROW_WITH_ERROR_MSG macro

Signed-off-by: Takamasa Horibe <[email protected]>

* update for behavior_velocity_planner

Signed-off-by: Takamasa Horibe <[email protected]>

* fix for exception error msg

Signed-off-by: Takamasa Horibe <[email protected]>

---------

Signed-off-by: Takamasa Horibe <[email protected]>
@tkimura4 tkimura4 enabled auto-merge May 16, 2023 05:19
@kyoichi-sugahara kyoichi-sugahara marked this pull request as draft May 16, 2023 05:20
auto-merge was automatically disabled May 16, 2023 05:20

Pull request was converted to draft

@kyoichi-sugahara kyoichi-sugahara marked this pull request as ready for review May 16, 2023 05:22
@codecov-commenter
Copy link

codecov-commenter commented May 16, 2023

Codecov Report

Patch coverage: 19.53% and project coverage change: +11.63 🎉

Comparison is base (63ca797) 3.44% compared to head (786ee53) 15.07%.

❗ 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                @@
##           beta/v0.8.0     #443       +/-   ##
================================================
+ Coverage         3.44%   15.07%   +11.63%     
================================================
  Files               99     1411     +1312     
  Lines             7485    99113    +91628     
  Branches          4099    29793    +25694     
================================================
+ Hits               258    14944    +14686     
- Misses            7033    68141    +61108     
- Partials           194    16028    +15834     
Flag Coverage Δ *Carryforward flag
differential 17.12% <17.64%> (+13.67%) ⬆️
total 13.84% <19.40%> (?) Carriedforward from 32529e7

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

Impacted Files Coverage Δ
...nterface/include/osqp_interface/osqp_interface.hpp 0.00% <ø> (ø)
common/osqp_interface/src/csc_matrix_conv.cpp 66.03% <ø> (ø)
...ommon/osqp_interface/test/test_csc_matrix_conv.cpp 9.70% <ø> (ø)
common/osqp_interface/test/test_osqp_interface.cpp 15.00% <ø> (ø)
...rviz_plugin/src/pose_history_footprint/display.cpp 0.00% <ø> (ø)
...ier4_planning_rviz_plugin/include/path/display.hpp 0.00% <ø> (ø)
...planning_rviz_plugin/include/path/display_base.hpp 0.00% <ø> (ø)
...on/tier4_planning_rviz_plugin/src/path/display.cpp 0.00% <0.00%> (ø)
...viz_plugin/src/velocity_steering_factors_panel.cpp 0.00% <0.00%> (ø)
...ht_rviz_plugin/src/traffic_light_publish_panel.cpp 0.00% <0.00%> (ø)
... and 59 more

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

@tkimura4 tkimura4 enabled auto-merge May 16, 2023 07:51
Copy link
Collaborator

@tkimura4 tkimura4 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tkimura4 tkimura4 merged commit 9e14ea4 into tier4:beta/v0.8.0 May 16, 2023
@kyoichi-sugahara kyoichi-sugahara deleted the feat/improve-planning-interface-test branch July 22, 2024 02:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants