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

【PPSCI Export&Infer No.22】VP_NSFNet4 #864

Merged
merged 5 commits into from
Apr 24, 2024

Conversation

smallpoxscattered
Copy link
Contributor

Copy link

paddle-bot bot commented Apr 23, 2024

Thanks for your contribution!

export_path: ./inference/VP_NSFNet4
pdmodel_path: ${INFER.export_path}.pdmodel
pdpiparams_path: ${INFER.export_path}.pdiparams
output_keys: ['p', 'u', 'v', 'w']
Copy link
Collaborator

Choose a reason for hiding this comment

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

删除,跟MODEL.output_keys重复了


pred = {
store_key: pred[infer_key]
for store_key, infer_key in zip(cfg.INFER.output_keys, pred.keys())
Copy link
Collaborator

Choose a reason for hiding this comment

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

所有INFER.output_keys替换成MODEL.output_keys

Copy link
Contributor Author

Choose a reason for hiding this comment

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

没 我不知道为什么 输出被打乱了 要这么对应 输出才是和预测一样的

Copy link
Collaborator

Choose a reason for hiding this comment

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

没 我不知道为什么 输出被打乱了 要这么对应 输出才是和预测一样的

应该是一个已知的问题,可能是jit.save、jit.load存在的BUG导致导出模型的输出顺序跟动态图不一致。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

那我是不是得删掉

Copy link
Collaborator

Choose a reason for hiding this comment

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

那我是不是得删掉

先这样吧,等修复之后我再改

for store_key, infer_key in zip(cfg.INFER.output_keys, sol.keys())
}
fig, ax = plt.subplots(1, 4, figsize=(16, 4))
cmap = plt.cm.get_cmap("jet")
Copy link
Collaborator

Choose a reason for hiding this comment

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

所有的plt.cm.get_cmap替换成matplotlib.colormaps.get_cmap

Copy link
Contributor Author

Choose a reason for hiding this comment

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

所有的plt.cm.get_cmap替换成matplotlib.colormaps.get_cmap

老师 改好了

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

@HydrogenSulfate HydrogenSulfate merged commit 0717f06 into PaddlePaddle:develop Apr 24, 2024
2 of 4 checks passed
huohuohuohuohuo123 pushed a commit to huohuohuohuohuo123/PaddleScience that referenced this pull request Aug 12, 2024
* [SCI Export&Infer No.24] biharmonic2d

* P[PSCI Export&Infer No.724] biharmonic2d fix

* add export&infer nsfnet4

* add export&infer nsfnet4
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor HappyOpenSource 快乐开源活动issue与PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants