Skip to content
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

GOMC parser #117

Closed
msoroush opened this issue Apr 24, 2019 · 3 comments
Closed

GOMC parser #117

msoroush opened this issue Apr 24, 2019 · 3 comments

Comments

@msoroush
Copy link

Hi everyone,

I am working on implementing free energy calculation in GOMC and I have been working on GOMC parser for alchemical-analysis and you can find my implementation in my GitHub. I attached my results here and you can find the raw data in samples/gomc/met-wat-NVT.

I have few questions and concerns about alchemical-analysis and I would appreciate if your could give me some feedback.

  1. alchemical-analysis modifies the raw data as mentioned in previous issue Overwriting .xvg files #95. This causes problem when I run alchemical-analysis on unfinished simulation data file.
  2. Forward and backward analysis on unfinished simulation results in wrong value. I think we need to look into all files and use the minimum simulation time/steps to perform the analysis.
  3. The overlap matrixes in the result directory that I attached is very strange. Summation of probability in each row is one, but not for each column. Is this behavior expected or not? When I plot my energy distribution for neighboring states, they have very good overlap as you can see in attached images. So why I am seeing no overlap (between lambda 9,10,11) in overlap matrix?
  4. How do we calculate the correlated and uncorrelated samples?

I would appreciate your help and suggestion.

result.zip
energyHist
energyplot
Screen Shot 2019-04-24 at 12 04 23 PM

@davidlmobley
Copy link
Member

Hi, we're no longer supporting alchemical-analysis and are moving towards alchemlyb (github.com/alchemistry/alchemlyb) instead, and long-term will be migrating the plotting functionality, etc. there. alchemlyb already has much of the key functionality and there's general agreement that we should be porting in the remaining aspects (overlap matrix plots, etc.).

Essentially, the issues you're running into relate to those driving this change: While functional, alchemical-analysis is not well designed enough to be easily maintainable, and, when applying it to calculations other than those for which it was originally envisioned, one frequently runs into problems. There's also no testing framework in place, meaning that it's very difficult to know if "fixes" might accidentally break something. In contrast, the alchemlyb framework is much better designed, easily maintainable, modular, etc. We also have automated testing in place (and plans to continue extending that).

So, I'd highly encourage you to check that out and consider contributing there, and move away from alchemlyb.

@orbeckst
Copy link

Note that @msoroush added the GOMC parser to alchemlyb alchemistry/alchemlyb#77 so I think this issue can be safely closed.

@davidlmobley
Copy link
Member

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants