Modified requirement to >= 8 columns

lorenzo-rovigatti / tacoxDNA

A collection of tools for DNA modelling

http://tacoxdna.sissa.it/

GNU General Public License v3.0

17 stars 21 forks source link

Modified requirement to >= 8 columns #27

Closed ohenrich closed 3 years ago

ohenrich commented 3 years ago

Hi Lorenzo,

The format of data files written with the write_data command has changed to more than 8 columns. This is because the command was previously not supported for ellipsoidal particles (the only option was to write binary restart files).

I relaxed the sanity check to >= 8 columns and the conversion seems to be fine.

CU O

lorenzo-rovigatti commented 3 years ago

Hey Oli,

thanks for the PR. Note that I'll be deleting this branch since I have since switched the master from python2 to python3.

ohenrich commented 3 years ago

Hi Lorenzo, Thanks for the info. Good idea to upgrade everything to python3. I'll delete the python3 branch right away from my fork! CU O