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

【SCU】【PPSCI Export&Infer No.17】poiseuille_flow #1030

Merged
merged 18 commits into from
Dec 4, 2024

Conversation

PolaKuma
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

#788
为pipe案例添加了export和inference功能

Copy link

paddle-bot bot commented Nov 25, 2024

Thanks for your contribution!

@PolaKuma
Copy link
Contributor Author

@HydrogenSulfate 拜托佬再看看!感谢~

examples/pipe/poiseuille_flow.py Outdated Show resolved Hide resolved
Comment on lines +435 to +437
model_u = ppsci.arch.MLP(**cfg.MODEL.u_net)
model_v = ppsci.arch.MLP(**cfg.MODEL.v_net)
model_p = ppsci.arch.MLP(**cfg.MODEL.p_net)
Copy link
Collaborator

Choose a reason for hiding this comment

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

  1. 看起来缺少这部分代码代码,image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done. 之前测试可以导出就没加这部分代码,已经修改好了。

solver.export(input_spec, export_path)


def inference(cfg: DictConfig):
Copy link
Collaborator

Choose a reason for hiding this comment

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

inference应该使用Predictor,加载export导出的模型进行计算,而不是使用solver

HydrogenSulfate
HydrogenSulfate previously approved these changes Dec 4, 2024
Copy link
Collaborator

@HydrogenSulfate HydrogenSulfate left a comment

Choose a reason for hiding this comment

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

LGTM

docs/zh/examples/labelfree_DNN_surrogate.md Outdated Show resolved Hide resolved
@luotao1 luotao1 merged commit 213bd31 into PaddlePaddle:develop Dec 4, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource Pro 进阶版快乐开源活动,更具挑战性的任务
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants