Skip to content
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 EfficientNet #649

Merged
merged 52 commits into from
Jan 25, 2022
Merged

[Feature] Support EfficientNet #649

merged 52 commits into from
Jan 25, 2022

Conversation

0x4f5da2
Copy link
Contributor

Motivation

Based on #289

Modification

add efficientnet and model conversion script.

@codecov
Copy link

codecov bot commented Jan 16, 2022

Codecov Report

Merging #649 (520b47f) into dev (321ad09) will increase coverage by 0.47%.
The diff coverage is 88.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #649      +/-   ##
==========================================
+ Coverage   81.78%   82.26%   +0.47%     
==========================================
  Files         118      120       +2     
  Lines        6820     7042     +222     
  Branches     1174     1228      +54     
==========================================
+ Hits         5578     5793     +215     
+ Misses       1082     1075       -7     
- Partials      160      174      +14     
Flag Coverage Δ
unittests 82.26% <88.00%> (+0.47%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
mmcls/models/backbones/efficientnet.py 87.58% <87.58%> (ø)
mmcls/models/backbones/__init__.py 100.00% <100.00%> (ø)
mmcls/models/utils/inverted_residual.py 100.00% <100.00%> (ø)
mmcls/datasets/multi_label.py 75.00% <0.00%> (-3.95%) ⬇️
mmcls/datasets/base_dataset.py 91.11% <0.00%> (-2.08%) ⬇️
mmcls/models/utils/augment/mixup.py 96.66% <0.00%> (ø)
mmcls/models/utils/augment/cutmix.py 96.82% <0.00%> (ø)
mmcls/models/utils/augment/identity.py 100.00% <0.00%> (ø)
mmcls/models/utils/augment/utils.py 100.00% <0.00%> (ø)
mmcls/models/losses/asymmetric_loss.py 97.50% <0.00%> (+0.35%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 321ad09...520b47f. Read the comment docs.

configs/_base_/models/efficientnet_es.py Outdated Show resolved Hide resolved
mmcls/models/backbones/efficientnet.py Outdated Show resolved Hide resolved
mmcls/models/backbones/efficientnet.py Outdated Show resolved Hide resolved
mmcls/models/backbones/efficientnet.py Outdated Show resolved Hide resolved
@0x4f5da2 0x4f5da2 requested a review from mzr1996 January 17, 2022 07:08
} # yapf: disable

# Parameters to build different kinds of architecture.
# From left to right: scaling factor for width, scaling factor for depth,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The statement of width and depth is not very intuitive, it is recommended to supplement.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The width and depth are explained in the image of README

@mzr1996 mzr1996 changed the title [Feature]EfficientNet [Feature] Support EfficientNet Jan 25, 2022
Copy link
Member

@mzr1996 mzr1996 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mzr1996 mzr1996 merged commit d56170a into open-mmlab:dev Jan 25, 2022
Ezra-Yu pushed a commit to Ezra-Yu/mmclassification that referenced this pull request Feb 14, 2022
* add config for resnest test

* fix config

* add label smoothing

* add memcached

* minor fix

* fix bug

* fix config

* add config

* minor fix

* fix configs

* use EResize

* change interpolation

* add more configs

* add docsting

* add unittest

* remove unnecessary changes

* minor fix

* add more docstring

* fix linting

* add efficient backbone

* add config

* add Edge Residual

* fix bug

* remove unnecessary files

* refactor

* add resize in crop to ensure crop size is output size

* fix bug and add comments

* test

* fix

* add more configs

* add more configs

* add more configs

* fix bug

* add model zoo

* fix

* reorganize code

* add edge tpu

* add edge tpu converter

* rename

* update readme

* reorganize code and config

* Rename configs of EfficientNet, and add metafile & model_zoo

* Remove `backend='pillow'`

* Add comments about EfficientNet-EdgeTPU

* Rename the convert tool of EfficientNet.

* Refactor EfficientNet and update docstring.

* Update EfficientNet-EdgeTPU config

* Fix unit tests

Co-authored-by: lixinran <lixr423@outlook.com>
Co-authored-by: lixinran <lixinran@sensetime.com>
Co-authored-by: mzr1996 <mzr1996@163.com>
mzr1996 added a commit to mzr1996/mmpretrain that referenced this pull request Nov 24, 2022
* add config for resnest test

* fix config

* add label smoothing

* add memcached

* minor fix

* fix bug

* fix config

* add config

* minor fix

* fix configs

* use EResize

* change interpolation

* add more configs

* add docsting

* add unittest

* remove unnecessary changes

* minor fix

* add more docstring

* fix linting

* add efficient backbone

* add config

* add Edge Residual

* fix bug

* remove unnecessary files

* refactor

* add resize in crop to ensure crop size is output size

* fix bug and add comments

* test

* fix

* add more configs

* add more configs

* add more configs

* fix bug

* add model zoo

* fix

* reorganize code

* add edge tpu

* add edge tpu converter

* rename

* update readme

* reorganize code and config

* Rename configs of EfficientNet, and add metafile & model_zoo

* Remove `backend='pillow'`

* Add comments about EfficientNet-EdgeTPU

* Rename the convert tool of EfficientNet.

* Refactor EfficientNet and update docstring.

* Update EfficientNet-EdgeTPU config

* Fix unit tests

Co-authored-by: lixinran <lixr423@outlook.com>
Co-authored-by: lixinran <lixinran@sensetime.com>
Co-authored-by: mzr1996 <mzr1996@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants