Skip to content

LAMMPS engine updates#8

Open
dwhswenson wants to merge 1034 commits into
jhprinz:lammps-py3from
dwhswenson:jhp_lammps
Open

LAMMPS engine updates#8
dwhswenson wants to merge 1034 commits into
jhprinz:lammps-py3from
dwhswenson:jhp_lammps

Conversation

@dwhswenson

Copy link
Copy Markdown

This PR is a sub-PR of openpathsampling#697. If you merge this now, it will become part of that PR (not part of the OPS core).

I'm giving a talk about the OPS/LAMMPS interface on Tuesday, so I thought I should learn how to use it. I put together a little TPS demo -- LJ phase transition from gas to droplet, which was part of a the PRL on nested TPS that came out this week (Peter and Gabor Csanyi). I'll add that example sometime soon, too.

Updates:

  • corrected renames of configurations to statics and momentum to kinetics
  • modifed the LAMMPS engine so it can (optionally) take a topology, which makes it compatible with MDTraj (so we can use MDTraj for CVs, or as a gateway to NGLView, etc.)
  • Added a LAMMPSComputeCV class that allows us to wrap a LAMMPS compute statement in an OPS CV.

Still to do: write unit tests and docstrings for these updates. But that might be a while. Your choice on whether you want to merge this in as part of your main PR or wait on it until the tests are in place.

dwhswenson and others added 30 commits January 23, 2020 18:25
Still need to complete the analysis cleanup.
…til_decorrelated

`run_until_decorrelated` should account for time reversibility
…alysis

Fix pandas API change: set_value -> at
…omvelocity

Refactor RandomVelocity so engine can do the work
Fix pandas Deprepaction and Future Warnings
…l_ensemble

Fix missing newline in the VisitAllStatesEnsemble
dwhswenson and others added 30 commits December 3, 2020 12:53
Ran tests and reloaded old files, so this shouldn't break things
np.array with no shape should return float
…lify

SimStore: Simplify OPS Storage constructor
fix numpy 1.19 DeprecationWarning: tostring()
Fix: OPS hangs when external engine errors/exits unexpectedly
…houldn't be put on master"

This reverts commit c2d47f0.
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.

7 participants