Skip to content
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

Oeproperties dev #397

Draft
wants to merge 131 commits into
base: master
Choose a base branch
from
Draft

Oeproperties dev #397

wants to merge 131 commits into from

Conversation

vtripath65
Copy link
Collaborator

No description provided.

etiennepalos and others added 30 commits November 6, 2023 11:38
… by default to QUICK output. External file support in progress.
Initial implementation  of ESP_GRID / serial version
ESP_GRID : DEFAULT, will print X, Y, Z, and ESP_TOTAL to file, all in atomic units.

ESP_GRID ESP_PRINT_TERMS : Requires ESP_GRID and ESP_PRINT_TERMS flags.
                           Prints ESP_NUC, ESP_ELEC, and ESP_TOTAL.

ESP_GRID EXTGRID_ANGSTROM : Same as ESP_GRID except X, Y, and Z are converted to Angstrom,
                            i.e. as provided in input file.
Merge master into esp_development
Removed allmod, implicit definitions and made the module
definition only.
I previously commented out MPI commands to enable testing of
ESP OEI's. Now uncommented part of those lines which does not
break the code.
Now ESP calculation runs (serially) in
MPI code, i.e., SCF eveluation is MPI
parallelized.
New default branch: 
ESP works in the MPI code. SCF is MPI parallelized, ESP part is not yet.
Additionally, cleaned up the oeproperties module by removing allmods and implicit
variable definitions.
We changed the order of calculation by
considering all the points for any shell
pair before moving to the next. This
reduced the overhead due to the atomic
add operations.
If the coordinates are being read from data file,
the input file does not need to have coordinates.
The facility to read external charges is not yet
implemented.
Merging master branch after updates to the Github actions and tests.
@agoetz agoetz marked this pull request as draft February 20, 2025 03:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants