Skip to content

Feature: Support outputting partial charge and wave functions under PW basis when kpar > 1 #6242

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 20 commits into from
May 31, 2025

Conversation

AsTonyshment
Copy link
Collaborator

@AsTonyshment AsTonyshment commented May 28, 2025

Linked Issue

Fix #6227

What's changed?

  • Minor documentation fix.
  • Changed the logic of Parallel_Grid::reduce, so that the operation could be done under each pool instead of pool No. zero.
  • Support outputting partial charge and wave functions under PW basis when kpar > 1.

@AsTonyshment AsTonyshment marked this pull request as draft May 28, 2025 09:10
@AsTonyshment AsTonyshment marked this pull request as ready for review May 30, 2025 08:59
@AsTonyshment AsTonyshment requested a review from mohanchen May 30, 2025 10:07
@mohanchen mohanchen added Features Needed The features are indeed needed, and developers should have sophisticated knowledge Refactor Refactor ABACUS codes The Absolute Zero Reduce the "entropy" of the code to 0 labels May 31, 2025
@mohanchen mohanchen merged commit aee5691 into deepmodeling:develop May 31, 2025
14 checks passed
@AsTonyshment AsTonyshment deleted the fix_getwf_sepk_doc branch May 31, 2025 12:35
kluophysics pushed a commit to kluophysics/abacus-develop that referenced this pull request Jun 5, 2025
…W basis when `kpar > 1` (deepmodeling#6242)

* Fix if_separate_k description in the doc

* Use nkstot instead of nks when kpar > 1

* Fix doc: set_wf -> get_wf

* Fix kpar nks -> nkstot

* Fix write_cube and pgrid reduce logic

* Delete cout and nks -> nkstot for out_wfc_re_im

* Add parameter reduce_all_pool to control reduce behavior

* Delete useless parameters in get_pchg_pw

* Fix incorrect pchg when if_separate_k is false due to lack of reduce across diff pools

* Add integrate test 113_PW_get_pchg_kpar

* Add integrate test 114_PW_get_pchg_sepk

* Add integrate test 115_PW_get_wf_kpar

* Add MPI compilation macros

* Update 115_PW_get_wf_kpar to avoid random behavior in CI/CD test

* Change param of test 115_PW_get_wf_kpar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Features Needed The features are indeed needed, and developers should have sophisticated knowledge Refactor Refactor ABACUS codes The Absolute Zero Reduce the "entropy" of the code to 0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Partial charge density and wavefunction ouput under PW basis is incorrect when kpar is on
2 participants