Skip to content

5431 torch add113 drop17 baseimage2210#5460

Merged
wyli merged 18 commits intodevfrom
5431-torch-add113-drop17
Nov 7, 2022
Merged

5431 torch add113 drop17 baseimage2210#5460
wyli merged 18 commits intodevfrom
5431-torch-add113-drop17

Conversation

@wyli
Copy link
Copy Markdown
Contributor

@wyli wyli commented Nov 3, 2022

Fixes #5431
Fixes #5452

Description

  • deprecating pytorch 1.7 tests
  • adds pytorch 1.13 tests
  • upgrade base image to pytorch 22.10

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
  • In-line docstrings updated.
  • Documentation updated, tested make html command in the docs/ folder.

@wyli wyli force-pushed the 5431-torch-add113-drop17 branch 2 times, most recently from 8c538ef to 5f0bac7 Compare November 3, 2022 07:58
@YanxuanLiu YanxuanLiu changed the title 5431 torch add113 drop17 5431 torch add113 drop17 [skip ci] Nov 3, 2022
@wyli wyli force-pushed the 5431-torch-add113-drop17 branch 2 times, most recently from d9f1f3e to 76cc73d Compare November 3, 2022 10:46
@wyli wyli changed the title 5431 torch add113 drop17 [skip ci] 5431 torch add113 drop17 Nov 3, 2022
@wyli wyli changed the title 5431 torch add113 drop17 5431 torch add113 drop17 baseimage 2210 Nov 3, 2022
@wyli wyli changed the title 5431 torch add113 drop17 baseimage 2210 5431 torch add113 drop17 baseimage2210 Nov 3, 2022
@wyli wyli requested a review from YanxuanLiu November 3, 2022 14:00
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Nov 3, 2022

seems to work fine with the github runner, would be great to test the same environments for blossom @YanxuanLiu before you approve this PR

@wyli wyli marked this pull request as ready for review November 3, 2022 14:02
@wyli wyli force-pushed the 5431-torch-add113-drop17 branch from d80cbd3 to abc58b3 Compare November 6, 2022 22:08
wyli added 15 commits November 6, 2022 22:36
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Lee Newberg <[email protected]>

Re-writes version numbers as strings rather than numbers because 3.1 !=
3.10, etc. Nothing is broken at present but in the future as, e.g., the
Python version for some things goes from 3.8 to 3.9 to 3.10, this could
end up being an unexpected problem.

<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).

Signed-off-by: Lee Newberg <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Wenqi Li <[email protected]>
@wyli wyli force-pushed the 5431-torch-add113-drop17 branch from c0fc5ae to c23f06b Compare November 6, 2022 22:37
Signed-off-by: Wenqi Li <[email protected]>
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Nov 7, 2022

/build

@wyli wyli requested a review from YanxuanLiu November 7, 2022 09:58
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Nov 7, 2022

this is tested on blossom @YanxuanLiu @Nic-Ma I think it's ready

@wyli wyli requested a review from Nic-Ma November 7, 2022 14:17
@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Nov 7, 2022

/build

@wyli
Copy link
Copy Markdown
Contributor Author

wyli commented Nov 7, 2022

/build

@wyli wyli enabled auto-merge (squash) November 7, 2022 16:13
@wyli wyli merged commit 3f1eac8 into dev Nov 7, 2022
@wyli wyli deleted the 5431-torch-add113-drop17 branch November 7, 2022 16:58
bhashemian pushed a commit to bhashemian/MONAI that referenced this pull request Nov 23, 2022
Fixes Project-MONAI#5431 
Fixes Project-MONAI#5452

### Description
- deprecating pytorch 1.7 tests
- adds pytorch 1.13 tests
- upgrade base image to pytorch 22.10

### Types of changes
<!--- Put an `x` in all the boxes that apply, and remove the not
applicable items -->
- [x] Non-breaking change (fix or new feature that would not break
existing functionality).
- [ ] Breaking change (fix or new feature that would cause existing
functionality to change).
- [x] New tests added to cover the changes.
- [x] Integration tests passed locally by running `./runtests.sh -f -u
--net --coverage`.
- [x] Quick tests passed locally by running `./runtests.sh --quick
--unittests --disttests`.
- [x] In-line docstrings updated.
- [ ] Documentation updated, tested `make html` command in the `docs/`
folder.

Signed-off-by: Wenqi Li <[email protected]>
Signed-off-by: Behrooz <[email protected]>
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.

Upgrade PyTorch base docker to 22.10 tests for torch 1.13 (and drop support for torch 1.7.x?)

3 participants