Skip to content

feat: v0.3.19 rebase - #1540

Merged
asa-naki merged 12 commits into
beta/v0.3.19from
feat/v0.3.19_rebase
Sep 18, 2024
Merged

asa-naki merged 12 commits into
beta/v0.3.19from
feat/v0.3.19_rebase

Conversation

@asa-naki

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

Copy link
Copy Markdown

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 <fumihito.ito@tier4.jp>
Co-authored-by: ito-san <57388357+ito-san@users.noreply.github.com>
)

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

* feat(behavior_path_planner): resample output path



* update param

Signed-off-by: Takayuki Murooka <takayuki5168@gmail.com>
Co-authored-by: Takayuki Murooka <takayuki5168@gmail.com>
* ci: add dispatch-push-event workflow

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

* fix: change APP KEY

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

* chore: use strategy

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

* chore: change trigger

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

* pre-commit fixes

Signed-off-by: Keisuke Shima <19993104+KeisukeShima@users.noreply.github.com>

* 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 <19993104+KeisukeShima@users.noreply.github.com>
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 <uken.ryu@tier4.jp>

* style(pre-commit): autofix

* fix spelling commandline->command_line

Signed-off-by: Owen-Liuyuxuan <uken.ryu@tier4.jp>

---------

Signed-off-by: Owen-Liuyuxuan <uken.ryu@tier4.jp>
Co-authored-by: Owen-Liuyuxuan <uken.ryu@tier4.jp>
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 <takeshi.iwanari@tier4.jp>

* style(pre-commit): autofix

---------

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

---------

Signed-off-by: Owen-Liuyuxuan <uken.ryu@tier4.jp>
Signed-off-by: takeshi.iwanari <takeshi.iwanari@tier4.jp>
Co-authored-by: Yuxuan Liu <619684051@qq.com>
Co-authored-by: Owen-Liuyuxuan <uken.ryu@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Co-authored-by: takeshi-iwanari <takeshi.iwanari@tier4.jp>
Co-authored-by: Akihisa Nagata <54956813+asa-naki@users.noreply.github.com>
* Adapted from PR #1458

Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>

* Adapted from PR #1627

Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>

* fix parameter name

Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>

* Adapted from PR #2647

Signed-off-by: Shigekazu Fukuta <shigekazu.fukuta@tier4.jp>

* 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 <shigekazu.fukuta@tier4.jp>
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 <yuki.takagi@tier4.jp>

* add existence check

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>

---------

Signed-off-by: Yuki Takagi <yuki.takagi@tier4.jp>
Co-authored-by: Shigekazu Fukuta <107168699+sfukuta@users.noreply.github.com>
@asa-naki
asa-naki requested a review from sfukuta September 18, 2024 04:00
@sfukuta

sfukuta commented Sep 18, 2024

Copy link
Copy Markdown

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

@sfukuta

sfukuta commented Sep 18, 2024

Copy link
Copy Markdown

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

@sfukuta

sfukuta commented Sep 18, 2024

Copy link
Copy Markdown

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

@asa-naki

Copy link
Copy Markdown
Author

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

22f00a9
でrevertしました。

@asa-naki

asa-naki commented Sep 18, 2024 •

Copy link
Copy Markdown
Author

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

追加しました。

@sfukuta

sfukuta commented Sep 18, 2024

Copy link
Copy Markdown

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

@asa-naki

Copy link
Copy Markdown
Author

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

更新しました。

@sfukuta

sfukuta commented Sep 18, 2024 •

Copy link
Copy Markdown

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

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

@asa-naki

Copy link
Copy Markdown
Author

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

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

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

@sonarqubecloud

Copy link
Copy Markdown

@asa-naki
asa-naki merged commit cc1ae5f into beta/v0.3.19 Sep 18, 2024
@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