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

RI-HFX for K-points (with gradients and ADMM) #2998

Merged
merged 1 commit into from
Sep 19, 2023
Merged

Conversation

abussy
Copy link
Contributor

@abussy abussy commented Sep 18, 2023

This PR introduces a new method for RI-HFX with K-point sampling. The exact exchange matrix is calculated in real-space for all interacting periodic images, before it is Fourier transformed to reciprocal space. Then, the complex Hermitian KS matrix is built, and the eigenvalue problem solved for each K-point. Details of the method will be published in a paper in the near future.

The method relies on the DBT library for sparse-tensor algebra, and on the libint library for analytical 3- and 2-center ERI evaluation. Forces and stress tensor calculations are also implemented. The method can be sped-up with the ADMM approximation (ADMM2, ADMMP, ADMMQ and ADMMS).

Careful testing was conducted. Equivalence to Gamma-point supercell calculations was verified, as well as convergence with the number of k-points. DEBUG runs were systematically ran for forces and stress tensors. Four new regtest directories were added, covering KP-RI-HFX energy and forces, with and without ADMM.

@dev-zero was instrumental in helping develop the ADMM approximation with k-point sampling.

@abussy abussy force-pushed the kp_ri_hfx branch 2 times, most recently from 216c22d to b8e5439 Compare September 19, 2023 09:18
IF (dft_control%do_admm) &
CPABORT("No ADMM implemented with kpoints")
!IF (dft_control%do_admm) &
! CPABORT("No ADMM implemented with kpoints")
Copy link
Contributor

Choose a reason for hiding this comment

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

maybe drop completely now :-)

Co-authored-by: Tiziano Müller <tiziano.mueller@chem.uzh.ch>
@fstein93
Copy link
Contributor

Closes #2913

@dev-zero
Copy link
Contributor

@fstein93 fyi, the canonical HFXk reference implementation will come with a second PR from my side

@abussy abussy merged commit e26bbca into cp2k:master Sep 19, 2023
34 checks passed
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.

None yet

3 participants