-
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
[Fix] Fix magnitude_range in RandAug #249
Conversation
Codecov Report
@@ Coverage Diff @@
## master #249 +/- ##
==========================================
+ Coverage 72.82% 72.85% +0.02%
==========================================
Files 88 88
Lines 4453 4457 +4
Branches 782 784 +2
==========================================
+ Hits 3243 3247 +4
Misses 1109 1109
Partials 101 101
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
This reverts commit 128af36.
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
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.
Good
* add increasing in solarize and posterize * fix linting * Revert "add increasing in solarize and posterize" This reverts commit 128af36. * revise according to comments
Hi,
In this pull request, we allow users to set magnitude range as
(max_val, min_val)
or(min_val, max_val)
. Pls kindly take a look. Thank you.