abacus-develop icon indicating copy to clipboard operation
abacus-develop copied to clipboard

Test: supplementary use of Memory::record to record memory

Open Religious-J opened this issue 1 year ago • 1 comments

Reminder

  • [x] Have you linked an issue with this pull request?
  • [ ] Have you added adequate unit tests and/or case tests for your pull request?
  • [x] Have you noticed possible changes of behavior below or in the linked issue?
  • [ ] Have you explained the changes of codes in core modules of ESolver, HSolver, ElecState, Hamilt, Operator or Psi? (ignore if not applicable)

Linked Issue

#3657 Milestone 1

Unit Tests and/or Case Tests for my changes

No

What's changed?

In response to issue #3657, the peak memory application recorded using ModuleBase::Memory::record is quite different from that recorded using ModuleBase::TITLE. Therefore, after preliminary statistical analysis of memory application during program running, supplement the key functions of missing records in ModuleBase::Memory::record method. (such as Symmetry::rhog_symmetry() , XC_Functional::gradcorr(), Charge::allocate() ...)

Religious-J avatar May 08 '24 15:05 Religious-J

Good for your first PR, please modify with comments.

I get it, I will modify it according to the comments, thanks a lot for your review !

Religious-J avatar May 09 '24 04:05 Religious-J