Temporarily lift requirement to detect GPUs in CI

Remove leftover function declaration from nbnxm atomdata

MR !1483 neglected to remove the declaration of function
nbnxn_atomdata_init. This MR rectifies that. Also removed
and unneeded include.

Apply clang-tidy11 to mdrun

Unify constructor of nbnxn_atomdata_t

Constructor for t_mde_delta_h_coll

Increase const correctness of t_mdatoms use in md.cpp

Previously two instances of t_mdatoms were made in do_md, once const
and one not. This replaces all instances where the non-const was not
used but could be with const usage. Also, non-const usage is now done
by calling MDAtoms->mdatoms() to indicate non-const usage more clearly.

Remove inputrec from initialize_lambdas signature

Apply clang-tidy11 to correlationfunctions

Apply clang-tidy11 to simd

Apply clang-tidy11 to random

Apply clang-tidy11 to commandline

Fix MSVC build

The enum class must have collided with some internal type in MSVC, so we
rename it.

Clang-11 fixes for applied forces

Specialize jacobi() for 3 dimensions

Make SETTLE setup code platform-agnostic

Apply clang-11 fixes for fileio

Fix beningn GCC 10.2 Wstringop-truncation warnings

- `get_pdb_atomnumber`
  GCC was worried that we're copying up to 4 bytes to a buffer of
  length 4, not leaving the space for terminating \0. The source string
  is guaranteed to have length 3 + terminator, so not a problem.
  Changed to strcpy, since it should be safe here.
  While at it, renamed a variable, gave a name to a magic constant,
  added a static_assert to ensure safety.

- `read_vsite_database`
  GCC was complaining that we're copying 4094 bytes from a string of
  length 4094, possibly truncating the terminating \0. The string was
  guaranteed to have a terminator in its first 4093 bytes (fgets2 will
  put it no later than STRLEN-2, and we also skip the first character).
  But to make GCC happy, I see no harm in adding 1 more byte to strncpy.

Another warning from GCC is an actual error, fixed in MR !1487.

Clang-11 fixes for PME and FFT

Prepare virial calculation for legacy md-vv pressure coupling

Make LINCS setup code platform-agnostic

Improve UpdateGroups testing infra

Made and used a class fixture, used scopes to clarify lifetimes,
updated naming to follow general style, separated tests of the effect
of temperature upon radius calculations.

Fixes from clang-tidy-11 for signal handling

Vectors in history_t

Fix MSVC build

Fix doxygen of the SYCL atomicFetchAdd

More renaming for UpdateGroups

In preparation for introducing a class for managing the use and data for
UpdateGroups, other things need to change so we'll have clarity.

Refs #4004

Finish removing t_mdatoms from update module

Remove t_atoms from finish_update and only pass needed params.

Also cleaned up the includes and forward declares a bit.

Remove isnormal() check around the atomic increment

Mostly remove t_mdatoms from update signatures

clang-tidy-11 fixes for gmxpreprocess

Introduce TimeStep and Offset strong types

When building integration algorithms in modular simulator, the time
step (for propagations) and the scheduling offset (for algorithms called
periodically) are the most frequently used type of arguments. They are
also currently the only arguments which are passed by simple types
(int and double, respectively).

The current change introduces strong types, increasing both safety and
readability. Note that these strong types can easily be adapted to
inheriting from a general base class if one is introduced (refs #4005).

Fix nbnxm hipSYCL kernels 64-wide exec on AMD

Fix doxygen for MessageStringCollector

Refs #4004

Let LoggerTestHelper write cleaner tests

Now we can write tests that the logger produces no output, e.g. no
warnings occur when a happy path is able to run.

Refs #4004

Use index instead of pointer in t_mde_delta_h_coll

Several members of t_mde_delta_h_coll were just pointers into a
master array of t_mde_delta_h that are held by t_mde_delta_h_coll.
This change makes it so only an int instead of a pointer is stored.
After this it should be much easier to make t_mde_delta_h_coll
default constructible, which is planned for a followup.

Also removed 2 unused members.

A renaming of the members can also be left for a followup, once
the whole t_mde_delta_h_coll struct is modernized.

Remove uses of std::bind

clang-tidy-11 correctly warns against these unnecessarily complex
constructs.

Refs #3897

Use consistent naming for update groups

Prepares to introduce an UpdateGroups class that contains the groupings

Refs #4004

Rename some enum names in ModuleMultiThread

clang-tidy-11 fixes for topology, gpu_utils, and selection

Fix clang-format

Was missing from previous commit.

Fix possible UB in ArrayRef construction

The code could cause possible UB when construction ArrayRefs from
nullptrs.

Added nullptr check to only construct empty ArrayRef in that case.

Use enum class for gmx_omp_nthreads

clang-tidy-11 fixes for utils

clang-tidy-11 fixes mdrun tests

Split off from !1433

clang-tidy-11 fixes for listed_forces

Apply clang-tidy11 to gmxana files with no dependencies

Splitt off from !1433. These are only the gmxana files that do not
depend on other clang-tidy11 changes. The files gmx_analyze.cpp,
gmx_nmr.cpp, gmx_bar.cpp, gmx_wheel.cpp and gmx_energy.cpp all have
dependencies, mostly on enum modernization. They can be a subsequent
MR.

clang-tidy-11 fixes for options

Split off from !1433

clang-tidy-11 fixes for hardware

Split off from !1433

clang-tidy-11 fixes for analysisdata

Make .rtp/.tdb bond types into enum class

Make Compartment, Channel, ChannelHistory and Domaint into enum classes

Make XDR type into enum class

Add Gaurav to list of contributors

Set matching number of ranks for OpenCL GPU tests

Removed a duplicated quote

Rename files to follow style

Fix AWH input reading

Tests happened at the wrong time before all AWH and FEP parameters had
been initialized, leading to aborts due to failing checks.

Also fix initialization of dim parameters.

Fixes #4027

Remove thread-MPI limitation for GPU direct PME-PP communication

Allows use of direct-GPU communication for PP-PME communication when
running with "real" MPI, including on multiple compute nodes,
through new CUDA-aware MPI communication code paths.

Implements part of #2891
Refs: #2915 #3960

Introduce position and velocity scaling propagators

Use host buffer instead of host accessors in SYCL LeapFrog

Pass forcerec arguments instead of forcerec to gmx_nb_free_energy_kernel

Modernize wallcycle counting

Fix misc-misplaced-const clang-tidy warning

The problem was introduced in aa66efd3179a671aace62b827c54aadc3d89c1ee
(MR !1374).

As MPI_Comm is typedef'd as a pointer to a structure, declaring it const
would be of little use: the pointer itself would be const, not the data
it is pointing to.

Implement Nose-Hoover thermostat for leap-frog modular simulator

This implements the Nose-Hoover temperature coupling algorithm for the
leap-frog version of modular simulator. Velocity verlet could in theory
use this coupling algorithm as well, but the legacy version uses the
Trotter-decomposed version instead, which will be introduced to modular
simulator in a follow-up commit. To ensure equivalence of the legacy and
the modular code paths, Nose-Hoover will not be enabled for md-vv in
modular simulator until the Trotter scheme is merged.

The implementation added in this change is straightforward, as it extends
the existing general velocity-scaling temperature coupling object by
another algorithm. It is marginally more complex than the existing
algorithms (Berendsen and v-scale) due to the need of checkpointing the
extended ensemble degree of freedom for exact continuation.

Refs #3423

Add GMX_GPU_SYCL_NO_SYNCHRONIZE environment variable

Implement dynamic pair search for EM

Replace static trajectoryanalysismoduledata methods with instance methods

This paves the way for multiple concurrent frames analyzing
selections in parallel, via individual analysismoduledata objects

Remove thread-MPI limitation for GPU PP Halo exchange

Allows use of direct-GPU communication for PP halo exchange when
running with "real" MPI, including on multiple compute nodes,
through new CUDA-aware MPI communication code paths.

Implements part of #2891
Refs: #2915 #3960

Use array for term and foreign_term in gmx_enerdata_t

Add basic position restraints test

Fix check source

2e8da9f2ad7536ed0da612a977c6bfe0f89bc071 added a function argument
without updating the doxygen. This adds the missing argument description.

Pass cFREEZE and cTC to update after DD partitioning

Improve docs and naming for MdModulesNotifiers

Remove explicit dependency of EnergyData on thermostat / barostats

Add unit test for the nonbonded fep kernel.

Workaround for compilation with hipSYCL 0.9.1

Expose init_interaction_const and move to correct header/source

Relax default tolerance on pressure comparisons

Fixes #3975

Replace defines with constexpr in ishift

Constructor for gmx_ekindata_t

Moved init_ekin_data from tgroup header/source to group and made
it the constructor for gmx_ekindata_t. Also made gmx_ekindata_t a
class since the number of threads is only needed by constructor
and destructor and can thus be private. Further refactoring of
members of gmx_ekindata_t would eliminate the need for the nthreads
data member entirely. Other members we not made private to avoid a
ripple of changes adding "()" to numerous lines in several files.

Prepare SelectionCollection for parallel analysis

Add a fourth coordinate to virial tests

Allow thermostat to scale velocities before and after step

Velocity scaling thermostats are currently only able to scale
velocities before the respective propagator step. This change allows
to also scale velocities at the end of the step. This functionality
is needed by the Nose-Hoover thermostat.

This requires an additional template parameter in the `updateVelocities`
function, and an additional callback in the
`PropagatorThermostatConnection` struct to allow temperature coupling
algorithms to get a view to the end velocity scaling.

This also sorts the forward declarations in modularsimulatorinterfaces.h
alphabetically to simplify follow-up changes.

Refs #3423

Add a note on early return in device buffer operations

Functions that (re)allocate, clear and copy to or from
device buffer have an early return in case of zero operation
size. This is a safety mechnism that allows calling these
functions even if the buffers are not intialized, if the
size of the respective buffer is set to zero. A note is
added to the devicebuffer.h that explains why these early
returns are needed.

Clean some headers in NBNXM GPU

1. Remove declarations of non-existing functins.
2. Remove declarations of internal helper functions,
   move some of the helper functions up in the
   respective sources so that their declaration can
   be removed from the headers.
3. Remove unnecessary includes.

Refs #2608

Propagate from current rather than previous position

The current implementation was using the previous position as a starting
point of position propagation. This is conceptually wrong: We want to
propagate starting from the current position.

This had no influence currently, as none of the algorithms present has
multiple position propagations per step. Moving forward, this does however
pose the risk of bugs.

Add GpuEventSynchronizer::isMarked to SYCL verion

This function is needed to work around #3988 when implementing GPU
update (#3932).

template: fix build

Move ScalingMatrix to common GPU update code.

Also, add a constructor to initialize it from an old-style matrix.

Currently, this struct is used only in CUDA, but would be used in SYCL
soon.

Refs #3932.

Add early returns in clearDeviceBuffer(...) in OpenCL and CUDA

Clearing of empty buffer can cause the OpenCL/CUDA API failure.
Early return when requested size to clear is zero allows to
avoid this failure not calling the API for empty buffers.

This also makes the definition of the clearDeviceBuffer(...) in
OpenCL, CUDA similar to its definition in SYCL, where the early
return is already present.

Fixes #4002.

Pass ref_t to initialize_lambdas

Fix several errors when compiling with MSVC

Minor forcerec cleanup

* gmx_bool -> bool
* rename a variable
* rvec -> RVec
* more array and vector
* now has default destructor

Clean up leaks

Rename NBAtomData into NBAtomDataGpu

NBAtomData can only be present in GPU code, hence it should has
Gpu prefix in its name.

Replace gmx_mtop_ilistloop_all

Move domdec/partition into gmx namespace; minor cleanup

* Move functions in partition header into gmx namespace
* Localize variables
* Remove an in-out parameter that is never used

Ensure coordinates are copied for dipole moment calculation

With GPU update, we should wait for the coordinates to be copied to host
before using them for calculating the dipole moment.

This did not seem to cause any issues with CUDA but was causing a unit
test failure with SYCL (with !1329):

    SYCL_BE=PI_OPENCL GMX_USE_GPU_BUFFER_OPS=1 GMX_FORCE_UPDATE_DEFAULT_GPU=1 ./bin/mdrun-test --gtest_filter=EwaldSurfaceTerm/EwaldSurfaceTermTest.WithinTolerances/0

Refs #3930, #3932

Make FileOptionTypes enum class

Changed to enum class and made mapping to legacy types use
EnumerationArray.

Removed Unknown type as it wasn't used except for error checking. Used
Count type instead in its place.

Clarify connection of temperature / pressure coupling and propagators

Temperature and pressure coupling and propagators are linked, as most
thermostats and barostats apply some type of scaling to positions and
/ or velocities. The temperature and pressure coupling objects must
therefore be linked to the propagators to properly function.

The current implementation did not allow for much flexibility, as it
only allowed propagators to decide whether to connect to a thermostat
or barostat or not. To allow for more complex algorithms, more
flexibility is required. To that end, this change introduces a
PropagatorTag object, which tags the propagators, and allows the
temperature and pressure coupling objects to pick which propagator to
act on. Additionally, this moves the responsibility of connecting to
the coupling algorithm rather than the propagator, which makes more
sense conceptually.

Refs #3423