Skip to content

Add deprecation message for deploy tools#697

Merged
mzr1996 merged 5 commits intoopen-mmlab:devfrom
RunningLeon:deprecate_deploy
Feb 23, 2022
Merged

Add deprecation message for deploy tools#697
mzr1996 merged 5 commits intoopen-mmlab:devfrom
RunningLeon:deprecate_deploy

Conversation

@RunningLeon
Copy link
Copy Markdown
Collaborator

@RunningLeon RunningLeon commented Feb 17, 2022

Motivation

Since MMDeploy has been released, the deployment tools scattered in each OpenMMLab codebase will be deprecated in future.

Modification

Add deprecation message in deployment tools.
Used package colorama to print cross-platform colored text.

BC-breaking (Optional)

None

Use cases (Optional)

And the message would look like it in Ubuntu18.04 terminal:
image

@RunningLeon
Copy link
Copy Markdown
Collaborator Author

@lvhan028 Hi, any comment on this PR?

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 17, 2022

Codecov Report

Merging #697 (e0b5275) into dev (1a91a7f) will increase coverage by 3.25%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #697      +/-   ##
==========================================
+ Coverage   81.78%   85.03%   +3.25%     
==========================================
  Files         118      121       +3     
  Lines        6820     7393     +573     
  Branches     1174     1278     +104     
==========================================
+ Hits         5578     6287     +709     
+ Misses       1082      916     -166     
- Partials      160      190      +30     
Flag Coverage Δ
unittests 85.03% <ø> (+3.25%) ⬆️

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

Impacted Files Coverage Δ
mmcls/apis/train.py 19.17% <0.00%> (-2.08%) ⬇️
mmcls/datasets/base_dataset.py 91.11% <0.00%> (-2.08%) ⬇️
mmcls/core/__init__.py 100.00% <0.00%> (ø)
mmcls/utils/__init__.py 100.00% <0.00%> (ø)
mmcls/models/necks/__init__.py 100.00% <0.00%> (ø)
mmcls/models/utils/__init__.py 100.00% <0.00%> (ø)
mmcls/models/backbones/repvgg.py 96.84% <0.00%> (ø)
mmcls/models/backbones/resnet.py 100.00% <0.00%> (ø)
mmcls/core/evaluation/__init__.py 100.00% <0.00%> (ø)
mmcls/core/optimizers/__init__.py 100.00% <0.00%> (ø)
... and 33 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 1a91a7f...e0b5275. Read the comment docs.

@RunningLeon RunningLeon changed the title [WIP]: Add deprecation message for deploy tools Add deprecation message for deploy tools Feb 17, 2022
@RunningLeon RunningLeon requested a review from mzr1996 February 17, 2022 08:33
Copy link
Copy Markdown

@lvhan028 lvhan028 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 changed the base branch from master to dev February 23, 2022 03:39
RunningLeon and others added 2 commits February 23, 2022 17:41
@RunningLeon RunningLeon requested a review from mzr1996 February 23, 2022 10:39
Copy link
Copy Markdown
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 9ee4157 into open-mmlab:dev Feb 23, 2022
mzr1996 added a commit to mzr1996/mmpretrain that referenced this pull request Nov 24, 2022
* add derepcation message for deploy tools

* add space

* Update tools/deployment/onnx2tensorrt.py

Co-authored-by: Ma Zerun <mzr1996@163.com>

* remove colorama package

* use plaintext to avoid import mmcv

Co-authored-by: Ma Zerun <mzr1996@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants