-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[Feature] Repeated Aug and Sampler Registry #588
Conversation
Can we use a dataset wrapper to implement this function? |
@mzr1996 Probably not. Repeated Augmentation requires the repeated samples co-exist in the same batch while the |
Codecov Report
@@ Coverage Diff @@
## master #588 +/- ##
==========================================
+ Coverage 80.11% 81.33% +1.22%
==========================================
Files 113 116 +3
Lines 6526 6720 +194
Branches 1120 1156 +36
==========================================
+ Hits 5228 5466 +238
+ Misses 1153 1103 -50
- Partials 145 151 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Co-authored-by: Ma Zerun <mzr1996@163.com>
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
RepAug is a necessary part to re-implement 'ResNet Strikes Back'
Modification
Use cases
Just add
sampler = dict(type="RepeatAugSampler")
to the config file