Check for interprocess facets when assembling interior facet integrals

[jorgensd]

A long-standing issue in DOLFINx is that we do not warn the user that using GhostMode.none with interior facet integrals can result in undefined behavior.
This includes:
#2556
and an attempt of addressing this was done in: #2574, but was rejected due to its non-trivial computations.

We do have interprocess_facets, which is a bi-product of create_entities which we can use to check that we have sufficient information when packing integration entities:

dolfinx/cpp/dolfinx/mesh/Topology.h

Line 196 in d5f56d6

const std::vector<std::int32_t>& interprocess_facets() const;

interprocess_facets are facets owned by the current process that is shared with other processes.

[chrisrichardson]

dolfinx/cpp/dolfinx/mesh/topologycomputation.cpp

Line 98 in 292c623

/// @returns Local indices and index map

Looks like it also needs updating docs

[garth-wells]

Not a change is this PR, but what are "interprocess_facets"? The mesh::Topology::interprocess_facets docstring is unclear.

[garth-wells]

Not a change is this PR, but what are "interprocess_facets"? The mesh::Topology::interprocess_facets docstring is unclear.

I've improved the docstring.

[garth-wells]

Nice.