-
Notifications
You must be signed in to change notification settings - Fork 11
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
Why is SerialMover used? #229
Labels
Comments
SerialMover enables a Gibbs Sampling scheme, where particles (ie, XYZ, RigidBodies, Scales) are all moved once. Nowhere is strictly recommending this mover with incremental scoring function, or at least I don't quite understand your question. |
Agree with Riccardo, given a list of movers (eg. individual bead
movers,rigid body movers..) it enables performing an MC step once per
mover in the order specified in the list (Gibbs sampling essentially). The
SerialMover only specifies how the Gibbs sampling is done and doesn't
impose any requirements on the scoring function.
…On Fri, Mar 17, 2017 at 3:50 AM, Riccardo Pellarin ***@***.*** > wrote:
SerialMover enables a Gibbs Sampling scheme, where particles (ie, XYZ,
RigidBodies, Scales) are all moved once. Nowhere is strictly recommending
this mover with incremental scoring function, or at least I don't quite
understand your question.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#229 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AH7fbxSy-ZhD-_kX--goFD7T_rps7Zpjks5rmmVZgaJpZM4MgTGv>
.
|
Should we close this? |
Yes, thanks, you guys are absolutely right. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Perhaps I'm wrong, but it looks like
IMP.core.SerialMover
is used inIMP.pmi.samplers.MonteCarlo
to propose movers in order, one for each Monte Carlo step. My understanding is that this mover is only recommended when using an incremental scoring function.The text was updated successfully, but these errors were encountered: