Fixes to the gravity code

This fixes #363. The long-range gravity task has to consider all the cells that were assigned to it at the rebuild time. 

Additional improvements:
 - Drift the multipoles while activating the tasks,
 - Avoid FPEs in the FFT task,
 - Adjust the number of tasks per top-level required to run.

See merge request !430

Small cosmetic improvements to the documentation of the FMM code.



See merge request !432

Added debugging checks to verify that the particles are in the correct frame after the shift in DOPAIR1 and DOPAIR2. The check takes into account directions and drift since last rebuild.

Corrected the same check in the vectorized versions of the pair interactions. This was not using the correct criterion for the maximal displacement any particle can have had since the last rebuild.

See merge request !431

Added debugging checks to verify that the particles are in the correct frame after the shift in DOPAIR1 and DOPAIR2. The check takes into account directions and drift since last rebuild.

Fixed the gravity bugs reported in #363. Need to find a good rebuild criterion for gravity-only runs.

Updated tolerances for GCC compilation.



See merge request !429

Run naive interactions

Added an option to run the naive interactions instead of the optimized ones. These are to be used for debugging purposes.

See merge request !428

Patch vectorised pair interactions

Implements small optimisations to `runner_dopair1_density_vec` suggested in #354:
 - Break out of the loop checking for active particles if the cell itself is inactive
 - Do the calculation in the frame of `cj` and not `ci` since this is where the shift vector brings us.
 - Correctly apply the rshift value on the axis linking cells.

See merge request !407

Fix to #363. Drifting decision needs to use the same multipole information as the actual tree walk. Also fix some gravity_cache padding values.

Added the missing DOSELF1_NAIVE() interaction function. Added checks that particles are drifted also in the subset functions.

Use H5Lexists to check whether /Units exists in IC files before attempting to read them

Updated to use H5Lexists to check whether user has input units into 
their Initial Conditions files. This avoids error rasises from H5G* when
opening files without units and reverting back to the ones specified in
the parameterfile. We now also let users know when their units are being
read from their ICs explicitly.

See merge request !427

their Initial Conditions files. This avoids error rasises from H5G* when
opening files without units and reverting back to the ones specified in
the parameterfile. We now also let users know when their units are being
read from their ICs explicitly.

Clean up task and threadpool plotting scripts

Merges MPI and non-MPI task dump plotting and analysis scripts into single
versions that deal with both outputs.

Various tweaks so that we can generate and display task and threadpool
plots together for comparison (non-MPI only).

See #337.

See merge request !425

Symmetric version of DOPAIR2

Here is a much much better version. I now reproduce the pattern that was used in the original version but with correct loop bounds and use of rshift. 

It's now almost on par with the old (but wrong) version. 

See merge request !426

Also remove debugging output

Previously worked the other way around, which is incorrect.

When given a range of 0 autorange the tasks and plot the threadpoool using the autoranged limits of the tasks

Applied @jwillis' correction to the second loop of DOPAIR2. dx_max was entering the external loop with the wrong sign.