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(control_evaluator): cherry pick/control evaluator to v4.0.0 #1644

Merged

Conversation

danielsanchezaran
Copy link

Description

Cherry-picks:

9a64647 (HEAD -> cherry-pick/control-evaluator-to-v4.0.0, origin/cherry-pick/control-evaluator-to-v4.0.0) fix metrics topic name
1a3caa5 fix planning evaluator
ed0787f feat(planning_evaluator): add lanelet info to the planning evaluator (autowarefoundation#7781)
5eac890 fix(autoware_planning_evaluator): fix unreadVariable (autowarefoundation#8352)
3d83ad1 feat(control_evaluator): add goal accuracy longitudinal, lateral, yaw (autowarefoundation#9155)
4ac4f03 test(autoware_control_evaluator): add test for autoware_control_evaluator. (autowarefoundation#9114)
ef321d1 docs(control_evaluator): update readme (autowarefoundation#8829)
92ca5f9 feat(evalautor): rename evaluator diag topics (autowarefoundation#8152)
2111e76 feat(planning_evaluator,control_evaluator, evaluator utils): add diagnostics subscriber to planning eval (autowarefoundation#7849)
a8b0d47 refactor(control_evaluator): use class naming standard and use remapped param name (autowarefoundation#7782)
eabd0f2 feat(control_evaluator): add lanelet info to the metrics (autowarefoundation#7765)

Related links

Parent Issue:

  • Link

How was this PR tested?

Notes for reviewers

None.

Interface changes

Effects on system behavior

None.

danielsanchezaran and others added 11 commits November 14, 2024 13:04
…ndation#7765)

* add route handler

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

* add lanelet info to diagnostic

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

* add const

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

* add kinematic state info

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

* clean

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

* remove unusde subscriptions

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

* clean

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

* add shoulder lanelets

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

* fix includes

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

---------

Signed-off-by: Daniel Sanchez <[email protected]>
…ed param name (autowarefoundation#7782)

use class naming standard and use remapped param name

Signed-off-by: Daniel Sanchez <[email protected]>
…nostics subscriber to planning eval (autowarefoundation#7849)

* add utils and diagnostics subscription to planning_evaluator

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

* add diagnostics eval

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

* fix input diag in launch

Signed-off-by: kosuke55 <[email protected]>

---------

Signed-off-by: Daniel Sanchez <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
Co-authored-by: kosuke55 <[email protected]>
* feat(evalautor): rename evaluator diag topics

Signed-off-by: kosuke55 <[email protected]>

* perception

Signed-off-by: kosuke55 <[email protected]>

---------

Signed-off-by: kosuke55 <[email protected]>
* update readme

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

* add maintainer

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

* Update evaluator/autoware_control_evaluator/package.xml

Add extra maintainer

Co-authored-by: Tiankui Xian <[email protected]>

---------

Signed-off-by: Daniel Sanchez <[email protected]>
Co-authored-by: Tiankui Xian <[email protected]>
…ator. (autowarefoundation#9114)

* init

Signed-off-by: xtk8532704 <[email protected]>

* tmp save.

Signed-off-by: xtk8532704 <[email protected]>

* save, there is a bug

Signed-off-by: xtk8532704 <[email protected]>

* update package.xml

Signed-off-by: xtk8532704 <[email protected]>

* coverage rate 64.5

Signed-off-by: xtk8532704 <[email protected]>

* remove comments.

Signed-off-by: xtk8532704 <[email protected]>

---------

Signed-off-by: xtk8532704 <[email protected]>
…autowarefoundation#9155)

* feat(control_evaluator): add goal accuracy longitudinal, lateral, yaw

Signed-off-by: Kasunori-Nakajima <[email protected]>

* style(pre-commit): autofix

* fix: content of kosuke55-san comments

Signed-off-by: Kasunori-Nakajima <[email protected]>

* fix: variable name

Signed-off-by: Kasunori-Nakajima <[email protected]>

* fix: variable name

Signed-off-by: Kasunori-Nakajima <[email protected]>

---------

Signed-off-by: Kasunori-Nakajima <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
…ion#8352)

* fix:unreadVariable

Signed-off-by: kobayu858 <[email protected]>

* fix:unreadVariable

Signed-off-by: kobayu858 <[email protected]>

---------

Signed-off-by: kobayu858 <[email protected]>
…utowarefoundation#7781)

add lanelet info to the planning evaluator

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

@TomohitoAndo TomohitoAndo left a comment

Choose a reason for hiding this comment

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

LGTM

@TomohitoAndo TomohitoAndo merged commit a88fbde into beta/x2_gen2/v0.29.0 Dec 13, 2024
22 of 24 checks passed
@TomohitoAndo TomohitoAndo deleted the cherry-pick/control-evaluator-to-v4.0.0 branch December 13, 2024 03:46
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.

6 participants