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

add conversion matchers for torchvision builtin models #508

Merged
merged 6 commits into from
Nov 25, 2024

Conversation

guozixu2001
Copy link
Contributor

PR Docs

新增 torchvision 转换规则,对应文档:
PaddlePaddle/docs#6959

PR APIs

torchvision.models.alexnet
torchvision.models.densenet121
torchvision.models.densenet161
torchvision.models.densenet169
torchvision.models.densenet201
torchvision.models.googlenet
torchvision.models.inception_v3
torchvision.models.mobilenet_v2
torchvision.models.mobilenet_v3_large
torchvision.models.mobilenet_v3_small
torchvision.models.resnet101
torchvision.models.resnet152
torchvision.models.resnet18
torchvision.models.resnet34
torchvision.models.resnet50
torchvision.models.resnext101_64x4d
torchvision.models.resnext50_32x4d
torchvision.models.shufflenet_v2_x0_5
torchvision.models.shufflenet_v2_x1_0
torchvision.models.shufflenet_v2_x1_5
torchvision.models.shufflenet_v2_x2_0
torchvision.models.squeezenet1_0
torchvision.models.squeezenet1_1
torchvision.models.vgg11
torchvision.models.vgg11_bn
torchvision.models.vgg13
torchvision.models.vgg13_bn
torchvision.models.vgg16
torchvision.models.vgg16_bn
torchvision.models.vgg19
torchvision.models.vgg19_bn
torchvision.models.wide_resnet101_2
torchvision.models.wide_resnet50_2

Copy link

paddle-bot bot commented Nov 16, 2024

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Nov 16, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 18, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 18, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 18, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 18, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 19, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 19, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 19, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 19, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 19, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 19, 2024
@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 20, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 20, 2024
@zhwesky2010
Copy link
Collaborator

infoflow 2024-11-20 14-52-58

@guozixu2001 验证集转失败了

@guozixu2001
Copy link
Contributor Author

infoflow 2024-11-20 14-52-58

@guozixu2001 验证集转失败了

这个对应的转换前的 torch 调用是什么样子的呀?
这个 torch 的 api 是不支持“无关键字”形式的调用的,我感觉应该是因为用了“无关键字”形式的参数才会报这个错。

@zhwesky2010
Copy link
Collaborator

infoflow 2024-11-20 14-52-58
@guozixu2001 验证集转失败了

这个对应的转换前的 torch 调用是什么样子的呀? 这个 torch 的 api 是不支持“无关键字”形式的调用的,我感觉应该是因为用了“无关键字”形式的参数才会报这个错。

你直接在CI里看吧,在CI日志里可以搜索相应的报错,就是我上面截图的。学会根据CI调试也是开发流程中一项重要能力。

@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Nov 21, 2024
@PaddlePaddle PaddlePaddle unlocked this conversation Nov 21, 2024
Copy link
Collaborator

@zhwesky2010 zhwesky2010 left a comment

Choose a reason for hiding this comment

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

  1. progress不是应该被删掉了吗,这个CI是如何通过的
  2. 这个只能测 expect_code 吗,安装了torchvision 也完全无法运行吗

paddle_code = textwrap.dedent(
"""
import paddle
alexnet = paddle.vision.models.alexnet(progress=False, pretrained=False)
Copy link
Collaborator

Choose a reason for hiding this comment

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

progress不是应该被删掉了吗,这个CI是如何通过的

@zhwesky2010 zhwesky2010 merged commit 79acc7d into PaddlePaddle:master Nov 25, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants