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.

Sign up for GitHub

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

Jump to bottom

feat(tier4_perception_launch): add radar far tracking merger launcher #5251

Closed
wants to merge 40 commits into from
Closed

feat(tier4_perception_launch): add radar far tracking merger launcher #5251

wants to merge 40 commits into from

Conversation

YoshiRi
Copy link
Contributor

@YoshiRi YoshiRi commented Oct 7, 2023

Description

Related links

Tests performed

Notes for reviewers

Interface changes

Effects on system behavior

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.

  • The PR follows the pull request guidelines.
  • The PR has been properly tested.
  • The PR has been reviewed by the code owners.

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.
  • The PR is ready for merge.

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

@github-actions github-actions bot added type:documentation Creating or refining documentation. (auto-assigned) component:perception Advanced sensor data processing and environment understanding. (auto-assigned) component:launch Launch files, scripts and initialization tools. (auto-assigned) labels Oct 7, 2023
YoshiRi added 27 commits October 9, 2023 11:46
@YoshiRi
init package: migrate from object merger
10f93fd 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add node outline and check build passed
868ad2b 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add util functions to interpolate tracked objs
3f205fc 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add object merger function using interpolation
a70e4a6 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
create object merger utils
2dd4027 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add kinematics velocity merger
789d8af 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add association and merger
4b714d4 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
rename perception_utils to object_recognition_utils
81bce34 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add comment and complete main subscriber
c492c83 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add parameter control and rename some executable files
b6dcc86 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
refactoring: fix apparent bugs
f62b7b4 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add debug tools to check association
ea60dd2 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
temporary fix: radar tracker node name to anon
c798307 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
debug: data association tuning
a742d04 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
rename copyright and add merger util function
f415f3b 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add tracker_state and update association
98de2d1 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
update decorative tracker by using tracker_state
fe9f0c6 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
update system around measurement state function
fbf00b5 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
fix radar object not merged problem
0f5f16e 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add existence probability control
c9c8a6a 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
create const function
0c6ce7b 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
change association settings depend on measurement and tracker state
bc8459e 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
fix association matrix
6fe8808 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
put hardcoded node parameter to yaml file
9a885a9 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
move tracker state parameter to yaml file
c8f405b 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
remove prediction failed objects
546944f 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
fix bug when none closest time sub objects found
db804f2 
Signed-off-by: yoshiri <[email protected]>
YoshiRi added 12 commits October 9, 2023 11:46
@YoshiRi
add velocity diff gate in association
15198b9 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
fix object interpolation problem
d49f7ab 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
use fixed object interpolation
1c48528 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add README
29d5a8d 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add interpolated sub object publisher for debug
927fc18 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add debug message and fix interpolation
d544b88 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
update README
db6a6fe 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
fix unintended changes in radar tracking launch
fd4d2f6 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
refactor: put node parameter to config yaml file
a7c21ab 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
update tracking merger launch
a03079a 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
update perception launch to use tracking merger with boolean flag
5935630 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi
add tracking merger to processing flow
b28a0c0 
Signed-off-by: yoshiri <[email protected]>
@YoshiRi YoshiRi force-pushed the feat/add_radar_far_tracking_merger_launcher branch from 000c857 to b28a0c0 Compare October 9, 2023 02:54
@YoshiRi YoshiRi closed this Oct 13, 2023
@YoshiRi
Copy link
Contributor Author

YoshiRi commented Oct 13, 2023

Closed because discussed in #5269

@YoshiRi YoshiRi deleted the feat/add_radar_far_tracking_merger_launcher branch October 16, 2023 10:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@miursh miursh Awaiting requested review from miursh miursh will be requested when the pull request is marked ready for review miursh is a code owner

@technolojin technolojin Awaiting requested review from technolojin technolojin will be requested when the pull request is marked ready for review technolojin is a code owner

@yukkysaito yukkysaito Awaiting requested review from yukkysaito yukkysaito will be requested when the pull request is marked ready for review yukkysaito is a code owner

Assignees
No one assigned
Labels
component:launch Launch files, scripts and initialization tools. (auto-assigned) component:perception Advanced sensor data processing and environment understanding. (auto-assigned) type:documentation Creating or refining documentation. (auto-assigned)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@YoshiRi