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

writing NUMLP section into psf file #14

Closed
wants to merge 1 commit into from
Closed

writing NUMLP section into psf file #14

wants to merge 1 commit into from

Conversation

JohannesKarwou
Copy link

Thanks for the support of LPs in ParmEd - that's really helpful!

As mentioned by @wiederm in ParmEd#1157 there is still a problem when writing a psf file containing CHARMM LP (the lone pair section is always empty).

I needed this functionality so i tried to fix this. Although I am not a seasoned python coder I think I found the problem and I came up with an idea for a fix. If you are interested have a look at the code maybe it can help you!

@swails
Copy link
Owner

swails commented Mar 24, 2022

Thanks -- I'll try to put together a test case for this.

"F",
atom.frame.local_position[0],
atom.frame.local_position[1],
atom.frame.local_position[2],
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The local_position for this frame isn't quite right. It's transformed relative to what is in the PSF file so this doesn't work.

I had to change things a bit to get this functionality to work. I also added support for writing Drude anisotropy records correctly. It's reordered relative to what CHARMM puts in the PSF, but I've checked programmatically and it looks to all be correct. I've added test cases for it as well.

"F",
atom.frame.local_position[0],
atom.frame.local_position[1],
atom.frame.local_position[2],
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The local_position for this frame isn't quite right. It's transformed relative to what is in the PSF file so this doesn't work.

I had to change things a bit to get this functionality to work. I also added support for writing Drude anisotropy records correctly. It's reordered relative to what CHARMM puts in the PSF, but I've checked programmatically and it looks to all be correct. I've added test cases for it as well.

@swails
Copy link
Owner

swails commented May 31, 2022

This is now implemented with test cases.

@swails swails closed this May 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants