-
Notifications
You must be signed in to change notification settings - Fork 215
【PPSCI Export&Infer No.1】 Add export and inference for deephpms_schrodinger #1170
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
base: develop
Are you sure you want to change the base?
Conversation
Thanks for your contribution! |
这样可以了吗? |
请 @lxw123123 审核 |
@lxw123123 请求审核 |
@HydrogenSulfate 请求审核 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
基本完成了export和inference的开发,通过eval_schrodinger.py和infer_schrodinger.py导出本地自测结果(原代码文件名过长无法正常加载,使用新建文件方式跳过部分中间文件)
@@ -11,7 +11,7 @@ defaults: | |||
hydra: | |||
run: | |||
# dynamic output directory according to running time and override name | |||
dir: outputs_schrodinger/${now:%Y-%m-%d}/${now:%H-%M-%S}/${hydra.job.override_dirname} | |||
dir: outputs_schrodinger/${now:%Y-%m-%d}/${now:%H-%M-%S}/schrodinger_output |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个不用改
#export settings | ||
EXPORT: | ||
pretrained_model_path: ${output_dir}/checkpoints/latest | ||
export_path: ${output_dir}/inference_model |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
直接放INFER字段里即可,不需要单独添加EXPORT字段,跟其它案例保持一致
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
你的PR是添加export&inference,为什么要再写一个evaluate呢?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
export&infernece功能请封装到函数加到原有案例文件中,不需要单独新建文件
mkdocs.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
mkdocs.yml里有deephpms,不需要重复添加
在导出过程中,遇到了无法使用静态图导出的问题,slover一运行就中断,使用json.dump和paddle.save保存模型参数,然后基于此进行推理 |
@HydrogenSulfate 这样笔试算过了吗 |
我已经完成了【PPSCI Export&Infer No.1】 Add export and inference for deephpms_schrodinger by jiangandhao · Pull Request #1170 · PaddlePaddle/PaddleScience的提交,可以根据PaddleScience\docs\zh\examples\deephpms.md跑通,笔试算过了吗?
…------------------ 原始邮件 ------------------
发件人: "PaddlePaddle/PaddleScience" ***@***.***>;
发送时间: 2025年6月20日(星期五) 下午2:59
***@***.***>;
***@***.******@***.***>;
主题: Re: [PaddlePaddle/PaddleScience] 【PPSCI Export&Infer No.1】 Add export and inference for deephpms_schrodinger (PR #1170)
@HydrogenSulfate commented on this pull request.
In examples/deephpms/conf/schrodinger.yaml:
> @@ -11,7 +11,7 @@ defaults: hydra: run: # dynamic output directory according to running time and override name - dir: outputs_schrodinger/${now:%Y-%m-%d}/${now:%H-%M-%S}/${hydra.job.override_dirname} + dir: outputs_schrodinger/${now:%Y-%m-%d}/${now:%H-%M-%S}/schrodinger_output
这个不用改
In examples/deephpms/conf/schrodinger.yaml:
> +#export settings +EXPORT: + pretrained_model_path: ${output_dir}/checkpoints/latest + export_path: ${output_dir}/inference_model
直接放INFER字段里即可,不需要单独添加EXPORT字段,跟其它案例保持一致
On examples/deephpms/eval_schrodinger.py:
你的PR是添加export&inference,为什么要再写一个evaluate呢?
On examples/deephpms/infer_schrodinger.py:
export&infernece功能请封装到函数加到原有案例文件中,不需要单独新建文件
On mkdocs.yml:
mkdocs.yml里有deephpms,不需要重复添加
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个文件不需要在提交代码中出现
修改完小问题后,可以算作完成这个笔试题目 |
infer原本导出路径和eval的导出路径是类似的,都是类似examples\deephpms\outputs_schrodinger\2025-06-21\09-53-55\DATASET_PATH=\datasets\NLS.mat的,按照要求修改,改成了类似examples\deephpms\outputs_schrodinger\2025-06-21\09-53-55\的
|
|
我这里检查没有问题 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
验证跑通
|
PR types
PR changes
Describe