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(mrm_emergency_stop_operator): add support for real time param reconfigure for mrm_emergency_stop #6994

Conversation

danielsanchezaran
Copy link
Contributor

@danielsanchezaran danielsanchezaran commented May 13, 2024

Description

Add support for updating mrm emergency stop with rqt_reconfigure. Also added the vector include requested by pre-commit.

Tests performed

PSim

Using the AEB module to test the changes, I lowered the min jerk and acceleration and the ego cannot stop reasonably fast (as expected), see video:

cap-.2024-05-13-17-26-16.mp4

Not applicable.

Effects on system behavior

Not applicable.

Interface changes

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.

  • There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@github-actions github-actions bot added the component:system System design and integration. (auto-assigned) label May 13, 2024
@danielsanchezaran danielsanchezaran marked this pull request as ready for review May 13, 2024 08:29
Copy link
Contributor

@mkuri mkuri left a comment

Choose a reason for hiding this comment

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

LGTM

@danielsanchezaran danielsanchezaran added the run:build-and-test-differential Mark to enable build-and-test-differential workflow. (used-by-ci) label May 13, 2024
@danielsanchezaran danielsanchezaran enabled auto-merge (squash) May 13, 2024 10:47
@danielsanchezaran danielsanchezaran merged commit 2f2438e into autowarefoundation:main May 13, 2024
33 of 35 checks passed
@danielsanchezaran danielsanchezaran deleted the feat/param-update-support-mrm-emergency-stop branch May 13, 2024 10:58
vividf pushed a commit to vividf/autoware.universe that referenced this pull request May 16, 2024
…configure for mrm_emergency_stop (autowarefoundation#6994)

add support for real time param reconfigure for mrm_emergency_stop

Signed-off-by: Daniel Sanchez <[email protected]>
Signed-off-by: vividf <[email protected]>
danielsanchezaran added a commit to tier4/autoware.universe that referenced this pull request May 30, 2024
…configure for mrm_emergency_stop (autowarefoundation#6994)

add support for real time param reconfigure for mrm_emergency_stop

Signed-off-by: Daniel Sanchez <[email protected]>
karishma1911 pushed a commit to Interplai/autoware.universe that referenced this pull request Jun 3, 2024
…configure for mrm_emergency_stop (autowarefoundation#6994)

add support for real time param reconfigure for mrm_emergency_stop

Signed-off-by: Daniel Sanchez <[email protected]>
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
None yet
Development

Successfully merging this pull request may close these issues.

2 participants