Skip to content

Refactor: Remove the global dependence of all remained functions in DeePKS. #5835

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

Merged
merged 18 commits into from
Jan 9, 2025

Conversation

ErjieWu
Copy link
Collaborator

@ErjieWu ErjieWu commented Jan 8, 2025

What's changed?

  • Remove the global dependence of functions related to pdm, phialpha, gedm, gevdm in DeePKS.
  • Move some precalc variables for DeePKS output from FORCE_STRESS to LCAO_deepks_interface.
  • Simplify some functions for saving data into .npy file in DeePKS.

@mohanchen mohanchen added The Absolute Zero Reduce the "entropy" of the code to 0 DeePKS Issues related to the DeePKS labels Jan 9, 2025
@ErjieWu ErjieWu merged commit 24abddd into deepmodeling:develop Jan 9, 2025
14 checks passed
@ErjieWu ErjieWu deleted the refactor branch January 9, 2025 14:53
Fisherd99 pushed a commit to Fisherd99/abacus-BSE that referenced this pull request Mar 31, 2025
…eePKS. (deepmodeling#5835)

* Remove global dependence of cal_gevdm and rearrange the calling order for simplifying.

* Move some checks from FORCE_STRESS to LCAO_Deepks_interface.

* Remove the global dependence of cal_e_delta_band.

* Move cal_gedm to deepks_basic.cpp

* Remove the global dependence of functions related to pdm in DeePKS.

* Revert "Remove the global dependence of functions related to pdm in DeePKS."

This reverts commit 7a97a95.

* Remove global dependence of pdm related functions in DeePKS.

* Fix the compile bug of DeePKS UT test.

* Remove the global dependence of functions related to phialpha in DeePKS.

* Simplify some function for LCAO_deepks_io.

* Update FORCE_STRESS.cpp

* [pre-commit.ci lite] apply automatic fixes

* Update esolver_ks_lcao.cpp

* Update LCAO_deepks_interface.cpp

---------

Co-authored-by: pre-commit-ci-lite[bot] <117423508+pre-commit-ci-lite[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DeePKS Issues related to the DeePKS The Absolute Zero Reduce the "entropy" of the code to 0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants