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

Auto equil #89

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Conversation

matteoaldeghi
Copy link

This adds the option to use detectEquilibration from the timeseries module to detect the equilibrated region of the windows automatically. It modifies the uncorrelate function to do this. When using the option, it is still possible to specify how much to discard at the beginning with the '-s' flag, but the time specified ends up being only a lower-bound equilibration time in this case. At the moment I made it incompatible with the '-f' flag for the time-reverse plots as I imagine it would result in something different from what the plots were intended to be.

I ran it on a couple of different sets of xvg files from gromacs and had no issues, however, considering I am not that familiar with the code, it is probably worth double checking it works indeed as you would expect and does not interfere with anything else...

@davidlmobley
Copy link
Member

@shuail - can you have a look at this and see if you'd like this and if it works for you?

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

Successfully merging this pull request may close these issues.

2 participants