-
Notifications
You must be signed in to change notification settings - Fork 51
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
Gaussian Quadrature for TI #302
Comments
@hl2500 Thanks for raising this. Sorry we don't have plan for this right now as I mainly uses MBAR. However, we do welcome contributions on more estimators and I'm happy to review any PR that shall implement this. |
The issue with Gaussian quadrature is that the lambdas need to be chosen precisely for it to work, and generally the right lambdas to get Gaussian quadrature to work are not the right lambdas to minimize the variance along the dH/dl curve. If there are particular papers that show how Gaussian integration can do better than trapezoid rule, feel free to post. |
Until we have either evidence showing a clear advantage or someone willing to create a PR with test implementation, I'll close the issue as "not planned". Please feel free to respond and if necessary, we can re-open. |
Thanks for the comments. @xiki-tempula @mrshirts @orbeckst I will try to submit a PR soon. According to this paper: https://pubs.acs.org/doi/10.1021/acsomega.9b04233?ref=pdf, Therefore, I think it might be worth providing an optional functionality. |
I will point out that there was apparently no attempt to optimize where the lambda points were placed in 13 lambda trapezoidal rule, which is known to be less efficient. The advantage of trapezoidal rule is that it works no matter where the lambdas are set - Gaussian quadrature requires the lambdas to be set in different places, which would require having proper validation the lambdas are set correctly - though that wouldn't be that hard to write code for. Not sampling the endpoints is probably a bad thing - if you can't simulated at lambda=0 or lambda=1, something is up with your implementation. |
Hello, I'm wondering if alchemlyb could support gaussian quadrature to estimate free energy from TI, other than trapezoidal rule?
The text was updated successfully, but these errors were encountered: