Fix handling of counter resets
authorMark Abraham <mark.j.abraham@gmail.com>
Fri, 27 Apr 2018 16:35:34 +0000 (18:35 +0200)
committerMark Abraham <mark.j.abraham@gmail.com>
Wed, 22 Aug 2018 13:59:10 +0000 (15:59 +0200)
commit1aa4fa40738209253abae5e3ebb8d3cf01b46950
tree26ec336114f62f97637651718b391437a985fe78
parent4ad3d382ad4e45961e1105254208f6e54008deeb
Fix handling of counter resets

There is no reason for or need to change max_hours, ir->nsteps or
step_rel when doing a counter reset.

This makes clear that the behaviour for the combination

mdrun -maxh t -resetstep n

matches the documentation of -maxh.

Updated the API for walltime_acccounting and its usage, because
elapsed time is an insufficiently clear context. Changed the names of
the start and stop functions so that no callers can silently rely on
semantics that have changed.

Avoided variables such as elapsed_time and max_hours, which were
insufficiently precisely worded.

Refs #1781

Change-Id: I16c96985f43a7b4ac75b94f378da3d05914d6986
src/gromacs/ewald/pme-only.cpp
src/gromacs/mdlib/sim_util.cpp
src/gromacs/mdrun/md.cpp
src/gromacs/mdrun/minimize.cpp
src/gromacs/mdrun/tpi.cpp
src/gromacs/timing/walltime_accounting.cpp
src/gromacs/timing/walltime_accounting.h