mlund / faunus

A Framework for Metropolis Monte Carlo Simulation of Molecular Systems
https://faunus.readthedocs.io
MIT License
65 stars 34 forks source link

Add LangevinDynamics move #298

Closed rc83 closed 4 years ago

rc83 commented 4 years ago

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

rc83 commented 4 years ago

I keep commits as breadcrumbs to easily accommodate any future change requests. I will squash them together and force-push, when approved.

mlund commented 4 years ago

Besides the failing unittest, merge at will!