Skip to content

Conversation

@A-006
Copy link
Collaborator

@A-006 A-006 commented Mar 5, 2025

Linked Issue

Fix #3720

What's changed?

  • To optimize the computation, we adopted an addition-based approach instead of using division and square root operations. The cell numbers are pre-computed before executing the main function to enhance performance.
  • Thanks to @dzzz2001, we integrated OpenMP to leverage parallel computing in our algorithm. This implementation demonstrates strong scaling, indicating significant performance improvements with increased computational resources.

Here is the time table of the perf and openmp
图片

@mohanchen mohanchen added Performance Issues related to fail running ABACUS Refactor Refactor ABACUS codes labels Mar 6, 2025
@mohanchen mohanchen merged commit e93d3a7 into deepmodeling:develop Mar 7, 2025
14 checks passed
Fisherd99 pushed a commit to Fisherd99/abacus-BSE that referenced this pull request Mar 31, 2025
* first version

* change add

* add the change

* change

* change gtest

* add change

* change the file

* add format

* modify back test

* add the change in the check_atom_stru

* move openmp

* add globlv for the question

* delete vector

* update bug
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Performance Issues related to fail running ABACUS Refactor Refactor ABACUS codes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bad performance of "check_structure()" for large systems

2 participants