Skip to content
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

Vectorize IOU calcs for axis-aligned bboxes #731

Merged
merged 2 commits into from
Aug 30, 2024
Merged

Vectorize IOU calcs for axis-aligned bboxes #731

merged 2 commits into from
Aug 30, 2024

Conversation

ntlind
Copy link
Contributor

@ntlind ntlind commented Aug 30, 2024

Improvements

  • Refactor IOU calculations to use vector operations, bringing axis-aligned bbox benchmark time below 10 seconds
  • Remove pandas[performance] since we're no longer using numba's JIT decorator. benchmark times didn't change after removal

@ntlind ntlind requested review from czaloom and ekorman as code owners August 30, 2024 17:37
@ntlind ntlind self-assigned this Aug 30, 2024
@ntlind ntlind merged commit 04d7e76 into main Aug 30, 2024
14 checks passed
@ntlind ntlind deleted the refactor_iou_calc branch August 30, 2024 18:05
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.

2 participants