Skip to content

DeepSortTracker tracker_ids updated only with IOU. sohould be with combined distance ? #69

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

Open
fenaux opened this issue May 28, 2025 · 1 comment

Comments

@fenaux
Copy link

fenaux commented May 28, 2025

thanks for this repo

In
trackers/core/deepsort/tracker.py
_get_associated_indices
the association between detections and trackers is made using combined_dist (combined distance from IOU and appearance distance (line 160 and followings)

but to update the detections (line 280) function "update_detections_with_track_ids is used"
and this function (line 112) only use IOU to match indices

It seems that to be consistent the same combined distance should be used

@soumik12345
Copy link
Collaborator

Thanks for pointing this out.
We have an open PR to fix this #17

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants