-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Optimize quality computation #8990
Conversation
1c73934
to
657fb32
Compare
657fb32
to
67a6df3
Compare
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #8990 +/- ##
===========================================
- Coverage 73.83% 73.40% -0.44%
===========================================
Files 417 415 -2
Lines 44602 44193 -409
Branches 4031 3859 -172
===========================================
- Hits 32934 32439 -495
- Misses 11668 11754 +86
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Motivation and context
Most noticeable in tasks with large images (e.g. 2k x 4k pixels) and with ellipse annotations. The result is that for a sample task with 157 GT frames and with lots of small ellipse annotations, peak memory consumption falls from 20GB to under 2GB.
ellipse()
is quite slow with big masks)How has this been tested?
Checklist
develop
branchLicense
Feel free to contact the maintainers if that's a concern.