-
Notifications
You must be signed in to change notification settings - Fork 9.4k
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
Implement Cutout #3521
Implement Cutout #3521
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3521 +/- ##
==========================================
+ Coverage 58.88% 59.04% +0.15%
==========================================
Files 205 205
Lines 13696 13731 +35
Branches 2310 2315 +5
==========================================
+ Hits 8065 8107 +42
+ Misses 5254 5248 -6
+ Partials 377 376 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
More unit tests can be added to increase the coverage. |
Generally LGTM. |
Hi @hellock , the coverage has been improved. It seems like every statement (except for |
Is there anything wrong with the latest CI? It failed to install CUDA. |
Please note that this is slightly different from the current version in mmdet-projects.
To be more specific,
n_holes
can be given as a tuple to specify the range andcutout_shape
can be given as float value in order to be compatible with STAC.