-
Notifications
You must be signed in to change notification settings - Fork 526
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
[Feature Request] Support eval_descriptor
for PT DeepEval
#4112
Comments
I implemented this function some time ago by copying all the |
github-project-automation
bot
moved this to Todo
in Multiple backend support for DeePMD-kit
Sep 26, 2024
|
njzjz
added a commit
to njzjz/deepmd-kit
that referenced
this issue
Oct 14, 2024
Fix deepmodeling#4112. Signed-off-by: Jinzhe Zeng <[email protected]>
github-merge-queue bot
pushed a commit
that referenced
this issue
Oct 15, 2024
Fix #4112. <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced a method for evaluating descriptors using the deep potential model. - Added functionality to control descriptor evaluation during model operations. - **Bug Fixes** - Removed conditional skip for descriptor evaluation tests, enhancing test coverage for PyTorch models. - **Tests** - Added a new test class for neighbor list setups in descriptor evaluation. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Signed-off-by: Jinzhe Zeng <[email protected]>
github-project-automation
bot
moved this from Todo
to Done
in Multiple backend support for DeePMD-kit
Oct 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Summary
Support
eval_descriptor
for PTDeepEval
.Detailed Description
The abstract method is here:
deepmd-kit/deepmd/infer/deep_eval.py
Lines 172 to 222 in d2b6243
However, how to do it for PT needs to be clarified. PT descriptor has a different
forward
method.Further Information, Files, and Links
No response
The text was updated successfully, but these errors were encountered: