-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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] Update digit_version #778
Conversation
Codecov Report
@@ Coverage Diff @@
## master #778 +/- ##
==========================================
+ Coverage 85.37% 85.44% +0.06%
==========================================
Files 107 107
Lines 5847 5859 +12
Branches 960 962 +2
==========================================
+ Hits 4992 5006 +14
+ Misses 673 670 -3
- Partials 182 183 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
* update digit_version * add unittest * fix import
* add vipnas link * use papername instead of keyline in keyword matching * use unified paper title format * remove print
* add cn pr no * add comma
Motivation
The previous
digit_version
has some problems, refers to open-mmlab/mmcv#1185 and open-mmlab/mmcv#1235Modification
Copy the
digit_version
from mmcv and use it to parse version.BC-breaking (Optional)
Does the modification introduce changes that break the backward-compatibility of the downstream repos?
No
Use cases (Optional)
If this PR introduces a new feature, it is better to list some use cases here, and update the documentation.
Checklist