-
Notifications
You must be signed in to change notification settings - Fork 46
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
Improve gradient printout #283
Comments
I can work on this one. |
Thanks @akhilshajan . @Madu86 please let Akhil know what else we need to keep in mind when changing output format (your test scripts, QUICK tests reference output, Amber tests reference output etc). Thanks. |
Good suggestion but not sure if we want to do this as we approach a release. You will have to change runtest script, reference test output files and file based QM/MM interface in AMBER. @agoetz Can you please confirm the last one? Is this quick_module.f90? Separate reference output files are no longer available for AMBER tests. |
This is a good point to avoid large changes right before a release. On the other hand more people are starting to use QUICK and it would be good to make changes in output format sooner than later to avoid breaking any interfaces people may write. |
The file based QM/MM interface in AMBER is in
|
Currently the gradient is printed in a single column and there is a typo, VARIABLES is missing an A.
I suggest to print it by atom. Renaming to ATOM will also fix the typo.
I can fix it but I don't know when I find time. It might required updating tests and/or test scripts. If someone works on this please comment and mention others so we don't have two people working on it.
The text was updated successfully, but these errors were encountered: