Skip to content

[New Sample] Add vovnet39a model computational graph and extraction script#670

Merged
JewelRoam merged 3 commits intoPaddlePaddle:developfrom
darkerkiller:add-resnet18-sample
Mar 9, 2026
Merged

[New Sample] Add vovnet39a model computational graph and extraction script#670
JewelRoam merged 3 commits intoPaddlePaddle:developfrom
darkerkiller:add-resnet18-sample

Conversation

@darkerkiller
Copy link
Contributor

@darkerkiller darkerkiller commented Mar 7, 2026

Model: vovnet39a
Framework: Pytorch
Dependency: timm, torch
Content: 1. Switched model to vovnet39a from timm.
2. Corrected the directory structure to samples/timm/vovnet39a/ as guided.
3. Passed local Validation API and completely passed all pre-commit style hooks.

@paddle-bot
Copy link

paddle-bot bot commented Mar 7, 2026

Thanks for your contribution!

@paddle-bot paddle-bot bot added the contributor External developers label Mar 7, 2026
@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Mar 9, 2026
@JewelRoam
Copy link
Collaborator

这个resnet18和timm以及torchvision里边的是一样的吗?如果是的话会有重复,可找到其它模型试试

@JewelRoam
Copy link
Collaborator

JewelRoam commented Mar 9, 2026

你可以看一下当前GraphNet/samples下面的结构,例如你的timm_samples只需要放在timm目录就好了,之前的torch_samples也需要去掉。如更换了模型,记得把原来的git rm并修改PR描述,或者重新单独起PR

@JewelRoam
Copy link
Collaborator

另外,code style检查也记得根据CI里面的指引pre-commit一下

@JewelRoam JewelRoam changed the title [New Sample] Add resnet18 computational graph and extraction script [New Sample] Add vovnet39a model computational graph and extraction script Mar 9, 2026
Copy link
Collaborator

@JewelRoam JewelRoam left a comment

Choose a reason for hiding this comment

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

已合入,感谢贡献!

@JewelRoam JewelRoam merged commit 290027e into PaddlePaddle:develop Mar 9, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants