GRHD: more specific boundry conditions [only incorporated for FV+slope_limiter] (!179)

This adds new options under `GRHD.bc`:
- `tov_reflective_origin`: impose reflection symmetry at coordinate origin, impose vacuum values at outer bdry
- `tov_symmetric_domain`: impose vacuum values at both domain boundary ends

Also streamlined the setup of the bdry values for FV.
While doing so I realized that the inflow bdry in the Bondi test was wrongly treated.
There we cannot extrapolate interior data, nor just copy what we have in the interior, unless we are very far out (which in my test we aren't).
Instead, what we do now there is use the initial data, which is the analytical solution and extrapolate that.
This improves the behavior of the solution to no longer drift after the first time step. Nice.

The formulae isn't complicated, but I still messed up doing so.

Fixing it makes TOV evolutions in symmetric domains now remain symmetric, kind of.

Because `minmod` is used with almost all slope limiters, I had to update quite a few reftest too, although they overall quality of the results did not change much.

I wondering whether this fix makes now the WENO version of DG also work, will check in a separate PR.

GRHD: fix implementation of `atm_evolve` (!177)

Follow up to !176

Adds the following new parameter:
- `GRHD.atm_evolve`: If `false` then the RHSs of all state variables are set to zero at any grid point where we identified the data to describe atmosphere.

This is parameter is implemented to work with all formulations and schemes, hopefully.

GHRD: Add Marquina's numerical flux for FV+slope limiter and doublecartoon (!175)

Adds the following new parameters
- `GRHD.fv_numerical_flux = "marquina"`: Numerical flux implementation following Font et al 2000, PRD 61 044011; see Font et al 2002, PRD 65 084024 for TOV tests.

We also update the TOV initial data module to include a function with which one should be able to generate all the configurations that we use in the ref tests.

GRHD: Implement FV+slope limiters for doublecartoon version (!174)

Implements the FV+slope limiter machinery for the doublecartoon formulation.

While testing I noticed that the boundary conditions for a reflection boundary were implemented incorrectly.
To this end I quickly drafted a `BdryBufferedData` type with which one can abstract the buffer zones away.
This solution is not optimal performance wise, because it incurs quite some allocations (although I think it should be stack-allocateable...).

~~I have not bothered to update the spherical1d formulation, but instead inserted a `TODO`. So this needs to be updated in a separate PR.~~
Nevermind, I already fixed the todos, in order to make the reftests work.

Fixing the bdry conditions helped to improve the quality of the results, however, we are not even close yet to results from the literature.
Because of that I also now implemented the Roe method instead of the LLF method to compute the numerical fluxes, because that method
was also used in the paper Font et al 2000 PRD 61 044011.
This improves things too, but we are still worse than those results.
In particular, we use more resources and work in lower dimensions than that paper and yet we have larger errors ...
So this requires further investigations.

---

Also adds the following new parameters:
- `fv_numerical_flux`: the double cartoon version now implements the LLF and Roe flux.
- new option for `slope_limiter_method`: `MC`, this limiter is taken from Font et al 2002, PRD 65 084024.

SRHD: Fix FV slope limiter method to use up-to-date primitives in evolution (!173)

Due to the `cons2prim` methods being tied to `D, S, tau` registered as `dynamic_variables`, we need some juggling to get the primitives of the current step.

This inconvenience is due to
1. not (yet) separating the time stepping from the flux computation for the FV methods,
2. and the 'hard coding' of the primitives to the `cons2prim` methods.

Side note: We face the same issue with the `GRHD` version.

---

This PR also adds some more reftests to cover the `blastwave1` exercise.
To this end we also updated the FV solver to work with non-periodic bdry conditions with bdry values taken from initial data.

GRHD: Random improvements to TOV star test (!172)

- Implement AV treatment for `doublecartoon` formulation
- Implement Reconstruction treatment for `doublecartoon` formulation
- clean up logic in `callbacks.jl`
- Implement `fv_central` for `doublecartoon` formulation.

Also adds new parameters to `GRHD` section:
- `c2p_enforce_atm`: If true then atmosphere is enforced, regardless if its causal or not.
- `freeze_vars`: A list of variables for which `rhs` should be set to zero. Atm only supported by the doublecartoon formulation.

New parameter for the `TCI` section:
- `threshold_inv`: Invert the threshold check.

GRHD: Add initial data for Bondi accretion with infalling matter (!169)

This adds the following parameters to the `GRHD` section
- `id`: new option `bondi_accretion_infall`
- `id_bondi_accretion_infall_rs`: Radius below which the low-density fluid is initially set.
- `id_bondi_accretion_infall_ρl`: The density of the low-density fluid.
- `analyze_error_reference_solution`: the name of a reference solution that is to be used for the error estimation

---

This test is taken from Bugner's thesis 2017, and a paper by Radice, Rezolla 2011.

---

This PR also disables the questionable rescaling of the viscosity that was added for the TOV tests.
If needed it can be restored, but should be hidden behind a proper parameter.

CI: Add another CI target that runs the integration tests with 8 threads (!170)

@with_signature: Add `batch` option which enables multithreading of for loops using `Polyester.jl` (!168)

Usage:
```julia
@with_signature [batch=true] myfunction(eq::SomeEquation)
   @accepts ...
   # compute goes here
   @returns ...
end
```

I did some minimal profiling. There is a noticeable speed-up, but I did not measure how effective it is.
For the time being, I am happy when it just runs a little faster.

Evolve: use inline if in ode_step! for LSERK4 (!167)

This helps the profile outputs to associate the measurements to the same line.

- Don't impose initial data values at interior boundary. This is an inflow boundary, but we cheat around that and just use the interior flux there.
- Add a higher resolved version of the Bondi accretion data. This is necessary to get acceptable results in a convergence study.

SRHD: perform c2p when wrapping data in callback (!164)

TBH this is a bandaid to ensure that primitives are up-to-date before outputting.

In the future there should be a proper mechanism to ensure that.

GRHD: Use WeakBoundaries() for CubicSplineInterpolation (!163)

Sometimes our grid bounds don't align exactly with the bounds from the data. This boundary treatment is more forgiving in that sense.

Adds two new parameters:
- `variables0d_analysis_error`: A list of variable names for which the error wrt the initial data values and computed as the L2 norm shall be computed. For each enqueued variable name a `Output.variables0d` entry is added which consists of the variable name and a `_err_norm` postfix. E.g. `rho` adds a 0d output `rho_err_norm`.
- `variables1d_analysis_error`: Similar to `variables0d_analysis_error`, but instead of computing the L2 norm, the difference over the mesh is provided as an `Output.variables1d` entry with name equal to variable name plus `_err` postfix.

---

This is a rushed implementation, because I need that output now.

But this mechanism should be 1) partly factored out into the `dg1d` project, and 2) we should not need extra parameters.
About 2): It should be enough to ask for `rho_err` or `rho_err_norm` in 1d or 0d output fields. Doing so would require us to reserve names ending with `_err` and `_err_norm`, but that should be fine.

Furthermore, we should rename the postfixes to something like `_id_err` and `_id_err_l2`, to make their nature more clear.

EulerEq: Allow use of more than one AV indicator (!161)

This adds two new options
- `av_indicator_variables`: a list of variables to which the AV indicator
  should be applied to. This generates one viscosity per variable.
- `av_combine_operator`: an operater with which multiple viscosities
  are reduced to a single one.

We also re-activated the `euler_sod_shock_tube_entropyav` reftest, which now seems to give acceptable results.
Also updated the `euler2d_kelvin_helmholtz_entropyav` reftest, although that should not have been necessary.
However, 2d experiments are not declared stable anyways, so will need to check those later again regardless.

EulerEq: implement the `bc="from_id"` option for slope limited FV methods (!160)

EulerEq: use the correct initial data for the Sod shock tube test (!159)

GRHD: fix limiting of velocity components (!158)

This was a subtle mistake where applying the velocity limiter build into the Kastaun scheme was not correctly applied to components of the velocity.

EulerEq: Rework Navier-Stokes regularization (!155)

The goal is to reproduce plots for the shock tube tests from the Zingan+ 2013 paper.

We rework the Navier-Stokes regularization to work more like in the Zingan+ 2013 paper. Before we tried to mimic a Guermond+ 2014 paper, but they did not provide any shock tube tests.

This PR adds new parameters to the `EulerEq` section:
- `av_navierstoke_prho, av_navierstoke_pt`: these are effective Prandtl numbers that can be used to control how much of the viscosity coefficient for the momentum regularization should be added to the density and energy regularization.
- `id_smooth`: If true smooth initial data profile with a Bernstein filter
- `av_drag`: Dragging factor to throttle AV evolution
- `av_recompute_substeps`: If true AV computation is scheduled to run on each substep.

