Skip to content

fix(operation_mode_transition_manager): modify the engage condition#2271

Merged
rej55 merged 2 commits intobeta/v0.48from
zhi/fix/engage-condition-check-bypass
Aug 5, 2025
Merged

fix(operation_mode_transition_manager): modify the engage condition#2271
rej55 merged 2 commits intobeta/v0.48from
zhi/fix/engage-condition-check-bypass

Conversation

@zhiwango
Copy link

@zhiwango zhiwango commented Aug 4, 2025

This PR will cherry-pick autowarefoundation#11097 that fixes a problem.
When check_engage_condition_ is set as false, bool isModeChangeAvailable() will return true before enable_engage_on_driving_ is checked.

Here is the ticket
For more details, please refer to this thread.

…utowarefoundation#11097)

* check engage on driving before check engage condition

Signed-off-by: Zhi Wang <zhi.wang@tier4.jp>

* fix

Signed-off-by: Zhi Wang <zhi.wang@tier4.jp>

---------

Signed-off-by: Zhi Wang <zhi.wang@tier4.jp>
@github-actions
Copy link

github-actions bot commented Aug 4, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@zhiwango zhiwango added the run:build-and-test-differential After beta/v0.41 label Aug 4, 2025
@zhiwango zhiwango requested a review from rej55 August 5, 2025 04:10
@rej55 rej55 merged commit f59fbd8 into beta/v0.48 Aug 5, 2025
25 of 27 checks passed
@rej55 rej55 deleted the zhi/fix/engage-condition-check-bypass branch August 5, 2025 06:26
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.

2 participants