Skip to content

[Feature] Move 'Install MMCV' to a independent CI item.#602

Merged
xvjiarui merged 8 commits intoopen-mmlab:masterfrom
clownrat6:new_mmcv_ci
Jun 17, 2021
Merged

[Feature] Move 'Install MMCV' to a independent CI item.#602
xvjiarui merged 8 commits intoopen-mmlab:masterfrom
clownrat6:new_mmcv_ci

Conversation

@clownrat6
Copy link
Copy Markdown
Contributor

  1. Add torch_version and cuda_version into matrix;

  2. Add 'Install MMCV_CPU' CI item to install mmcv-full in cpu environment;

  3. Add 'Install MMCV_GPU' CI item to install mmcv-full in gpu environment;

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 16, 2021

Codecov Report

Merging #602 (4538821) into master (5d46314) will decrease coverage by 0.30%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #602      +/-   ##
==========================================
- Coverage   86.26%   85.95%   -0.31%     
==========================================
  Files         101      101              
  Lines        5278     5234      -44     
  Branches      854      828      -26     
==========================================
- Hits         4553     4499      -54     
  Misses        561      561              
- Partials      164      174      +10     
Flag Coverage Δ
unittests 85.95% <ø> (-0.31%) ⬇️

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

Impacted Files Coverage Δ
mmseg/models/backbones/mobilenet_v3.py 94.82% <0.00%> (-2.66%) ⬇️
mmseg/models/segmentors/encoder_decoder.py 88.35% <0.00%> (-2.50%) ⬇️
mmseg/models/backbones/cgnet.py 94.63% <0.00%> (-2.13%) ⬇️
mmseg/models/backbones/mobilenet_v2.py 71.08% <0.00%> (-1.86%) ⬇️
mmseg/models/backbones/hrnet.py 82.45% <0.00%> (-1.85%) ⬇️
mmseg/models/backbones/fast_scnn.py 97.02% <0.00%> (-1.07%) ⬇️
mmseg/models/backbones/unet.py 94.87% <0.00%> (-0.93%) ⬇️
mmseg/models/backbones/vit.py 86.15% <0.00%> (-0.90%) ⬇️
mmseg/models/segmentors/base.py 44.11% <0.00%> (-0.84%) ⬇️
mmseg/models/decode_heads/decode_head.py 91.35% <0.00%> (-0.31%) ⬇️
... and 5 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 5d46314...4538821. Read the comment docs.

include:
- torch: 1.3.0+cpu
torchvision: 0.4.1+cpu
torch_version: 1.3.0
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We may add device to specify cuda / cpu.

if: ${{matrix.torchvision == '0.4.1'}}
run: pip install Pillow==6.2.2
- name: Install PyTorch
run: pip install torch==${{matrix.torch}} torchvision==${{matrix.torchvision}} -f https://download.pytorch.org/whl/torch_stable.html
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

We need to install different device pytorch version.

matrix:
python-version: [ 3.6, 3.7 ]
torch: [ 1.3.0+cpu, 1.5.0+cpu, 1.5.0+cu101, 1.6.0+cu101, 1.7.0+cu101, 1.8.0+cu101 ]
torch: [ 1.3.0, 1.4.0, 1.5.0, 1.6.0, 1.7.0, 1.8.0 ]
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

device:

@xvjiarui xvjiarui merged commit 608f842 into open-mmlab:master Jun 17, 2021
bowenroom pushed a commit to bowenroom/mmsegmentation that referenced this pull request Feb 25, 2022
* [Feature] Move 'Install MMCV' to a independent CI item.

* Merge MMCV install into MMSEG dependencies install

* Fix bug of 'Install MMCV'

* Remove duplicate CI items

* Fix torch device

* Split cpu env and gpu env into two CI project

* Fix some mmdet related bugs

* Fix mmcv-full install bug of build_cpu CI project.
aravind-h-v pushed a commit to aravind-h-v/mmsegmentation that referenced this pull request Mar 27, 2023
Fix params replication when sing the dummy checker.
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.

2 participants