We also fixed a sever issue in one of the formulae.
In the computation of `maxspeed` we accidentally compute `q/p` instead of `q/rho` as one part of the wave speed.
Fixing this seems to have a positive influence.
I am curious how this will effect the ref tests.
Answer: I had to update all associated reftests, but all changes were minor.

Initial data configuration taken from Zingan+ 2013 and Toro 1999.

c2p: Add option `c2p_set_atmosphere_on_failure` (!153)

Adds a new option `c2p_set_atmosphere_on_failure` to the `GRHD` and `SRHD` sections.
If `true` and the c2p solver fails to invert the master function, we reset all variables to atmosphere.

We also fixed that the running with `verbose` in `c2p` no longer resest to atmosphere.

fix invocation of substep callbacks for explicit schemes (!151)

- Mainly refactors `c2p` in order to allow to ignore the total energy density in the recovery procedure.
This is possible for cold equations of state, where we don't need to evolve tau.

- The `c2p` now also incorporates a conservative fixing that we adapted from from arXiv:1306.4953, Appendix.

- We had to update reftest data for the SRHD simple wave and GRHD bondi tests.
Especially the GRHD bondi tests improved from not using tau in the reconstruction.

- We also updated the default value `atm_treshold_factor` so that it is now slightly large than one.
This fixes some errors where we misidentified atmosphere states as being below atmosphere, before the c2p procedure started.

- Because this PR was cherry-picked from the fa/doublecartoon+av branch, we also brought over the changes in the naming of variables of the doublecartoon formulation (which should have went into another PR, but I was too lazy to entangle those changes first).

----

I noticed that the number of allocs in `rhs` sections are now quite increased.
This is particularly bad for the SRHD tests. My guess is this is due to this refactor now passing on both EoSs as parameters to the c2p routine.

This PR replaces the SRHD's own c2p with that of GRHD.

The following parameters are added to the SRHD section:
- `id_smooth = false`: If `true` initial data is smoothed with a Bernstein filter.
- `av_regularization = "none"`: Controls which AV regularization is to be used.
- `c2p_cold_eos_parameters = [ 100.0, 2.0 ]`: Parameters of cold polytropic EoS for c2p, analogous to GRHD.
- `av_drag = 0.0`: AV dragging factor to control how rapid AV can change, analogous to GRHD and EulerEq.
- `av_recompute_substeps = false`: If `true` AV recomputation is triggered on every substep in the evolution.

Copy current Manifest.toml to output folder (!148)

GRHD: Use a dedicated cold EoS as a fallback in c2p (!147)

This adds new parameters to the `GRHD` section:
- `c2p_cold_eos_parameters`: Contains parameters for the `Polytrope` EoS that is used as a fallback in c2p.

This kind of 'abstraction' is needed so that we can re-purpose this c2p for the SRHD project.

EulerEq: Fix 2d equations and timestep scaling (!146)

- The 2d version did compute the flux incorrectly.
- We used too strong of a scaling for the time step, e.g. N^-3 instead of N^-2. This is fixed for 1d and 2d case.
- Updated `euler2d_isentropic_flow` ref test to no longer run into a shock, because that example doesn't use any HRSC.
- Updated `euler_sod_shock_tube` test to run with low resolution. Also shorten `tend` so that data does not reach boundary.

This addresses two issues:
- Utilize cold EoS when we limit velocity. This should avoid artificial spikes in epsilon (internal energy density) due to the limiting, because the derived relations used in the Kastaun paper no longer hold.
- Make `atm_threshold_factor = 1.0` the default. In some tests I noticed that having this greater than 1.0 an artificial plateau in the evolved density appears, which I did not like. This seems to fix that.

This adds and modifies the following parameters in the `GRHD` section:
- (new) `c2p_dyamic_atm`: Controls whether a fluid-atmosphere interface is allowed to evolve.
- (renamed) `c2p_freeze_atm_reset -> c2p_enforce_causal_atm`; also updated the doc string

With `c2p_dyamic_atm=true` we now recompute the `c2p_atm_reset` mask before determining the `c2p_freeze_atm` mask.
Previously, we only ever ran with the `false` option, which would mean that we artificially imposed the interface as it was given in the initial data.

GRHD: remove stale isadmissible check from cons2prim (!144)

It sems that atm we can get away with slightly negative tau values.
Furthermore, I think the logic for correcting tau wasn't really sound, and I could not retrace anymore how I obtained that fix.

GRHD: More fixes to the atmosphere freezing mask in 2d (!141)

- fixed two neighbor indices
- fixed next-neighbor check to also account for -1.0 (invalid) values