OPM
Skip NNCs created by PINCH if one of the cells is collapsed.
Collapsed here means that the points of the top and lower plane on the same pillar coincide. Later those cells will be treated as inactive in preprocess.c and will not be part of the grid. when we create the grid topology. They would present numerical problems later anyway.
Note that this means that these cells (collapsed but PORV larger than MINPVV) will now be present a barrier preventing creation of a PINCH NNC.
Fixes the assertion:
flow: ./opm/grid/cpgrid/EntityRep.hpp:121: void Dune::cpgrid::EntityRep<codim>::setValue(int, bool) [with int codim = 0]: Assertion `index_arg >= 0' failed.
Closes #505
With this the collapsed cell will prevent creation of an NNC. It might turn out that this is not what we want, but it is certainly way better than a model aborting in an assert. Hence I think this should be merged but we should compare behavior with others.
I'd really like to see some unit tests for this in order to assess the behaviour.
Hei, just a frienldy reminder about the schedule for the release:
Bug fixing deadline for RC2: Thursday the 23rd of October, 6pm CET.
If we are backporting this, then we need to make a RC3.
