Skip to content

【PPSCI Export&Infer No.8】Add export and inference for amgnet_cylinder #1169

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 5 commits into from
Closed

【PPSCI Export&Infer No.8】Add export and inference for amgnet_cylinder #1169

wants to merge 5 commits into from

Conversation

jiangandhao
Copy link

PR types

PR changes

Describe

@paddle-bot Paddle-bot
Copy link

paddle-bot bot commented Jun 18, 2025

Thanks for your contribution!

@CLAassistant
Copy link

CLAassistant commented Jun 18, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

jiangandhao

This comment was marked as off-topic.

Sign in to view

HydrogenSulfate
HydrogenSulfate reviewed Jun 18, 2025
View reviewed changes
docs/zh/examples/amgnet.md Outdated
=== "amgnet_airfoil"

``` sh
python amgnet_airfoil.py mode=export EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_airfoil_pretrained.pdparams
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
python amgnet_airfoil.py mode=export EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_airfoil_pretrained.pdparams
python amgnet_airfoil.py mode=export

docs/zh/examples/amgnet.md Outdated
=== "amgnet_cylinder"

``` sh
python amgnet_cylinder.py mode=export EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_cylinder_pretrained.pdparams
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
python amgnet_cylinder.py mode=export EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_cylinder_pretrained.pdparams
python amgnet_cylinder.py mode=export

docs/zh/examples/amgnet.md Outdated
# curl https://paddle-org.bj.bcebos.com/paddlescience/datasets/AMGNet/data.zip -o data.zip
# unzip it
unzip data.zip
python amgnet_airfoil.py mode=infer EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_airfoil_pretrained.pdparams
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
python amgnet_airfoil.py mode=infer EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_airfoil_pretrained.pdparams
python amgnet_airfoil.py mode=infer

docs/zh/examples/amgnet.md Outdated
# curl https://paddle-org.bj.bcebos.com/paddlescience/datasets/AMGNet/data.zip -o data.zip
# unzip it
unzip data.zip
python amgnet_cylinder.py mode=infer EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_cylinder_pretrained.pdparams
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
python amgnet_cylinder.py mode=infer EVAL.pretrained_model_path=https://paddle-org.bj.bcebos.com/paddlescience/models/amgnet/amgnet_cylinder_pretrained.pdparams
python amgnet_cylinder.py mode=infer

examples/amgnet/amgnet_airfoil.py Outdated
Comment on lines 268 to 270



Copy link
Collaborator

Choose a reason for hiding this comment

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

多个空行请缩减为一行

examples/amgnet/amgnet_airfoil.py Outdated
Comment on lines 279 to 280


Copy link
Collaborator

Choose a reason for hiding this comment

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

同

examples/amgnet/amgnet_airfoil.py Outdated
Comment on lines 289 to 290


Copy link
Collaborator

Choose a reason for hiding this comment

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

同

examples/amgnet/amgnet_airfoil.py Outdated
)



Copy link
Collaborator

Choose a reason for hiding this comment

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

删除多余空行

examples/amgnet/amgnet_cylinder.py
Copy link
Collaborator

Choose a reason for hiding this comment

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

examples/amgnet/amgnet_airfoil.py一样的问题，请一起修改

@HydrogenSulfate
Copy link
Collaborator

@jiangandhao 可参考

PaddleScience/examples/allen_cahn/conf/allen_cahn_piratenet.yaml

Lines 82 to 98 in e0f0245

# inference settings
INFER:
pretrained_model_path: https://paddle-org.bj.bcebos.com/paddlescience/models/AllenCahn/allen_cahn_piratenet_pretrained.pdparams
export_path: ./inference/allen_cahn
pdmodel_path: ${INFER.export_path}.pdmodel
pdiparams_path: ${INFER.export_path}.pdiparams
onnx_path: ${INFER.export_path}.onnx
device: gpu
engine: native
precision: fp32
ir_optim: true
min_subgraph_size: 5
gpu_mem: 2000
gpu_id: 0
max_batch_size: 1024
num_cpu_threads: 10
batch_size: 1024

添加INFER字段到配置文件中

jiangandhao
jiangandhao commented Jun 18, 2025
View reviewed changes
Copy link
Author

@jiangandhao jiangandhao left a comment

Choose a reason for hiding this comment

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

应该没问题了

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Jun 19, 2025
@luotao1
Copy link
Collaborator

luotao1 commented Jun 19, 2025

PR标题请按照规范修改

@jiangandhao jiangandhao changed the title 完成export和inference功能，修改AMGNet文档 PPSCI Export&Infer No.8 Jun 19, 2025
@jiangandhao
Copy link
Author

Collaborator

已修改

@luotao1
Copy link
Collaborator

luotao1 commented Jun 19, 2025

https://www.paddlepaddle.org.cn/documentation/docs/zh/dev_guides/git_guides/codestyle_check_guide_cn.html

请安装precommit工具，来通过CodeStyle流水线的格式检查

@luotao1 luotao1 changed the title PPSCI Export&Infer No.8 【PPSCI Export&Infer No.8】Add export and inference for amgnet_cylinder Jun 19, 2025
@HydrogenSulfate
Copy link
Collaborator

@jiangandhao 你好，代码无法跑通，如果自测可以跑通的话请贴上跑通模型导出和推理的截图到PR中，谢谢

@luotao1 luotao1 mentioned this pull request Jun 19, 2025
Closed
@jiangandhao
Copy link
Author

0b2b653a-b317-4470-92bc-bc0de4c51bd8
这个是通过还是没通过？

@jiangandhao 你好，代码无法跑通，如果自测可以跑通的话请贴上跑通模型导出和推理的截图到PR中，谢谢

@HydrogenSulfate
Copy link
Collaborator

0b2b653a-b317-4470-92bc-bc0de4c51bd8 这个是通过还是没通过？

@jiangandhao 你好，代码无法跑通，如果自测可以跑通的话请贴上跑通模型导出和推理的截图到PR中，谢谢

上面这个截图里的是普通的测试流水线，并不会测试你PR新增的功能，而我说的自测是指：你这个PR是支持amgnet两个模型的导出和推理，那你需要本地自测是否能够跑通导出、推理，以及推理结果是否正确（和eval的结果相同）。

@HydrogenSulfate
Copy link
Collaborator

HydrogenSulfate commented Jun 19, 2025 •
edited
Loading

另外给开源仓库提交PR时，请安装pre-commit：
image
https://paddlescience-docs.readthedocs.io/zh-cn/latest/zh/development/#1

@jiangandhao jiangandhao closed this by deleting the head repository Jun 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HydrogenSulfate HydrogenSulfate HydrogenSulfate left review comments

Assignees

@luotao1 luotao1

@HydrogenSulfate HydrogenSulfate

Labels

contributor HappyOpenSource 快乐开源活动issue与PR

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jiangandhao @CLAassistant @HydrogenSulfate @luotao1