-
Notifications
You must be signed in to change notification settings - Fork 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
[Improvement] Rename config files according to the config name standard. #508
Conversation
Codecov Report
@@ Coverage Diff @@
## master #508 +/- ##
=======================================
Coverage 79.87% 79.87%
=======================================
Files 107 107
Lines 6093 6093
Branches 987 987
=======================================
Hits 4867 4867
Misses 1094 1094
Partials 132 132
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
55659b8
to
244460a
Compare
244460a
to
0291d9a
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.
@@ -0,0 +1,8 @@ | |||
_base_ = [ |
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.
should be mobilenet-v3-small_8xb16_cifar10.py
.
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.
…rd. (open-mmlab#508) * Update tnt config * Rename config files * Update docs * Update metafile in dev_scripts * Fix some files * Remove fp16 metafile and README. * Fix names in metafiles.
Motivation
We have come up with a config filename convention, but many config files don't use this convention.
To make the filenames more clear, change these config names.
Modification
Like the title
The comparison table:
BC-breaking (Optional)
No, we keep all original config files and add a deprecation warning.
Checklist
Before PR:
After PR: