Skip to content

Add ability to write CHARMM coordinate files. - #739

Merged
drroe merged 7 commits into
Amber-MD:masterfrom
drroe:charmmcorwrite
Jul 12, 2019
Merged

Add ability to write CHARMM coordinate files.#739
drroe merged 7 commits into
Amber-MD:masterfrom
drroe:charmmcorwrite

Conversation

@drroe

@drroe drroe commented Jul 11, 2019

Copy link
Copy Markdown
Contributor

Previously, could only read.

  [help Formats trajout cor]
    Write options for Charmm COOR: Keywords: cor; Extensions: '.cor'
	keepext                : Keep filename extension; write '<name>.<num>.<ext>'
	ext                    : Use 'extended' format (default when > 99999 atoms).
	segid <segid>          : Use <segid> as segment ID for all atoms.
	segmask <mask> <segid> : Use <segid> as segment ID for atoms selected by <mask>.

@drroe drroe self-assigned this Jul 11, 2019
@drroe
drroe merged commit 13f551f into Amber-MD:master Jul 12, 2019
@drroe
drroe deleted the charmmcorwrite branch July 12, 2019 12:14
@Olllom

Olllom commented Dec 5, 2019

Copy link
Copy Markdown

This is super helpful, Dan. Thanks!
A minor suggestion: writing out the periodic box vectors in the header comment could be a good idea. Only if it is not too much work of course.

@hainm

hainm commented Dec 5, 2019

Copy link
Copy Markdown
Contributor

A minor suggestion: writing out the periodic box vectors in the header comment could be a good idea.

@Olllom Just curious: why is that helpful?

@Olllom

Olllom commented Dec 5, 2019 via email

Copy link
Copy Markdown

@drroe

drroe commented Dec 6, 2019

Copy link
Copy Markdown
Contributor Author

writing out the periodic box vectors in the header comment could be a good idea.

It's not a problem. Can you give an example of the format you would prefer?

@Olllom

Olllom commented Dec 7, 2019

Copy link
Copy Markdown

Here's an example of what openmm writes into the pdb header:

REMARK   1 CREATED WITH OPENMM 7.3, 2019-02-28
CRYST1   30.584   30.584   30.584  90.00  90.00  90.00 P 1           1 

Maybe that would work.

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.

3 participants