Skip to content

Add SORT and DeepSORT tracker workflow blocks #1301

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

Conversation

yeldarby
Copy link
Contributor

@yeldarby yeldarby commented May 19, 2025

Summary

NOTE: Do not merge this until the next version of Trackers is released. It's developed against their @main branch & PyPi won't let us link to a GitHub commit in our requirements. (Marked as non-draft to ensure CI passes.) We also need to add support for using other reid models via Inference.

  • add SORT and DeepSORT tracker blocks with UI manifests
  • update requirements for trackers 2.0.1
  • implement base tracker cache fixes
  • add unit tests for trackers
  • pin trackers to 2.0.1 to avoid upcoming API change

Type of change

  • New feature (non-breaking change which adds functionality)

How has this change been tested, please provide a testcase or example of how you tested the change?

  • pytest -q tests/workflows/unit_tests/core_steps/trackers/test_sort_tracker_v1.py tests/workflows/unit_tests/core_steps/trackers/test_deep_sort_tracker_v1.py
  • Github Actions
  • Manually (WIP)

Any specific deployment considerations

WIP

Docs

  • Docs updated? What were the changes: Should be self-documenting from the Manifest

@yeldarby yeldarby mentioned this pull request May 19, 2025
2 tasks
@yeldarby yeldarby marked this pull request as ready for review May 19, 2025 02:17
@yeldarby yeldarby requested a review from capjamesg as a code owner May 19, 2025 02:20
@yeldarby yeldarby changed the base branch from workflows/trackers to main May 19, 2025 02:21
@yeldarby
Copy link
Contributor Author

Closing because we need to wait until we add custom embedding models to the platform. Will reopen when we get there.

@yeldarby yeldarby closed this Jun 17, 2025
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.

1 participant