-
Notifications
You must be signed in to change notification settings - Fork 147
Update C-molecule performance test walker counts. #4112
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
Conversation
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.
Thanks for adding batched_driver variants.
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.
Need non-ANL approval.
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.
Thank you. Looks reasonable, but have not made a test run. Check the nightlies after merge.
Test this please |
Proposed changes
This PR is a part of #4081 .
Update C-molecule performance test walker counts to more detailed performance check for the real simulation.
C-graphite performance test update will be followed with expanding its system size for the test. (Current graphite performance test has only one system size to check.)
What type(s) of changes does this code introduce?
Does this introduce a breaking change?
What systems has this change been tested on?
Intel Xeon
Checklist
Update the following with a yes where the items apply. If you're unsure about any of them, don't hesitate to ask. This is
simply a reminder of what we are going to look for before merging your code.