Repository navigation
feat(mrm_operator): add driving mode active flag - #13249
sasakisasaki merged 5 commits into
Conversation
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
|
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
sasakisasaki
left a comment
There was a problem hiding this comment.
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?
Codecov Report❌ Patch coverage is
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
*This pull request uses carry forward flags. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Signed-off-by: Takagi, Isamu <isamu.takagi@tier4.jp>
…takagi/autoware_universe into feat/add-driving-mode-active-flag
sasakisasaki
left a comment
There was a problem hiding this comment.
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.
…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>
…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>
…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>
…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>
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.