Refactored detection+tracking

[sfmig]

Description

What is this PR?

• Bug fix
• Addition of a new feature
• Other

Why is this PR needed?

• Review of part of the work from Saving mota output #180

What does this PR do?

• Factors out and makes the steps in the detect+track pipeline more modular.

• Removes some code duplications (e.g. when preparing data for sort, formating the detections dict...)

• Reviews typing and docstrings

• Adapts current tests to the changes

• Same MOTA on GT clip as reference -- planned integration tests for next PR.

Work focuses mostly on the detect+track pipeline, as the tracker evaluation will be covered in a separate PR.

References

#180

How has this PR been tested?

Integration run manually on reference clip. Existing tests pass locally.

Does this PR require an update to the documentation?

No.

Checklist:

• The code has been tested locally
• Tests have been added to cover all new functionality
• [ n/a ] The documentation has been updated to reflect any changes
• The code has been formatted with pre-commit

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 36.41975% with 103 lines in your changes missing coverage. Please review.

Project coverage is 47.69%. Comparing base (9b4722e) to head (bef4974).

Files with missing lines	Patch %	Lines
crabs/tracker/utils/io.py	24.24%	50 Missing ⚠️
crabs/tracker/track_video.py	40.90%	39 Missing ⚠️
crabs/tracker/evaluate_tracker.py	47.61%	11 Missing ⚠️
crabs/tracker/utils/tracking.py	66.66%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #241      +/-   ##
==========================================
- Coverage   47.97%   47.69%   -0.29%     
==========================================
  Files          24       24              
  Lines        1534     1562      +28     
==========================================
+ Hits          736      745       +9     
- Misses        798      817      +19     

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