Skip to content

Conversation

@DrRyanHuang
Copy link
Contributor

PR types

Others

PR changes

Others

Describe

  1. examples/shock_wave.md

@HydrogenSulfate
Copy link
Collaborator

提交代码之前请安装pre-commit,否则code style CI无法通过

pip install pre-commit
pre-commit install

@luotao1 luotao1 added the HappyOpenSource 快乐开源活动issue与PR label Nov 7, 2023
@HydrogenSulfate
Copy link
Collaborator

修改之前可以先把develop分支合并一下,解决冲突

@DrRyanHuang
Copy link
Contributor Author

yamlpy 文件修改无误之后,我再更新 shock_wave.md 😄😄

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.

这个改完可以开始改文档了,预训练模型地址如下:

https://paddle-org.bj.bcebos.com/paddlescience/models/shockwave/shock_wave_Ma2_pretrained.pdparams
https://paddle-org.bj.bcebos.com/paddlescience/models/shockwave/shock_wave_Ma0728_pretrained.pdparams

文档修改的时候参考这个文档的样式,使用卡片语法区分两个案例的训练、预测代码
image

@HydrogenSulfate
Copy link
Collaborator

HydrogenSulfate commented Nov 15, 2023

另一个实验的训练、预测命令可以通过python shock_wave.py -cn=shock_wave_Ma0.728里的-cn(config name)参数重新指定读取的配置文件

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 110135c into PaddlePaddle:develop Nov 15, 2023
@DrRyanHuang DrRyanHuang deleted the 1106 branch November 15, 2023 07:20
huohuohuohuohuo123 pushed a commit to huohuohuohuohuo123/PaddleScience that referenced this pull request Aug 12, 2024
* add shock_wave

* add py yaml

* add md

* need evaluate

* update evaluate

* fix code style

* change seed && add some config

* True => cfg.EVAL.eval_with_no_grad

* update md && rm .eval

* change line num

* update yml name

* add another config file

* change format

* rm eval download code

* fix doc
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