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

delphi (C++ v7.1) does not understand old radius file #35

Open
3 tasks done
graik opened this issue Aug 4, 2018 · 0 comments
Open
3 tasks done

delphi (C++ v7.1) does not understand old radius file #35

graik opened this issue Aug 4, 2018 · 0 comments
Assignees
Milestone

Comments

@graik
Copy link
Owner

graik commented Aug 4, 2018

C++ Delphi versions do not seem to understand the radii.siz (default.siz) file used together with previous versions. The beauty (and shortcoming) of the old file was that it only assigned a single radius per element. This does not seem to work any longer.

As a temporary workaround, I am adding the new "sample radius file" from the Delphi web site:
https://honiglab.c2b2.columbia.edu/software/DelPhi/doc/parseres.siz

  • added to biskit/data/delphi/radii_delphi_v7.siz
  • adapt atom and residue names in this file to Amber topology
  • make this the default radii file of Delphi wrapper

This file is optimized to be used with a different charge set. Not sure though, whether it makes any sense physically to combine these radii with amber charges. Better would be to re-create this file from the Amber topology. Radii are not given in the leap prep library files. Instead, they may be fetched from AmberTools/src/pbsa/sa_driver.F90, at least this is what is claimed here:
http://archive.ambermd.org/201610/0088.html

@graik graik added this to the 3.0 milestone Aug 4, 2018
@graik graik self-assigned this Aug 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant