MaginnGroup / Cassandra

Cassandra is a Monte Carlo package to conduct atomistic simulations.
https://cassandra.nd.edu/
GNU General Public License v3.0
41 stars 20 forks source link

lammpstrj converter script argument type fix #121

Closed rwsmith7531 closed 2 years ago

rwsmith7531 commented 2 years ago

Description

Previously, the lammpstrjconvert.py script would throw an error due to the argument nmols being taken as a list of strings by default. I fixed it by specifying type=int.

I also added pregen to the listed allowable sim types in the error message displayed when an invalid sim type keyword is read from the input file.

Related Issue

Closes #120

How Has This Been Tested?

The lammpstrjconvert.py script ran successfully after the implementation of the argument type fix.

Backward Compatibility

No change.

Post Submission Checklist

Please check the fields below as they are completed

Further Information, Files, and Links

Any additional information here, attach relevant text or image files and URLs to external sites, publications , etc.