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

[WIP][Add]Add forward analysis of control arm example #642

Merged

HydrogenSulfate merged 6 commits into PaddlePaddle:develop from lijialin03:control_arm

Nov 29, 2023

Contributor

lijialin03 commented Nov 15, 2023 •

edited

Loading

PR types

Others

PR changes

Others

Describe

add forward analysis of control arm example
add freeze and unfreeze in arch.Arch

HydrogenSulfate changed the title ~~[Add]Add forward analysis of control arm example~~ [WIP][Add]Add forward analysis of control arm example

lijialin03 force-pushed the control_arm branch from 76132ce to ea14de0 Compare

November 17, 2023 10:41

lijialin03 force-pushed the control_arm branch 2 times, most recently from f7225d9 to f90c3f6 Compare

November 27, 2023 03:30

lijialin03 added 4 commits

November 28, 2023 10:31


          Add forward analysis of control arm example

13be1f3


          use x-axis result as default

7502bc2


          add it in index and README

d882e06


          add inverse part

e41595c

lijialin03 force-pushed the control_arm branch from 7cb8e5e to e41595c Compare

November 28, 2023 10:33

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/forward_analysis.py Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/inverse_parameter.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

examples/control_arm/inverse_parameter.py Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate reviewed

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

HydrogenSulfate requested changes

View reviewed changes

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved

Collaborator

HydrogenSulfate commented Nov 28, 2023 •

edited

Loading

给 arch.Arch 加一下 freeze 和 unfreeze 方法（参考https://github.com/Lightning-AI/lightning/blob/197b22586a449fd24d6835c2b186d7900de05c93/src/lightning/pytorch/core/module.py#L1317-L1343），并用 model.freeze() 在逆问题中冻结掉另外两个模型，减少梯度反传时不必要的时间和显存开销。


          update 1

e7d48ce

HydrogenSulfate requested changes

View reviewed changes

ppsci/arch/base.py Outdated Show resolved Hide resolved

docs/zh/examples/control_arm.md Outdated Show resolved Hide resolved


          update 2

1e43a61

HydrogenSulfate approved these changes

View reviewed changes

Collaborator

HydrogenSulfate left a comment

LGTM

HydrogenSulfate merged commit d021faa into PaddlePaddle:develop

3 checks passed

huohuohuohuohuo123 pushed a commit to huohuohuohuohuo123/PaddleScience that referenced this pull request


          [WIP][Add]Add forward analysis of control arm example (PaddlePaddle#642)

b1cc770

* Add forward analysis of control arm example

* use x-axis result as default

* add it in index and README

* add inverse part

* update 1

* update 2

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet