-
Notifications
You must be signed in to change notification settings - Fork 295
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
Replica exchange exchanging biases #1184
Comments
Some suggestions on the syntax:
Is there a specific reason to choose the {a b c} notation instead of the usual a,b,c (also consistent with the @replicas syntax)? I would maybe argue that intuitively this
could be the implied default to
equivalent to
It makes sense to me intuitively that each "ARG" is interpreted as one "exchangeable bias". With this perspective
can be rewritten
Though the repetition may be undesirable… But the case where we are using different scalings of the same bias seems more like a special case than the general case. What do you think? |
Thanks @ollyfutur !
Good point
I agree
Good point
Both are possible (comma separated or space separated), and equivalent. I prefer to use space separated when there are matrices involved. About the default
Then, perhaps:
The result would be: For replica exchange umbrella sampling
For interpolation between two potentials
For scaling across replicas*
|
Another suggestion is So these "broadcasting rules" for
|
Currently, we only support replica exchange where coordinates are exchanged. This is necessary to avoid the need to "transfer the plumed object" to a different replica.
There's one special case however that could be straightforward to implement namely:
In this case, it is conceivable to have an action like this:
This action would store the current permutation of factors and, instantaneously, apply a the corresponding factor to the current replica. It would then try an exchange every 1000 paces, and append the permutation to the exchange.dat file, so that it can be read when restarting.
To have slightly more flexibility, we could have an array of arguments, e.g.
This could even enable replica exchange umbrella sampling
Nicely, this would work with all codes that support multiple replicas (including the vanilla gromacs versions, when this will be merged).
Any comments on the syntax/usability?
The text was updated successfully, but these errors were encountered: