-
Notifications
You must be signed in to change notification settings - Fork 582
pd: support paddle backend and water/se_e2_a
#4302
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
pd: support paddle backend and water/se_e2_a
#4302
Conversation
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.
CodeQL found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.
📝 WalkthroughWalkthroughThe pull request introduces significant updates across multiple files to enhance the integration of PaddlePaddle within the DeepMD framework. Key changes include updates to workflow configurations for testing, the addition of new classes and methods for Paddle-specific functionalities, and enhancements to existing tests to accommodate Paddle. The updates also include new JSON configuration files for models, improvements in error handling, and the introduction of new utility functions, all aimed at expanding the framework's capabilities and ensuring compatibility with PaddlePaddle. Changes
Sequence Diagram(s)sequenceDiagram
participant CI as CI Workflow
participant Test as Test Runner
participant Paddle as Paddle Backend
participant Model as DeepMD Model
CI->>Test: Trigger Tests
Test->>Paddle: Check Paddle Installation
alt Paddle Installed
Test->>Model: Load Model
Model->>Paddle: Run Model Evaluation
Paddle-->>Model: Return Results
Test-->>CI: Report Success
else Paddle Not Installed
Test-->>CI: Report Skipped Tests
end
Possibly related PRs
Suggested labels
Suggested reviewers
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
|
Hello @njzjz , I have a question about the naming of the functions |
water/se_e2_a
Do you mean deepmd-kit/deepmd/backend/backend.py Lines 179 to 201 in 38815b3
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## devel #4302 +/- ##
==========================================
- Coverage 84.50% 83.19% -1.31%
==========================================
Files 596 649 +53
Lines 56665 60967 +4302
Branches 3459 3461 +2
==========================================
+ Hits 47884 50721 +2837
- Misses 7654 9118 +1464
- Partials 1127 1128 +1 ☔ View full report in Codecov by Sentry. |
…y to coverage newly added code
About 2000+ lines have yet to be tested in the CI. Could you take a look at the coverage report? |
Thanks for the comments, redundant files&codes were removed. |
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.
Copilot reviewed 121 out of 136 changed files in this pull request and generated no suggestions.
Files not reviewed (15)
- deepmd/pd/infer/init.py: Evaluated as low risk
- deepmd/pd/entrypoints/init.py: Evaluated as low risk
- deepmd/dpmodel/model/make_model.py: Evaluated as low risk
- deepmd/main.py: Evaluated as low risk
- .github/workflows/test_python.yml: Evaluated as low risk
- deepmd/pd/model/init.py: Evaluated as low risk
- .github/workflows/test_cuda.yml: Evaluated as low risk
- deepmd/pd/model/atomic_model/dp_atomic_model.py: Evaluated as low risk
- deepmd/pd/model/atomic_model/energy_atomic_model.py: Evaluated as low risk
- deepmd/pd/model/descriptor/env_mat.py: Evaluated as low risk
- deepmd/pd/model/descriptor/base_descriptor.py: Evaluated as low risk
- deepmd/pd/loss/loss.py: Evaluated as low risk
- deepmd/pd/model/descriptor/descriptor.py: Evaluated as low risk
- deepmd/pd/model/descriptor/init.py: Evaluated as low risk
- deepmd/pd/model/atomic_model/init.py: Evaluated as low risk
|
@coderabbitai resolve |
|
@coderabbitai resolve |
... It seems to be not working. The comments may be required to be manually resolved |
|
@njzjz: Sorry that the resolve command didn't work earlier. I have now resolved the comments for you. ✅ Actions performedComments resolved. |
|
@HydrogenSulfate A segfault appears in the GPU test. https://github.com/deepmodeling/deepmd-kit/actions/runs/12005580706/job/33462440790 |
Thanks for report, I will check it out |
I have make a new environment with cuda11.8 and installed the nightly build paddle( nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2022 NVIDIA Corporation
Built on Wed_Sep_21_10:33:58_PDT_2022
Cuda compilation tools, release 11.8, V11.8.89
Build cuda_11.8.r11.8/compiler.31833905_0It seems a weird issue and is there any way to get the detailed error log from Test CUDA? |
It's run in a docker container, so I think it may be reproducible. Btw, the image is nvidia/cuda:12.6.2-cudnn-devel-ubuntu22.04 |
|
A bit strange, but the segfault disappeared. |
If we can add |

Split #4157 into several pull requests.
deepmd.pd.*) and related backend module unitests.Related PR to be merged:
Accuracy test
pytorch
paddle:
Summary by CodeRabbit
Release Notes
New Features
Bug Fixes
Tests
Documentation