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

QCEngine/NWChem interface minor update #446

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kaka-zuumi
Copy link

@kaka-zuumi kaka-zuumi commented Apr 10, 2024

Description

A minor update to make it so that the keywords "noautosym" and "nocenter" can be passed to NWChem

Changelog description

Added a few lines in "build_input" in "runner.py" which can take some input keywords and change the NWChem input file to add them.

Status

  • Code base linted
  • Ready to go

I've tested these changes on a personal installation of QCEngine/NWChem (for preparation for NWChemEx...)

Added some lines in "runner.py" so that the "noautosym" and "nocenter" keywords can be passed from compute(...) down to NWChem
@loriab
Copy link
Collaborator

loriab commented Apr 13, 2024

Have you tried nocom noreorient symmetry c1 if you're building the molecule from string https://github.com/MolSSI/QCEngine/blob/master/qcengine/programs/tests/test_ghost.py#L20-L21 . Or fix_com=True, fix_orientation=True, symmetry=c1 https://github.com/MolSSI/QCElemental/blob/master/qcelemental/models/molecule.py#L263-L286 if building molecule from schema?

I think that should do what you intend. qcng is supposed to return program native orientation if fix_com/_orient=F or return in AtomicInput input Cartesian orientation if fix_com/_orient=T.

@loriab
Copy link
Collaborator

loriab commented May 2, 2024

Did the alternate way of getting gradients back in input orientation end up working for you?

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.

None yet

2 participants