-
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
Replace entire link instead of potential by potential #584
Comments
This is definitely an unfortunate side effect, thanks for pointing it out. As an aside/work-around, links can also /remove/ interactions, which should at least clean up your itps. |
@pckroon I agree that this convention is the most sensible and easy(ish) to implement. Conceptually, however, it would make a lot of sense to allow grouping of interactions (i.e. an interaction group - not to be confused with grouping them in the output). At the end of the day this type of dihedral potentials represent one interaction in the MD input as well. Changing just one term is a highly unlikely scenario and rather one wants to change all of them. GROMACS used to strictly enforce this by having them specified as interaction type 9, but that somehow got lost and now 1 and 9 are essentially the same. In my view, if we were to completely abolish versions and just treat those interactions as a group, it would simplify a lot of the conceptual aspects of links. The code will likely be more complicated, which is why I am not pushing this right now. |
That would indeed be the correct solution (I think), but does indeed cost a bit more code. I think I'll have to bite the bullet though. My previous suggestion has too many edgecases where what you want/need to do is either not possible or super hard. For example, when you want to switch from a "versionless" dihedral with a single potential to a "versioned" dihedral with multiple potentials. |
In the case where you have a default link which is made up of a sum of several potentials, i.e.:
And you want to replace it in a specific case (particular aminoacid for example) by a link made up of a single potential. i.e.:
You are forced to introduce a second potential with
fc=0
in order to override the second part of the default link. i.e.:Otherwise, you end up with a mix of the default and the override. Would be cool (and help clean up .ff files) if we could somehow replace the entire link instead. Currently, we are ending up with dirtier
.ff
and.itp
files full of "zeroed" potentials.The text was updated successfully, but these errors were encountered: