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

feat: v0.3.19 rebase #1540

Merged
merged 12 commits into from
Sep 18, 2024
Merged

feat: v0.3.19 rebase #1540

merged 12 commits into from
Sep 18, 2024

Conversation

asa-naki
Copy link

@asa-naki asa-naki commented Sep 18, 2024

Description

本PRでは、#1290, #1285 の取り込みを行う。
v0.3.16.1のブランチに対して、beta/v0.3.18.1のrebaseを実施する。
rebaseによって、本来、beta/v0.3.19には不要なPRも同時に取り込みが実施されていたため、revertを実施した。
該当するrevertするべきPRは

revertについては

Related links

INTERNAL link

How was this PR tested?

対象のブランチに切り替えて、colcon buildが通ることを確認しました。

Notes for reviewers

過去に反映済みのコードのため、設計者確認は、適応漏れチェックのみで、
機能評価は、リリース評価で実施する。

revertでも追加でもないcommitが複数存在しますが、既に、v0.3.16.1で適応済みのため、
本PRの変更対象外であることを確認しました。

revert対象が上記の5つであることの確認はINTERNAL link
添付の資料に記載してあります。

Interface changes

None.

Effects on system behavior

None.

h-ohta and others added 10 commits September 17, 2024 20:32
…efoundation#4553) (#768)

fix(system_monitor): extend command line to display (autowarefoundation#4553)

Signed-off-by: ito-san <[email protected]>
Co-authored-by: ito-san <[email protected]>
)

