-
Notifications
You must be signed in to change notification settings - Fork 675
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(perception_online_evaluator): add perception_online_evaluator #6493
feat(perception_online_evaluator): add perception_online_evaluator #6493
Conversation
b9bfb22
to
7d14c4f
Compare
734dc8d
to
68da326
Compare
db58f5d
to
7205898
Compare
bdf286a
to
a756799
Compare
rename to checked it worked after rename |
a742d56
to
e70dd88
Compare
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: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
1762aa1
to
df0b9ef
Compare
Signed-off-by: kosuke55 <[email protected]>
Signed-off-by: kosuke55 <[email protected]>
I've canceled the This PR halves the time required for build-and-test-differential checks: If the test passes you can merge it. No need to do anything. If you make any further changes to this PR, please rebase to main. |
@xmfcx wow! Thank you very much!! I've been waiting for ci to finish for a long time...! |
@takayuki5168 @TakaHoribe @tkimura4 (simulaor launch) |
@kosuke55 I'm sorry that this job is taking forever for so many people. I am trying to speed it up as much as I can. |
…6493) * feat(perception_evaluator): add perception_evaluator Signed-off-by: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]> * fix build werror Signed-off-by: kosuke55 <[email protected]> * fix topic name Signed-off-by: kosuke55 <[email protected]> * clean up Signed-off-by: kosuke55 <[email protected]> * rename to perception_online_evaluator Signed-off-by: kosuke55 <[email protected]> * refactor: remove timer Signed-off-by: kosuke55 <[email protected]> * feat: add test Signed-off-by: kosuke55 <[email protected]> * fix: ci check Signed-off-by: kosuke55 <[email protected]> --------- Signed-off-by: kosuke55 <[email protected]> Signed-off-by: Kotaro Yoshimoto <[email protected]>
…utowarefoundation#6493) * feat(perception_evaluator): add perception_evaluator Signed-off-by: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]> * fix build werror Signed-off-by: kosuke55 <[email protected]> * fix topic name Signed-off-by: kosuke55 <[email protected]> * clean up Signed-off-by: kosuke55 <[email protected]> * rename to perception_online_evaluator Signed-off-by: kosuke55 <[email protected]> * refactor: remove timer Signed-off-by: kosuke55 <[email protected]> * feat: add test Signed-off-by: kosuke55 <[email protected]> * fix: ci check Signed-off-by: kosuke55 <[email protected]> --------- Signed-off-by: kosuke55 <[email protected]>
…utowarefoundation#6493) * feat(perception_evaluator): add perception_evaluator Signed-off-by: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]> * fix build werror Signed-off-by: kosuke55 <[email protected]> * fix topic name Signed-off-by: kosuke55 <[email protected]> * clean up Signed-off-by: kosuke55 <[email protected]> * rename to perception_online_evaluator Signed-off-by: kosuke55 <[email protected]> * refactor: remove timer Signed-off-by: kosuke55 <[email protected]> * feat: add test Signed-off-by: kosuke55 <[email protected]> * fix: ci check Signed-off-by: kosuke55 <[email protected]> --------- Signed-off-by: kosuke55 <[email protected]>
…utowarefoundation#6493) * feat(perception_evaluator): add perception_evaluator Signed-off-by: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]> * fix build werror Signed-off-by: kosuke55 <[email protected]> * fix topic name Signed-off-by: kosuke55 <[email protected]> * clean up Signed-off-by: kosuke55 <[email protected]> * rename to perception_online_evaluator Signed-off-by: kosuke55 <[email protected]> * refactor: remove timer Signed-off-by: kosuke55 <[email protected]> * feat: add test Signed-off-by: kosuke55 <[email protected]> * fix: ci check Signed-off-by: kosuke55 <[email protected]> --------- Signed-off-by: kosuke55 <[email protected]>
…utowarefoundation#6493) * feat(perception_evaluator): add perception_evaluator Signed-off-by: kosuke55 <[email protected]> tmp Signed-off-by: kosuke55 <[email protected]> update Signed-off-by: kosuke55 <[email protected]> add Signed-off-by: kosuke55 <[email protected]> add add update clean up Signed-off-by: kosuke55 <[email protected]> change time horizon Signed-off-by: kosuke55 <[email protected]> * fix build werror Signed-off-by: kosuke55 <[email protected]> * fix topic name Signed-off-by: kosuke55 <[email protected]> * clean up Signed-off-by: kosuke55 <[email protected]> * rename to perception_online_evaluator Signed-off-by: kosuke55 <[email protected]> * refactor: remove timer Signed-off-by: kosuke55 <[email protected]> * feat: add test Signed-off-by: kosuke55 <[email protected]> * fix: ci check Signed-off-by: kosuke55 <[email protected]> --------- Signed-off-by: kosuke55 <[email protected]>
Description
Add perception_evaluator
This module allows for the evaluation of how accurately perception results are generated without the need for annotations. It is capable of confirming performance and can evaluate results from a few seconds prior, enabling online execution.
current metrics are
blue arrow are old predicted path and red arrows are actual history path the objects followed.
smoothing path (window size=11)
rqt reconfigure
Tests performed
psim (max of time_horizonx=5)
perception_evaluator-2024-02-27_00.25.32.mp4
psim with rosbag(perception replyer) (max of time_horizonx=10)
perception_evaluator-2024-02-27_00.19.00.mp4
logging simulator
evaluator_description: feat/perception_evaluator
2024/02/27 https://evaluation.tier4.jp/evaluation/reports/1542d256-59b7-54c2-8d54-fc960508f9eb/?project_id=prd_jt
evaluator_description: feat/perception_evaluator time horizon 5
2024/02/27 https://evaluation.tier4.jp/evaluation/reports/ba008b2b-0242-52db-b18e-62781f6b554e/?project_id=prd_jt
evaluator_description: feat/perception_evaluator
2024/02/28 https://evaluation.tier4.jp/evaluation/reports/bbaa9fdc-cd30-56ef-b903-8e2908b1f674/?project_id=prd_jt
Effects on system behavior
Not applicable.
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.