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

Properly merge branch to contain MBAR fixes #80

Merged
merged 7 commits into from
Feb 25, 2016
Merged

Properly merge branch to contain MBAR fixes #80

merged 7 commits into from
Feb 25, 2016

Conversation

nathanmlim
Copy link
Collaborator

It looks like the branch zeroxvg was merged into master before the MBAR-fix branch was allowed to merge with zeroxvg. Currently, the master branch only contains commits before the the MBAR-fix.

@davidlmobley
Copy link
Member

Actually, this is what I thought we were doing as I wanted them to come in as two separate changes. So, this new PR is exactly what I wanted from you.

For the record:

This PR moves #79 from the zeroxvg branch into the main branch.

davidlmobley added a commit that referenced this pull request Feb 25, 2016
Fix MBAR uncertainty totals and handle statistical inefficiency calculation when there is zero free energy change at some lambda values (moving #79 into main branch)
@davidlmobley davidlmobley merged commit 9d0d6c8 into master Feb 25, 2016
@nathanmlim nathanmlim deleted the zeroxvg branch February 25, 2016 00:35
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.

None yet

2 participants