Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for reading Gromacs TNG trajectory files #768

Merged
merged 79 commits into from
Oct 18, 2019

Conversation

drroe
Copy link
Contributor

@drroe drroe commented Oct 18, 2019

Will read coordinates, box info, velocities, and forces from Gromacs TNG files. Makes use of the Gromacs TNG library (https://github.com/gromacs/tng), which will be bundled with cpptraj. Adds a test and updates manual/README.

Daniel R. Roe added 30 commits October 7, 2019 13:39
…iven file is TNG or not, look for the 'GENERAL INFO' block in the first 52 bytes. Keep track of file open status since calling the TNG close routine on a TNG file that is not open results in a segfault.
@drroe drroe self-assigned this Oct 18, 2019
@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2019

This pull request introduces 47 alerts when merging ca7b5d7 into 8f53021 - view on LGTM.com

new alerts:

  • 28 for FIXME comment
  • 11 for Multiplication result converted to larger type
  • 4 for No space for zero terminator
  • 3 for Comparison result is always the same
  • 1 for Use of potentially dangerous function

@hainm
Copy link
Contributor

hainm commented Oct 18, 2019

OMG, for real?

Daniel R. Roe added 2 commits October 18, 2019 10:51
…esent. Also make sure the link ordering is correct. Change define from NO_TNGFILE to HAS_TNGFILE
@lgtm-com
Copy link

lgtm-com bot commented Oct 18, 2019

This pull request introduces 47 alerts when merging b75bf53 into 8f53021 - view on LGTM.com

new alerts:

  • 28 for FIXME comment
  • 11 for Multiplication result converted to larger type
  • 4 for No space for zero terminator
  • 3 for Comparison result is always the same
  • 1 for Use of potentially dangerous function

@drroe
Copy link
Contributor Author

drroe commented Oct 18, 2019

For some reason the tng subdirectory was still looked at by lgtm. Going to merge anyway and deal with that later.

@drroe drroe merged commit f12d637 into Amber-MD:master Oct 18, 2019
@drroe drroe deleted the addtng branch October 18, 2019 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants