Skip to content

Some small updates, including timer, and eliminate clean function in esolver#6892

Merged
mohanchen merged 3 commits intodeepmodeling:developfrom
mohanchen:develop
Jan 24, 2026
Merged

Some small updates, including timer, and eliminate clean function in esolver#6892
mohanchen merged 3 commits intodeepmodeling:developfrom
mohanchen:develop

Conversation

@mohanchen
Copy link
Collaborator

Some small updates, including timer, and eliminate clean function in esolver

abacus_fixer added 3 commits January 23, 2026 20:25
1. Remove #ifdef __MPI from timer code, encapsulate in timer_wrapper.h
2. Move ESolver clean logic to after_all_runners method
3. Replace clean_esolver calls with direct delete p_esolver
4. Remove #ifdef __MPI from delete p_esolver
5. Add Cblacs_exit(1) in after_all_runners for LCAO calculations
@mohanchen mohanchen added Refactor Refactor ABACUS codes The Absolute Zero Reduce the "entropy" of the code to 0 labels Jan 24, 2026
Copy link
Collaborator

@AsTonyshment AsTonyshment left a comment

Choose a reason for hiding this comment

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

LGTM.

@mohanchen mohanchen merged commit ecb602e into deepmodeling:develop Jan 24, 2026
14 checks passed
Flying-dragon-boxing pushed a commit to Flying-dragon-boxing/abacus-develop that referenced this pull request Jan 26, 2026
…esolver (deepmodeling#6892)

* Refactor: Encapsulate timer functionality in timer_wrapper.h

* Refactor timer code and clean_esolver function

1. Remove #ifdef __MPI from timer code, encapsulate in timer_wrapper.h
2. Move ESolver clean logic to after_all_runners method
3. Replace clean_esolver calls with direct delete p_esolver
4. Remove #ifdef __MPI from delete p_esolver
5. Add Cblacs_exit(1) in after_all_runners for LCAO calculations

---------

Co-authored-by: abacus_fixer <mohanchen@pku.eud.cn>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

2 participants