-
Notifications
You must be signed in to change notification settings - Fork 46
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
Rtp parser - Abandon all hope, ye who review here #631
base: master
Are you sure you want to change the base?
Conversation
…es get a 'charge_group'
…it RTP interactions with links
@pckroon quick question before I review: does this mean the interactions that depend on others don't work then? |
Sorry, don't understand the question |
If I put in an abitrary rtp file, am I guaranteed to get the correct answer with martinize2? |
Aside from testing and termini, yes. |
Here's termini modifications that I think should work, except for a N-term GLY
|
As a sidenote, I tested this on 1UBQ using |
This pile fixes most of the RTP parser. It's not possible to generate the implicit interactions (specifically the dihedrals) using Links, since there sometimes is a selection criterion to decide which of all the options to keep, which also depends on already existing interactions. In addition, because the links that would add these interactions are super symmetric, it quickly becomes a headache.