Skip to content

feat(mrm_operator): add driving mode active flag - #13249

Merged
sasakisasaki merged 5 commits into
autowarefoundation:mainfrom
isamu-takagi:feat/add-driving-mode-active-flag
Aug 26, 2026
Merged

sasakisasaki merged 5 commits into
autowarefoundation:mainfrom
isamu-takagi:feat/add-driving-mode-active-flag

Conversation

@isamu-takagi

@isamu-takagi isamu-takagi commented Aug 21, 2026 •

Copy link
Copy Markdown
Contributor

Description

Add active flag for driving mode interface. This notifies that the output of the target mode is ready for actual use.

Related links

Parent Issue:

How was this PR tested?

The vehicle evaluation was performed with all PRs in #12497.

Notes for reviewers

None.

Interface changes

None.

Effects on system behavior

None.

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
@github-actions github-actions Bot added the component:system System design and integration. (auto-assigned) label Aug 21, 2026
@github-actions

Copy link
Copy Markdown

Thank you for contributing to the Autoware project!

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

Please ensure:

@sasakisasaki sasakisasaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Personally, there looks no regression detection mechanism for now. Could you add tests for the core logic, which can be run-able without ROS-node spin?

@sasakisasaki sasakisasaki added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label Aug 26, 2026
@codecov

codecov Bot commented Aug 26, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 20 lines in your changes missing coverage. Please review.
✅ Project coverage is 20.46%. Comparing base (bed0cb6) to head (d29f4b1).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
...perator/src/mrm_comfortable_stop_operator_core.cpp 0.00% 10 Missing ⚠️
..._operator/src/mrm_emergency_stop_operator_core.cpp 0.00% 10 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main   #13249   +/-   ##
=======================================
  Coverage   20.46%   20.46%           
=======================================
  Files        1949     1949           
  Lines      137092   137077   -15     
  Branches    48913    48909    -4     
=======================================
- Hits        28051    28050    -1     
+ Misses      86318    86305   -13     
+ Partials    22723    22722    -1     
Flag Coverage Δ *Carryforward flag
full-suite 20.46% <ø> (+<0.01%) ⬆️ Carriedforward from ad86f20

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

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
…takagi/autoware_universe into feat/add-driving-mode-active-flag

@sasakisasaki sasakisasaki left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Perhaps we can add tests for core logic. At this stage as the core logic is very small, I agree with merging this PR with the current files changed.

@sasakisasaki
sasakisasaki enabled auto-merge (squash) August 26, 2026 12:11
@sasakisasaki
sasakisasaki merged commit 241caaf into autowarefoundation:main Aug 26, 2026
30 checks passed
@github-project-automation github-project-automation Bot moved this from To Triage to Done in Software Working Group Aug 26, 2026
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 7, 2026
…13249)

* feat(mrm_operator): add driving mode active flag

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* remove unnecessary conditions

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

---------

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
Co-authored-by: Junya Sasaki <junya.sasaki@tier4.jp>
Signed-off-by: TetsuKawa <kawaguchitnon@icloud.com>
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 7, 2026
…13249)

* feat(mrm_operator): add driving mode active flag

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* remove unnecessary conditions

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

---------

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
Co-authored-by: Junya Sasaki <junya.sasaki@tier4.jp>
TetsuKawa pushed a commit to TetsuKawa/autoware_universe that referenced this pull request Sep 15, 2026
…13249)

* feat(mrm_operator): add driving mode active flag

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* remove unnecessary conditions

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

---------

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
Co-authored-by: Junya Sasaki <junya.sasaki@tier4.jp>
shmpwk pushed a commit to shmpwk/pilot_auto_system-mirror that referenced this pull request Sep 25, 2026
…autoware_universe#13249)

* feat(mrm_operator): add driving mode active flag

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

* remove unnecessary conditions

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>

---------

Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
Co-authored-by: Junya Sasaki <junya.sasaki@tier4.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:system System design and integration. (auto-assigned) run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci)

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants