-
Notifications
You must be signed in to change notification settings - Fork 5
Description
Hi @srmnitc,
I saw that you stared the repository, so I wanted to give you access. The reason why I removed public access is that some functions do not work at the moment, like extract_atom. Luckily the functions we need inside pyiron, work out of the box https://github.com/pyiron/pyiron/blob/master/pyiron/lammps/interactive.py#L606 .
The general idea is to provide a single core lammps process - running in a jupyter notebook - which then controls a mpi parallel lammps process. Currently this implementation only works with openmpi - figuring this out took me quite some time pyiron/pyiron#516 .
As this scheme is potentially interesting for users of lammps beyond the pyiron community I would be interested to make this a public package. If we get the standard commands working I am pretty sure we can also get Steve Plimpton - the core developer of lammps to implement it in the core lammps code. I met Steve last year when I visited Los Alamos.
So if you are interested to work on this feel free to continue the project, the Lammps developers are always happy about contributions and being named in the Lammps documentation helps to get more people interested in your work - that is what I learned from contributing the Lammps conda forge package https://github.com/lammps/lammps/blob/master/doc/src/Install_conda.rst .
Best regards,
Jan