[Feature] Support features_only in TIMMBackbone#668
Merged
mzr1996 merged 4 commits intoopen-mmlab:devfrom Jan 25, 2022
Merged
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #668 +/- ##
==========================================
+ Coverage 82.12% 82.14% +0.01%
==========================================
Files 119 119
Lines 6865 6893 +28
Branches 1184 1192 +8
==========================================
+ Hits 5638 5662 +24
- Misses 1063 1066 +3
- Partials 164 165 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Ezra-Yu
pushed a commit
to Ezra-Yu/mmclassification
that referenced
this pull request
Feb 14, 2022
* Support features_only in TIMMBackbone based on open-mmlab/mmsegmentation#998 * update test for mmdet * fix unit test for build_without_timm * Update docstring Co-authored-by: mzr1996 <mzr1996@163.com>
mzr1996
added a commit
to mzr1996/mmpretrain
that referenced
this pull request
Nov 24, 2022
* Support features_only in TIMMBackbone based on open-mmlab/mmsegmentation#998 * update test for mmdet * fix unit test for build_without_timm * Update docstring Co-authored-by: mzr1996 <mzr1996@163.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
See the discussion in open-mmlab/mmdetection#7020. In MMDet, MMSeg, and other downstream repos, we wish to directly use backbones supported by TIMM in MMCls. Therefore, it is necessary to support this option for downstream tasks.
This PR will close #665.
This PR (especially
test_timm_backbone_features_only) is based on open-mmlab/mmsegmentation#998.Modification
features_only=Truefeatures_only=Truetest_timm_backboneUse cases (Optional)
MMDetection
Here is an example config
retinanet_timm_tv_resnet50_fpn_fp16_4x4_1x_coco.py.The results at epoch 1 (
bbox_mAP_copypaste: 0.162 0.285 0.164 0.084 0.201 0.201) are similar to those ofretinanet_r50_fpn_fp16_1x_coco.py(bbox_mAP_copypaste: 0.164 0.284 0.168 0.082 0.189 0.203).MMSegmentation
Here is an example config
upernet_timm_resnet50d_512x512_20k_voc12aug.py.mIoU 69.75 at 2000 iter. umm... too high? I'm not familiar with mmseg, and the config may be wrong.
In any case, training and evaluation work.
Checklist
Before PR:
After PR: