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

[Feature Request] _Integrate Grimme DFTD-3 or DFTD-4 into DeepMD-kit #3099

Closed
Epikurz opened this issue Dec 31, 2023 · 1 comment · Fixed by #3101
Closed

[Feature Request] _Integrate Grimme DFTD-3 or DFTD-4 into DeepMD-kit #3099

Epikurz opened this issue Dec 31, 2023 · 1 comment · Fixed by #3101

Comments

@Epikurz
Copy link

Epikurz commented Dec 31, 2023

Summary

I would like to propose an enhancement for future versions of DeepMD-Kit. It would be highly beneficial if the software could integrate Grimme-D3 or D4 corrections directly into the potential energy functions trained with the PBE functional. This integration would eliminate the need to train potentials with van der Waals forces explicitly included.

From a theoretical standpoint, potentials without van der Waals forces could potentially utilize a smaller Rcut, offering computational advantages. I believe this enhancement would greatly improve the versatility and efficiency of DeepMD-Kit, particularly for users interested in training potentials using the PBE functional with van der Waals corrections.

Detailed Description

Current versions allow for the simultaneous use of DeepMD-kit and DFTD3 as calculators through ASE to achieve certain functionalities. However, when running LAMMPS, there is currently no straightforward method to perform D3 corrections concurrently with MD simulations. I would like to propose the capability to easily incorporate van der Waals force corrections when using LAMMPS for DeepPotential simulations. I believe this approach would be more convenient and accurate than training potential functions using DFT data pre-corrected with D3 corrections.

Further Information, Files, and Links

No response

@njzjz
Copy link
Member

njzjz commented Dec 31, 2023

I notice a recent work did the similar things: https://doing.org/10.1088/1361-648X/ad1278 The cutoff radius was added for simulations, but it's unclear whether there was a switch function.

I personally think it should be implemented into LAMMPS.

For the DeePMD-kit, the fastest way might be reimplementing the pairwise tabulate. So, the DeePMD-kit does not need to know the details about external potentials.

@njzjz njzjz linked a pull request Jan 2, 2024 that will close this issue
wanghan-iapcm pushed a commit that referenced this issue Jan 3, 2024
Add pairwise tabulation as an independent model, which can be summed
with DP (DP + PairTab) by the linear model, other than interpolation.
PairTab can be used for any pairwise potentials, e.g., d3, LJ, ZBL, etc.

Fix #3099.

---------

Signed-off-by: Jinzhe Zeng <[email protected]>
@njzjz njzjz closed this as completed Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants