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(behavior_path _planner): divide planner manager modules into dependent slots (#8117) #1882

Open
wants to merge 3 commits into
base: beta/x2_gen2/v0.29.2
Choose a base branch
from

Conversation

k-hazama-esol
Copy link

@k-hazama-esol k-hazama-esol commented Mar 3, 2025

Description

  • PIXで発生したtrajectoryが途切れる問題への予防策として、feat(behavior_path _planner): divide planner manager modules into dependent slots autowarefoundation/autoware.universe#8117 (以下#8117と記載)をcherry-pickする
  • cherry-pick後、cherry-pick範囲外の箇所でビルドエラーが生じたため、追加コミットでコードを修正。
    • 修正箇所
      • max_iteration_numの使用箇所を削除
      • canLaunchNewModule()のreturn処理を変更
    • ビルドエラーが生じた原因に対する考察
      • #8117のPRを取り込むためには、#8117のPR時点のバージョンの過去の変更を取り込む必要があった。
      • beta/x2_gen2/v0.29.2ではその変更が取り込まれていなかったため、修正が必要となった。

Related links

Parent Issue:

cherry-pick PR:

Related PR:

How was this PR tested?

  • Planning Simulator(Local)
    • PSIM上での自動走行および自動走行時にtrajectoryが途切れることによる停止が発生しないことを確認
  • Evaluator

本branchを対象としたpilot.auto.x2環境(cherry-pick/trajectory_intermittent_issue_fix/beta_x2_gen2_v0.29.2)および直近でEvaluatorにかけられた同バージョンのpilot.auto.x2環境(beta/4.0.2)との比較を以下の表に示す。

テスト対象 テスト総数 OK数 NG数
cherry-pick/trajectory_intermittent_issue_fix/beta_x2_gen2_v0.29.2 2615 2120 495
beta/4.0.2(2/19時点) 2615 2143 472

Evaluatorの比較結果ページへのリンクはこちら

上記結果より、大幅なOK/NGの増減が無いためデグレーションなく取り込めていると判断する。

Notes for reviewers

SonarCubeにて以下issueが発生(Quality Gateの判定はOK)
https://sonarcloud.io/project/issues?id=tier4_autoware.universe&pullRequest=1882&issueStatuses=OPEN,CONFIRMED&sinceLeakPeriod=true

またGitHubでのCI/CD実施時に以下エラーが発生
cppcheck-differential
DCO
pre-commit-optional
pre-commit.ci
spell-check-differential

上記issue/エラーについて、指摘箇所を確認する限りではcherry-pickした箇所のみで発生しており、追加コミットによるコード修正で発生した指摘ではない。今回のPRで指摘を修正するとなると、cherry-pickの範囲を越えることになるため、修正しないという方針で進めさせていただきたい。

Interface changes

None.

Effects on system behavior

None.

… canLaunchNewModule() to PR original (autowarefoundation#8117).

Signed-off-by: k-hazama-esol <[email protected]>
@k-hazama-esol k-hazama-esol force-pushed the cherry-pick/trajectory_intermittent_issue_fix/beta_x2_gen2_v0.29.2 branch from b5ada12 to ba241f4 Compare March 3, 2025 06:29
Copy link

sonarqubecloud bot commented Mar 3, 2025

@k-hazama-esol k-hazama-esol marked this pull request as ready for review March 3, 2025 08:19
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.

2 participants