-
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
[Improvement] Refactor Swin-Transformer #800
Conversation
Codecov Report
@@ Coverage Diff @@
## master #800 +/- ##
==========================================
+ Coverage 89.62% 89.67% +0.05%
==========================================
Files 113 114 +1
Lines 6263 6431 +168
Branches 989 1007 +18
==========================================
+ Hits 5613 5767 +154
- Misses 452 462 +10
- Partials 198 202 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
resolve some comments. |
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, it seems that the code coverage should be improved.
Coverage improved. The rest should come from the loading checkpoint part. |
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 and I will re-run the benchmark to double-check correctness.
* [Improvement] Refactor Swin-Transformer * fixed swin test * update patch emebd, add more tests * fixed test * remove pretrain_style * fixed padding * resolve coments * use mmcv 2tuple * refactor init_cfg Co-authored-by: Junjun2016 <hejunjun@sjtu.edu.cn>
* add LiteHRNet into README.md and README_CN.md * minor fix to webcam demo and docs
…b#800) * first commit * update changelog * add verify in onnx unittest * change unittest Co-authored-by: dreamerlin <528557675@qq.com>
Motivation
Refactor Swin-Transformer related code as open-mmlab/mmdetection#5748
BC-breaking (Optional)
We no longer support directly loading official weight here, instead, we will use the conversion script in #783