Skip to content

Conversation

@jwaldrop107
Copy link
Member

Description
Adds a module to interface to NWChem for gradient calculations via the MolSSI stack, along with tests and related refactoring.

@jwaldrop107 jwaldrop107 self-assigned this Apr 23, 2024
@CLAassistant
Copy link

CLAassistant commented Apr 26, 2024

CLA assistant check
All committers have signed the CLA.

@ryanmrichard
Copy link
Member

@jwaldrop107 is this waiting on anything or is it ready to merge?

@jwaldrop107
Copy link
Member Author

@ryanmrichard The NWChemEnergyAndGradientViaMolSSI module is a little hacked together at the moment. It's supposed to satisfy the EnergyNuclearGradientStdVectorD property type, but is appending the energy onto the end of the return value. This would be okay to merge if we removed that module, but I believe @kaka-zuumi was using that on his side.

@ryanmrichard
Copy link
Member

@jwaldrop107 I see. Thanks for the clarification. @jlheflin that means I lied to you with the changes I suggested to StructureFinder; undo them. You want to be using the 'NWChem : SCF Gradient' module for the gradient and the 'NWChem : SCF' module for the energy (at present this will run the energy calculation twice, so don't explore with too large of systems).

@ryanmrichard ryanmrichard mentioned this pull request Sep 26, 2024
1 task
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.

5 participants