Skip to content

Refactor: split timer::tick() to timer::start() and timer::end() - #7020

Merged
mohanchen merged 13 commits into
deepmodeling:developfrom
linpeize:timer_split
Mar 22, 2026
Merged

Refactor: split timer::tick() to timer::start() and timer::end()#7020
mohanchen merged 13 commits into
deepmodeling:developfrom
linpeize:timer_split

Conversation

@linpeize

@linpeize linpeize commented Mar 12, 2026

Copy link
Copy Markdown
  1. Refactor: split timer::tick() to timer::start() and timer::end()
  2. fix bugs of timer in functions

This will be beneficial in avoiding errors in time statistics

2. fix bug of timer in ORB_gaunt_table::Calc_Gaunt_CH()
@linpeize linpeize changed the title Refactor: split timer::tick() to timer::start() and timer::end() Refactor: split timer::tick() to timer::start() and timer::end() Mar 12, 2026
@linpeize
linpeize force-pushed the timer_split branch 2 times, most recently from 7762865 to 59bf1ca Compare March 12, 2026 16:54
@mohanchen mohanchen added Refactor Refactor ABACUS codes Feature Discussed The features will be discussed first but will not be implemented soon labels Mar 16, 2026
@linpeize
linpeize force-pushed the timer_split branch 2 times, most recently from 6ea8b2f to 20db7a1 Compare March 17, 2026 20:35
Comment thread source/source_lcao/module_ri/LRI_CV.hpp
Comment thread source/source_lcao/module_ri/Matrix_Orbs11.cpp
Comment thread source/source_lcao/module_ri/Matrix_Orbs21.cpp
Comment thread source/source_lcao/module_ri/Matrix_Orbs22.cpp
Comment thread source/source_lcao/module_ri/Matrix_Orbs11.cpp
Comment thread source/source_estate/module_pot/pot_xc_fdm.cpp

@mohanchen mohanchen left a comment

Copy link
Copy Markdown
Collaborator

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 370e67c into deepmodeling:develop Mar 22, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Discussed The features will be discussed first but will not be implemented soon Refactor Refactor ABACUS codes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants