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(aeb): set global param to override autoware state check (#1218) #668

Conversation

danielsanchezaran
Copy link

@danielsanchezaran danielsanchezaran commented Nov 15, 2024

Cherry-pick of autowarefoundation#1218

Mainly used by DLR V2 for planning_control evaluation
Note, requires universe changes: https://github.com/tier4/autoware.universe/compare/beta/v0.36...tier4:autoware.universe:cherry-pick/add-global-parm-to-disable-aeb-state-check?expand=1

These changes do not affect ego behavior.

  • set global param to override autoware state check

  • change variable for a more generic name

  • set var to false by default

  • move param to control component launch

  • change param name to be more straightforward


Note: Confirm the contribution guidelines before submitting a pull request.

Click the Preview tab and select a PR template:

Do NOT send a PR with this description.

…efoundation#1218)

* set global param to override autoware state check

Signed-off-by: Daniel Sanchez <[email protected]>

* change variable for a more generic name

Signed-off-by: Daniel Sanchez <[email protected]>

* set var to false by default

Signed-off-by: Daniel Sanchez <[email protected]>

* move param to control component launch

Signed-off-by: Daniel Sanchez <[email protected]>

* change param name to be more straightforward

Signed-off-by: Daniel Sanchez <[email protected]>

---------

Signed-off-by: Daniel Sanchez <[email protected]>
@danielsanchezaran danielsanchezaran merged commit 8256561 into beta/v0.36 Nov 15, 2024
14 checks passed
@danielsanchezaran danielsanchezaran deleted the cherry-pick/add-global-param-to-disable-aeb-state-check branch November 15, 2024 03:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants