opm-grid icon indicating copy to clipboard operation
opm-grid copied to clipboard

Enable DUNE grid tests

Open SoilRos opened this issue 5 months ago • 4 comments

For this to work it is additionally needed that

  • Entities return the "correct"[^1] number of faces (to count intersections in the tests).
  • The refinement does not throw. Instead, it returns false when it cannot make a refinement.
  • DUNE is at least version 2.10 due to https://gitlab.dune-project.org/core/dune-grid/-/merge_requests/734

[^1]: "Correct" here means with respect to the topology of a cube which is what the number of corners says. This is not correct for deformed entities though. Hence, this goes from wrong-wrong, to simply wrong.

SoilRos avatar Jul 31 '25 16:07 SoilRos

jenkins build this serial please

SoilRos avatar Jul 31 '25 16:07 SoilRos

jenkins build this serial please

SoilRos avatar Jul 31 '25 17:07 SoilRos

The CI does not seem to have DUNE 2.10, hence, this cannot be tested there. For now, I only can assure that this works locally, but it would be nice to have an automated test so that we can see if the test from DUNE are broken.

SoilRos avatar Jul 31 '25 17:07 SoilRos

I convert this PR to be a draft because dune 2.11 requires the grid to fulfill the grid concept check. And this will not work until

  • the geometry type is returned as a value #885,
  • the instantiations of template Codim<2>::Entity are disabled (see dune-grid!778), and
  • the grid correctly handles the entities that cannot be handled (this original PR has some things missing for the tests to pass). See #947.

SoilRos avatar Nov 13 '25 16:11 SoilRos