Skip to content

Replace deprecated arg in torchvision models - #6401

Merged
wyli merged 3 commits into
Project-MONAI:devfrom
bhashemian:torchvision_pretrain
Apr 20, 2023
Merged

wyli merged 3 commits into
Project-MONAI:devfrom
bhashemian:torchvision_pretrain

Conversation

@bhashemian

@bhashemian bhashemian commented Apr 19, 2023 •

Copy link
Copy Markdown
Member

Description

For torchvision models, the parameter pretrained is deprecated since 0.13 and may be removed in the future. Our TorchVisionFC model has been using this parameter. This PR updated TorchVisionFC to remove calling pretrained and instead to call weights with appropriate defaults without any change in the functionality (backward compatible).

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
  • Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.

Signed-off-by: Behrooz <3968947+drbeh@users.noreply.github.com>
@bhashemian
bhashemian requested review from Nic-Ma and wyli April 19, 2023 16:46
@bhashemian bhashemian changed the title Remove deprecated arg in torchvision models Replace deprecated arg in torchvision models Apr 19, 2023

@Nic-Ma Nic-Ma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me.
Please ensure it's compatible with PyTorch 1.7 - 2.0.

Thanks.

@Nic-Ma

Nic-Ma commented Apr 19, 2023

Copy link
Copy Markdown
Contributor

/build

Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli

wyli commented Apr 20, 2023

Copy link
Copy Markdown
Contributor

/build

@wyli
wyli enabled auto-merge (squash) April 20, 2023 06:57
@wyli
wyli merged commit 05dbc86 into Project-MONAI:dev Apr 20, 2023
@bhashemian
bhashemian deleted the torchvision_pretrain branch April 20, 2023 13:35
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.

3 participants