feat(behavior_path_planner): resample output path (#1604)

* feat(behavior_path_planner): resample output path



* update param

Signed-off-by: Takayuki Murooka <[email protected]>
Co-authored-by: Takayuki Murooka <[email protected]>
* ci: add dispatch-push-event workflow

Signed-off-by: Keisuke Shima <[email protected]>

* fix: change APP KEY

Signed-off-by: Keisuke Shima <[email protected]>

* chore: use strategy

Signed-off-by: Keisuke Shima <[email protected]>

* chore: change trigger

Signed-off-by: Keisuke Shima <[email protected]>

* pre-commit fixes

Signed-off-by: Keisuke Shima <[email protected]>

* Update .github/workflows/dispatch-push-event.yaml

* Update .github/workflows/dispatch-push-event.yaml

* style(pre-commit): autofix

* Update .github/workflows/dispatch-push-event.yaml

---------

Signed-off-by: Keisuke Shima <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ot for turn_signal processing (#909)

* fix(behavior_path): only apply spline interpolate for output, not for turn_signal processing

* fix implementation

* ci(pre-commit): autofix

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ds on the spline interpolated path (#929)

* add base points to resampled path in behavior_path

* Revert "fix(behavior_path): only apply spline interpolation for its output, not for turn_signal processing (#909)"

This reverts commit c80c986.

* ci(pre-commit): autofix

* fix insert

* fix: not interpolate behavior velocity path

* Revert "Revert "fix(behavior_path): only apply spline interpolation for its output, not for turn_signal processing (#909)""

This reverts commit e6dd540.

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…arefoundation#5191, autowarefoundation#5430) (#995)

* perf(system_monitor): fix program command line reading (autowarefoundation#5191)

* Fix program command line reading

Signed-off-by: Owen-Liuyuxuan <[email protected]>

* style(pre-commit): autofix

* fix spelling commandline->command_line

Signed-off-by: Owen-Liuyuxuan <[email protected]>

---------

Signed-off-by: Owen-Liuyuxuan <[email protected]>
Co-authored-by: Owen-Liuyuxuan <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

* fix(system_monitor): output command line (autowarefoundation#5430)

* fix(system_monitor): output command line

Signed-off-by: takeshi.iwanari <[email protected]>

* style(pre-commit): autofix

---------

Signed-off-by: takeshi.iwanari <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>

---------

Signed-off-by: Owen-Liuyuxuan <[email protected]>
Signed-off-by: takeshi.iwanari <[email protected]>
Co-authored-by: Yuxuan Liu <[email protected]>
Co-authored-by: Owen-Liuyuxuan <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: takeshi-iwanari <[email protected]>
Co-authored-by: Akihisa Nagata <[email protected]>
* Adapted from PR #1458

Signed-off-by: Shigekazu Fukuta <[email protected]>

* Adapted from PR autowarefoundation#1627

Signed-off-by: Shigekazu Fukuta <[email protected]>

* fix parameter name

Signed-off-by: Shigekazu Fukuta <[email protected]>

* Adapted from PR autowarefoundation#2647

Signed-off-by: Shigekazu Fukuta <[email protected]>

* ci(pre-commit): autofix

* fix build error

* ci(pre-commit): autofix

* remove comment line

* remove logic

* Delete parameters other than those added this time

* ci(pre-commit): autofix

* add stop vehicle check

* ci(pre-commit): autofix

* fix stop velocity threshold

* fix engage obstacle clear and stop threshold

* ci(pre-commit): autofix

---------

Signed-off-by: Shigekazu Fukuta <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* fix(obstacle_avoidance_planner): add empty check

* ci(pre-commit): autofix

* add invalid_argument

* delete empty check

* return code moved to end

* add warning log

* update rclcpp_debug

* delete debug log

* Delete unnecessary blank lines

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
* suppress launch

Signed-off-by: Yuki Takagi <[email protected]>

* add existence check

Signed-off-by: Yuki Takagi <[email protected]>

---------

Signed-off-by: Yuki Takagi <[email protected]>
Co-authored-by: Shigekazu Fukuta <[email protected]>
@sfukuta
Copy link

sfukuta commented Sep 18, 2024

#1068 は、eveから受入NGな機能となっています。
そのため、revertをお願いします。

@sfukuta
Copy link

sfukuta commented Sep 18, 2024

revertでも追加でもないcommitが複数存在します。
既に、適応ずみたのめ、対象外という事を確認のうえ、記載をお願いします。

@sfukuta
Copy link

sfukuta commented Sep 18, 2024

本チケットでは、過去に反映済みのコードのため、設計者テストが、適応漏れチェックのみで、
機能評価は、リリース評価で実施する旨を記載してください。

@asa-naki
Copy link
Author

#1068 は、eveから受入NGな機能となっています。 そのため、revertをお願いします。

22f00a9
でrevertしました。

@asa-naki
Copy link
Author

asa-naki commented Sep 18, 2024

revertでも追加でもないcommitが複数存在します。
既に、適応ずみたのめ、対象外という事を確認のうえ、記載をお願いします。
本チケットでは、過去に反映済みのコードのため、設計者テストが、適応漏れチェックのみで、
機能評価は、リリース評価で実施する旨を記載してください。

追加しました。

@sfukuta
Copy link

sfukuta commented Sep 18, 2024

#1068 descriptionにrevert情報として追加をお願いします。

@asa-naki
Copy link
Author

#1068 descriptionにrevert情報として追加をお願いします。

更新しました。

@sfukuta
Copy link

sfukuta commented Sep 18, 2024

対象のPRをrevertすることで、
path_intervalがパラメータ反映されていない状態であることをPsimで確認済み。

→ 今回のrevertが、4件ありますが、このパラメーターだけ確認したのはなぜでしょうか?
  他にも、確認が必要であれば、追加で実施をお願いします。

@asa-naki
Copy link
Author

対象のPRをrevertすることで、
path_intervalがパラメータ反映されていない状態であることをPsimで確認済み。

→ 今回のrevertが、4件ありますが、このパラメーターだけ確認したのはなぜでしょうか?   他にも、確認が必要であれば、追加で実施をお願いします。

autoware_launchの部分と記載を間違えていました。
パラメータの確認は本PRは不要と判断しています。

Copy link

sonarcloud bot commented Sep 18, 2024

@asa-naki asa-naki merged commit cc1ae5f into beta/v0.3.19 Sep 18, 2024
15 of 16 checks passed
@asa-naki asa-naki deleted the feat/v0.3.19_rebase branch September 18, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants