-
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] Support mmcls with NPU backend. #1072
Conversation
673d58a
to
6706ba8
Compare
Codecov ReportBase: 86.09% // Head: 85.98% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #1072 +/- ##
==========================================
- Coverage 86.09% 85.98% -0.12%
==========================================
Files 140 142 +2
Lines 9676 9906 +230
Branches 1677 1715 +38
==========================================
+ Hits 8331 8518 +187
- Misses 1092 1124 +32
- Partials 253 264 +11
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
To be compatible with the old version of MMCV, please avoid importing new API from MMCV.
Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
Added ascending device support in mmcv.
Modification
mmcls/apis/train.py
, We added an NPU device in the DDP scenario, and the mixing accuracy is enabled by default when using the NPUBC-breaking (Optional)
A new mmcv containing the NPU background needed.
Use cases (Optional)
We tested resnet50_8xb32_in1k.py with
Top-1 (%): 76.4
Checklist
Before PR:
After PR: