Fix incorrect AWH free-energy output with multiple walkers
authorMagnus Lundborg <magnus.lundborg@scilifelab.se>
Mon, 14 Dec 2020 11:22:48 +0000 (11:22 +0000)
committerMark Abraham <mark.j.abraham@gmail.com>
Mon, 14 Dec 2020 11:22:48 +0000 (11:22 +0000)
commit0de531a0773d86d3ffb6dcf402fdfc032ccbda72
tree411ca2ec3a90e87cc06ff7804e4e6e97435ea492
parente55807071630541473c89b55e35f5d497ac617df
Fix incorrect AWH free-energy output with multiple walkers

The free-energy was averaged exponentially over walkers using the wrong
sign for the exponent. The meant that the free-energy output was off
for large update sizes. This error was quadratic in the update interval
and the number of walkers. This likely has no measurable effects with
the default AWH free-energy update interval. But the effect is noticeable
with the not uncommon update interval of 1000 steps.

Fixes: #3828
docs/release-notes/2020/2020.5.rst
src/gromacs/awh/biasstate.cpp