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
SAPT(DFT) Timers & Misc. #1083
SAPT(DFT) Timers & Misc. #1083
Conversation
Overall looks good. Can you provide the output of a |
I have attached it at the bottom. It is for S22-1 and aug-cc-pVDZ. |
For those that would like to see it
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Was there a list of inconsistent tags btwn the various SAPT timers that you wanted consultation on, @yxie326?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This info will be super helpful. Thanks!
@loriab Here are the tags for SAPT0 and SAPT(DFT). I have modified (in this PR) the tags of SAPT(DFT) so that they match the style of FISAPT. I have not modified any of the tags in SAPT0 and many-body SAPT; those are having similar styles (i.e. doesn't have module name in the beginning).
|
Thanks, @yxie326. Main differences are (1) no I'll add my approval. Any adding of |
Description
Adds timers for SAPT(DFT) procedure to track bottle-necks for improving performance. Information printed into
timer.dat
. A few other minor modifications are also done (see below).Todos
Notable points that this PR has either accomplished or will accomplish.
sapt_proc.py
.timer.dat
to avoid overflowing.Checklist
Status
Attachment
timer_dat.txt
Seems that github doesn't like .dat files so I renamed it.