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

[PaddlePaddle Hackathon 2] 94. Add Paddle as a new backend of DeepXDE #562

Merged
merged 53 commits into from
May 20, 2022
Merged

[PaddlePaddle Hackathon 2] 94. Add Paddle as a new backend of DeepXDE #562

merged 53 commits into from
May 20, 2022

Conversation

AndPuQing
Copy link
Contributor

增加paddle作为DeepXDE的 backend

  • paddlepaddle暂时不支持L-BFGS优化器github issue
  • paddle中tensor.backward()方法不支持高阶求导,暂时不支持net.apply_feature_transform()或者net.apply_output_transform()方法

@lululxvi
Copy link
Owner

Please change the backend name "paddlepaddle" to "paddle"

@AndPuQing
Copy link
Contributor Author

Please change the backend name "paddlepaddle" to "paddle"

Done

@TCChenlong
Copy link

TCChenlong commented Mar 11, 2022

Hi, thanks for your contribution. Please submit the rfc to community first, template can refer to api_design_template. We will review the code after it passes, thanks!

@AndPuQing
Copy link
Contributor Author

Hi, thanks for your contribution. Please submit an rfc to the community first, template can refer to api_design_template. We will review the code after it passes, thanks!

ok i will wait for the rfc to pass the review

@AndPuQing AndPuQing changed the title 【PaddlePaddle Hackathon 2】94、增加paddle作为DeepXDE的 backend [PaddlePaddle Hackathon 2] 94. Add Paddle as a new backend of DeepXDE Mar 12, 2022
@lululxvi
Copy link
Owner

Please update according to the updated task description.

@AndPuQing
Copy link
Contributor Author

Please update according to the updated task description.

Paddle autograd does not support higher-order derivatives for sin and cos, I will try to fix this problem later.

@lululxvi
Copy link
Owner

OK. Lotka-Volterra equation is not required in the updated task.

deepxde/config.py Outdated Show resolved Hide resolved
deepxde/data/pde.py Outdated Show resolved Hide resolved
deepxde/gradients.py Outdated Show resolved Hide resolved
deepxde/model.py Outdated Show resolved Hide resolved
deepxde/model.py Outdated Show resolved Hide resolved
deepxde/nn/paddle/fnn.py Outdated Show resolved Hide resolved
docs/index.rst Outdated Show resolved Hide resolved
@@ -10,6 +10,7 @@ DeepXDE requires one of the following backend-specific dependencies to be instal
- TensorFlow 2.x: `TensorFlow <https://www.tensorflow.org/>`_>=2.2.0, `TensorFlow Probability <https://www.tensorflow.org/probability>`_>=0.10.0
- PyTorch: `PyTorch <https://pytorch.org/>`_>=1.9.0
- JAX: `JAX <https://jax.readthedocs.io>`_, `Flax <https://flax.readthedocs.io>`_, `Optax <https://optax.readthedocs.io>`_
- PaddlePaddle: `PaddlePaddle <https://www.paddlepaddle.org.cn>`_
Copy link
Owner

Choose a reason for hiding this comment

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

Is there a minimal version of paddle?

Copy link
Owner

Choose a reason for hiding this comment

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

Copy link
Owner

Choose a reason for hiding this comment

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

Should we use at least version 2.3.0?

deepxde/callbacks.py Outdated Show resolved Hide resolved
deepxde/config.py Outdated Show resolved Hide resolved
deepxde/model.py Outdated Show resolved Hide resolved
@lululxvi
Copy link
Owner

@AndPuQing There are still some issues above unresolved.

@AndPuQing
Copy link
Contributor Author

* 使用动态图的paddle.grad接口, 并使用paddle.jit.to_static将动态图转换成静态图

@AndPuQing Any plan on this?

The support for static graph, met with many difficulties. For our task, do not need to support, in the short term will not support

@lululxvi
Copy link
Owner

* 使用动态图的paddle.grad接口, 并使用paddle.jit.to_static将动态图转换成静态图

@AndPuQing Any plan on this?

The support for static graph, met with many difficulties. For our task, do not need to support, in the short term will not support

OK

@lululxvi lululxvi merged commit 770e7c1 into lululxvi:master May 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants