-
Notifications
You must be signed in to change notification settings - Fork 33
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
Add LangevinDynamics move #298
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@SHervoe-Hansen @rc83: Very impressive work from both of you. I have comments mostly of cosmetic origin and we can add further testing once this first commit has been merged.
I keep commits as breadcrumbs to easily accommodate any future change requests. I will squash them together and force-push, when approved. |
Besides the failing unittest, merge at will! |
Include a minimal example of LJ fluid.
Description
Add LangevinDynamics move including user documentation and a minimal example of a LJ fluid.
Thorough review is welcome as (i) the functionality is very new and (ii) the branch split 6 months ago hence several conflicts had to be addressed.
Other related but different fixes are also provided in separate commits, thus ‘rebase and merge’ is a good strategy for merging.
Contributes to #282.
Checklist
make test
passes with no errorsclang-format
):