Amber-MD / cpptraj

Biomolecular simulation trajectory/data analysis.
Other
138 stars 64 forks source link

Add ability to utilize and manipulate internal coordinates #1053

Closed drroe closed 1 year ago

drroe commented 1 year ago

Version 6.21.0.

Add commands/keywords related to internal coordinates to cpptraj:

Also adds the ability to read in units from Amber Prepin and OFF library files as COORDS data sets; the latter can be used directly with the sequence command. Updates the manual and adds new tests. Lots of additional fixes under the hood.

drroe commented 1 year ago

Sometimes CI runners are no fun. New issue popping up that has never been a problem before - clang/gfortran incompatibility:

Error: Test compile failed: clang++  -Wall  -O3     -std=c++11 -o testp testp.cpp /home/runner/work/cpptraj/cpptraj/lib/liblapack.a /home/runner/work/cpptraj/cpptraj/lib/libblas.a -lgfortran -lpthread    
  Checking built LAPACK/BLAS: 
Error: Check the output below for error messages:
/usr/bin/ld: cannot find -lgfortran: No such file or directory

Not sure how to fix it yet.

drroe commented 1 year ago

Turns out I had already fixed this transient issue (clang/gfortran) in configure, but only for OSX. I just took that code and made it apply to all OS - seems to have done the trick. 🎉