Skip to content

Conversation

@sbarkby
Copy link

@sbarkby sbarkby commented Apr 23, 2024

This PR introduces the Intersection Over Union complement (1.0 - IOU) as an alternative distance function for determining matches and the calculation of metrics, such as MOTP. The use of this distance function has been made configurable in the detection_cvpr_2019.json and tracking_nips_2019.json config files for detection and tracking evaluation respectively. Fortunately the codebase was already setup to cater for alternative distance functions, so the coding change proposed here is minimal.

Also note that this PR poses no change to the default operation of the evaluator, it will continue to use "center_distance" as it's distance function by default.

Copy link
Collaborator

@whyekit-motional whyekit-motional left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this feature @sbarkby!

I've done an initial pass on PR (and will look deeper into your code in subsequent passes)

@sbarkby
Copy link
Author

sbarkby commented Apr 24, 2024

Hi @whyekit-motional , thanks for the review, have made the changes you suggested, let me know if you need anything else.

Cheers. :)

@sbarkby
Copy link
Author

sbarkby commented May 8, 2024

Hi @whyekit-motional, just checking in, any updates?

@sbarkby
Copy link
Author

sbarkby commented Jul 9, 2024

Hi @whyekit-motional, just checking in, any updates? Does this need to be transferred to a different reviewer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants