-
Notifications
You must be signed in to change notification settings - Fork 683
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(mission_planner): add mrm route planner #4027
Conversation
Signed-off-by: yutaka <purewater0901@gmail.com>
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## main #4027 +/- ##
==========================================
+ Coverage 14.31% 15.42% +1.10%
==========================================
Files 1557 1355 -202
Lines 106900 96387 -10513
Branches 30777 29857 -920
==========================================
- Hits 15302 14863 -439
+ Misses 75028 65297 -9731
+ Partials 16570 16227 -343
*This pull request uses carry forward flags. Click here to find out more.
☔ View full report in Codecov by Sentry. |
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
planning/mission_planner/src/mission_planner/mission_planner.cpp
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: GitHub <noreply@github.com> Co-authored-by: mitsudome-r <mitsudome-r@users.noreply.github.com> Co-authored-by: Ryohsuke Mitsudome <43976834+mitsudome-r@users.noreply.github.com>
Description
Add MRM reroute implementation in the mission planner. If there is an MRM manager, and it requires rerouting in an appropriate situation, the mission planner creates a new route for the MRM manager. It also sends the MRM route to the behavior path planner. Note that if the reroute success, it only published mrm route via mrm_route_pub_.
Tests performed
Not applicable.
Effects on system behavior
If it's under MRM state, it can create an MRM route.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.
In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.
Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.
After all checkboxes are checked, anyone who has write access can merge the PR.