-
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
[Refactor] Remove deprecation #633
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #633 +/- ##
==========================================
+ Coverage 83.15% 85.01% +1.85%
==========================================
Files 126 120 -6
Lines 7630 7362 -268
Branches 1332 1271 -61
==========================================
- Hits 6345 6259 -86
+ Misses 1095 915 -180
+ Partials 190 188 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
3251027
to
bd9bbce
Compare
bd9bbce
to
7c4fd39
Compare
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.
* Remove deprecated `--options` in some tools * Remove deprecated eval hooks and fp16 hooks * Remove deprecated mixup&cutmix, pretrained and return_tuple arguments. * Remove deprecated `deprecated_options` in multi label dataset * Remove deprecated `formating.py` * Remove deprecated default mode of `LabelSmoothLoss` * Revert pretrained argument, and remove it's deprecation sign. * Update unit test
Motivation
Several components have been deprecated for a long time. Remove them now.
Modification
--options
in some toolsreturn_tuple
arguments.formating.py
LabelSmoothLoss
BC-breaking (Optional)
These components or options have been deprecated for a long time, users should have stopped using them.
Checklist
Before PR:
After PR: