Skip to content

fix(multi object tracker): tracker type to associate (#11132)#2290

Merged
TaikiYamada4 merged 1 commit intotier4:beta/v0.48from
technolojin:cherry-pick/multi-object-tracker/tracker-type-to-associate
Aug 12, 2025
Merged

fix(multi object tracker): tracker type to associate (#11132)#2290
TaikiYamada4 merged 1 commit intotier4:beta/v0.48from
technolojin:cherry-pick/multi-object-tracker/tracker-type-to-associate

Conversation

@technolojin
Copy link

cherry-pick of autowarefoundation#11132

…ion#11132)

* feat(multi_object_tracker): add tracker priority for object classification

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): introduce TrackerType enum and update tracker implementations

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* refactor(multi_object_tracker): remove confident_count_threshold parameter and related logic

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): update TrackerType enum and refactor tracker map usage

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): add can_assign_map for tracker type assignments and update related logic

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* refactor(tracker_processor): improve tracker sorting logic and remove debug output

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): make tracker_type private

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* refactor(tracker_processor): remove debug output from mergeOverlappedTracker function

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): refactor can_assign_map initialization and update tracker_map usage

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* style(pre-commit): autofix

* feat(trackers): initialize tracker_type in constructors for all tracker classes

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* refactor(processor): replace std::map with std::unordered_map for tracker_map and thresholds
refactor(multi_object_tracker): update error message for invalid association matrix size

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* feat(multi_object_tracker): refactor tracker type retrieval using unordered_map for improved efficiency

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

* style(pre-commit): autofix

* refactor(processor): rename channel_priority to tracker_priority for clarity in mergeOverlappedTracker

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>

---------

Signed-off-by: Taekjin LEE <taekjin.lee@tier4.jp>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@technolojin technolojin self-assigned this Aug 8, 2025
@github-actions
Copy link

github-actions bot commented Aug 8, 2025

Thank you for contributing to the Autoware project!

🚧 If your pull request is in progress, switch it to draft mode.

Please ensure:

@technolojin technolojin marked this pull request as ready for review August 8, 2025 08:30
@codecov-commenter
Copy link

codecov-commenter commented Aug 8, 2025

Codecov Report

❌ Patch coverage is 45.20548% with 40 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (beta/v0.48@cb5e1f8). Learn more about missing BASE report.

Files with missing lines Patch % Lines
...i_object_tracker/src/multi_object_tracker_node.cpp 0.00% 27 Missing ⚠️
...e_multi_object_tracker/src/processor/processor.cpp 66.66% 3 Missing and 2 partials ⚠️
...lti_object_tracker/lib/association/association.cpp 60.00% 3 Missing and 1 partial ⚠️
...ject_tracker/lib/tracker/model/vehicle_tracker.cpp 66.66% 2 Missing and 1 partial ⚠️
...tracker/lib/tracker/model/pass_through_tracker.cpp 0.00% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##             beta/v0.48    #2290   +/-   ##
=============================================
  Coverage              ?   38.10%           
=============================================
  Files                 ?       51           
  Lines                 ?     3800           
  Branches              ?     1198           
=============================================
  Hits                  ?     1448           
  Misses                ?     2241           
  Partials              ?      111           
Flag Coverage Δ
differential 38.10% <45.20%> (?)
differential-cuda 38.10% <45.20%> (?)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@TaikiYamada4 TaikiYamada4 merged commit 0bf6bb5 into tier4:beta/v0.48 Aug 12, 2025
38 of 40 checks passed
@technolojin technolojin deleted the cherry-pick/multi-object-tracker/tracker-type-to-associate branch August 13, 2025 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants