Skip to content

Commit

Permalink
Merge pull request #55 from molssi-seamm/dev
Browse files Browse the repository at this point in the history
Fixed Docker build.
  • Loading branch information
seamm committed Jun 28, 2024
2 parents fd86235 + 900db1d commit 8482ec6
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 12 deletions.
4 changes: 3 additions & 1 deletion HISTORY.rst
Original file line number Diff line number Diff line change
@@ -1,6 +1,8 @@
=======
History
=======
2024.6.28.1 -- Internal release to fix issue making Docker image.

2024.6.28 -- Added energy and forces to properties
* Added ability to get the energy and forces from single point calculations to supprt
e.g. energy scans.
Expand All @@ -9,7 +11,7 @@ History
* Improved analsys based on the output file.

2024.3.22 -- Corrected issue with e.g. heat flux calculations
* Corrected an issue running LAMMPS via Python, intorduced in the new scheme for
* Corrected an issue running LAMMPS via Python, introduced in the new scheme for
executing. It ignored parallelism.

2024.3.21 -- Switched to new installation scheme
Expand Down
2 changes: 1 addition & 1 deletion devtools/docker/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
FROM molssi/mamba141

COPY ./environment.yml /root/environment.yml
COPY ./lammps_step/data/seamm-lammps.yml /root/environment.yml

RUN mamba env update -f /root/environment.yml

Expand Down
10 changes: 0 additions & 10 deletions devtools/docker/environment.yml

This file was deleted.

4 changes: 4 additions & 0 deletions lammps_step/npt.py
Original file line number Diff line number Diff line change
Expand Up @@ -146,6 +146,10 @@ def get_input(self, extras=None):

self.description = []

_, configuration = self.get_system_configuration()
if configuration.periodicity == 0:
raise RuntimeError("Cannot run NPT dynamics on non-periodic system!")

P = self.parameters.current_values_to_dict(
context=seamm.flowchart_variables._data
)
Expand Down

0 comments on commit 8482ec6

Please sign in to comment.