Add `HAVE_STDBOOL_H` define for Alberta when `stdbool.h` is available

See merge request !769

Nils Friess authored 1 month ago and Santiago Ospina De Los Ríos committed 3 days ago

In older versions of Alberta, it tries to fix the issues around not having
proper bool in C by redefining bool as _Bool or vice versa, depending
on the language that it is compiled with (C or C++). Clang's stdbool header
does essentially the same and we end up with `typedef bool bool` which is a
compile error. This whole bool/_Bool logic is inside an ifdef in
Albert that is active if HAVE_STDBOOL_H is not defined, in which case
it just includes stdbool.h and uses that. So we can check during
CMake config if that header is available and set the flag accordingly.

Fix some issues with the VirtualizedGF in the python bindings

See merge request !774

Fix issues in python bindings due to changes in dune-vtk

See merge request !773

UGGrid: Remove the rudimentary saveState/loadState implementations

See merge request !770

The implementations of these methods looked like they do something
reasonable, but in fact they don't: They do call corresponding
UGGrid methods, but those methods never worked (in Dune!).

After giving it some thought it is now my opinion that the UGGrid
saveState/loadState functionality is not worth the time needed
to understand it and get it to work.  A modern snapshotting code
should be written from scratch in any case, using existing modern
standards as much as possible.

Therefore, this patch removes the dummy calls to SaveMultiGrid
and LoadMultiGrid.

Add an overload of printGrid that does not need an MPIHelper

See merge request !768

Set up default include directories on module target

See merge request !763

[ug] remove definition of D_SMALL

See merge request !766

The defintion fixes a missing include within dune-uggrid.

Set the correct FieldInfo size in vtktest

Closes #191

See merge request !765

[UGGrid] Use std::size_t as loop counter

See merge request !764

Recently, dune-uggrid has started to replace C arrays by FieldVectors
for storing point positions.  Therefore copying such positions
in dune-grid can now use the assignment operator of FieldVector
instead of hand-coded loops.  This makes the code shorter.  It also
saves a few unnecessary calls to UGGridRenumberer.

This fixes a few signed/unsigned warnings

[uggrid] Simplify domain creation

See merge request !761

[bugfix][HierarchicSearch] private --> protected.

See merge request !762

Recent versions of CreateBoundaryValueProblem simply allocated
an object of type STD_BVP (aka std_BoundaryValueProblem).

Therefore, don't call the method, and simply call 'new' instead.

Rather than by name.

All it does is a single assignment.  Instead of calling the method,
do the assignment directly.

This is one further reduction of the complexity of UGGrid setup.

It is not used in dune-uggrid anymore.

'void' is part of an old abstract interface, which only creates
unnecessary complexity for us.

Remove nobindings ci jobs since already in the default set

See merge request !758

Use less allocations on UGGrid intersection geometries

See merge request !753

Avoid storing coordinates on the heap by exploiting the fact that we only have
cubes and simplices up to 3D. When the number of corners is the same use an std::array, otherwise use a ReservedVector. This avoid many allocations during grid iteration.

The common case of an intersection is when it has only one face. For this case we use a variant that either holds one face, a vector of faces, or none. This way the common case is stored in the stack whereas the general case is still possible but needs dynamic allocation

There is no inherent need to store geometries in the heap. This changes the data layout to store them in the stack. Incidentally, this also moves coordinates into the geometries to avoid an extra allocation on the vector of coordinates.