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

[Thermo] Merge md() and molecular_dynamics() methods #35

Closed
leeping opened this issue Feb 9, 2014 · 2 comments
Closed

[Thermo] Merge md() and molecular_dynamics() methods #35

leeping opened this issue Feb 9, 2014 · 2 comments

Comments

@leeping
Copy link
Owner

leeping commented Feb 9, 2014

Currently the molecular_dynamics() method (implemented by me) returns a few specific pieces of information, such as the time series of energies, densities and so on. This wasn't general enough for the Thermo target so md() was added by Erik. At some point I would like to merge the two, and create a more general method for running molecular dynamics simulations + returning properties.

@leeping
Copy link
Owner Author

leeping commented Feb 9, 2014

Also mostly duplicated: gmxprefix.bash and rungmx.sh

@leeping
Copy link
Owner Author

leeping commented Feb 9, 2014

Here's a bit from an email thread, to remind myself:

  • Introduced a new method md() in gmxio.py. It is basically the
    same as molecular_dynamics() but does not do the analysis, because
    this will be different for the different quantities.

Cool. This could possibly replace molecular_dynamics() at some
point - I was not completely happy with how it was doing the analysis
since it returns a set of chosen quantities. Related to item (2)
above, it might be helpful for the engine to have methods that
retrieve the time series such as mdtraj_potentials(), mdtraj_rhos()
or something.

@leeping leeping closed this as completed Feb 20, 2014
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

No branches or pull requests

1 participant