Add fatal errors for VV and twin-range MTS
authorMark Abraham <mark.j.abraham@gmail.com>
Tue, 13 May 2014 15:52:50 +0000 (17:52 +0200)
committerGerrit Code Review <gerrit@gerrit.gromacs.org>
Thu, 22 May 2014 07:13:46 +0000 (09:13 +0200)
commit0278ef878daa58b5ea41e5998bd8f0d499d5944b
tree3f7a54c8729e34c74ec1c70045f944b10b047483
parent4459cfffe6b23c84b1e54cb45304a56ca3a4ae3b
Add fatal errors for VV and twin-range MTS

Michael never implemented the multiple-time stepping with the VV
integrator family and constraints (see code that calls
combine_forces() from update_coords() in src/mdlib/update.c). Probably
that means the multiple-time-step regime was not tested with VV
either. Strictly speaking, these new fatal errors have scope that is
wider than is clearly warranted, but it is not clear the
no-constraints VV path was only ever as bad as the broken leap-frog
path is (see #1400).

I suspect VV+constraints will work with the incoming fixes for
leap-frog, but until someone wants to use it (and why would they?),
then I'm not going to test that it works as well as it does with
leap-frog.

Change-Id: Ib61d0fb7661bca2101c04423a6af1744420c06ab
src/kernel/md.c
src/kernel/readir.c
src/mdlib/md_support.c