-
Notifications
You must be signed in to change notification settings - Fork 26
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
external calculator interface #13
Comments
For the ASE calculator I think it should be very easy because it is python. A good example to look at and model the ASE "lot" is For example, in the recent commits the only function that needs to be programmed is the I'm not familiar with how ASE works but it should be straightforward to write this function. Let me know how it goes. |
A general ASE Lot (taking an ASE calculator as argument) would be very useful. @stenczelt can you share your solution? |
I have written a minimal initial version, now I need to test it. Ideally, I would like to test with some dummy interatomic potential that is in ASE and see how it works, then with ORCA for example that is implemented in both this package and that one. |
Particularly I would like to write a wrapper that takes molecules as ASE atoms objects and performs double-ended or single-ended GSM with them, then returns an ASE-readable result object. Ie. if I or my colleagues are using this package, then we really only import that and pass it the objects we have from other parts of the workflow, or run a bash script with xyz files that pops out the results in xyz /+ txt info. Where should I place this in the package tree? |
Dear PyGSM developers,
I was wondering if you can guide me in how to implement an external calculator interface for this package. I am normally using ASE and calculators there or derived from their code.
So far I understood that for a calculator, I need to derive a class from
base_lot.Lot
and calculate the energy and gradients (forces) from the positions given. Though I am not entirely sure form the code how multiplicity and state should be handled, particularly if I am not worried about the multiplicity at all.With a bit of guidance I would be happy to try this and use an ASE calculator here, then merge the interface code in here if you want to keep it.
The text was updated successfully, but these errors were encountered: