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

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

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
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