From 46cb9012c05eb8cfc76fac4c8e62f1ef695d5fc7 Mon Sep 17 00:00:00 2001 From: Markus Blatt Date: Wed, 11 Nov 2020 18:30:59 +0100 Subject: [PATCH] Do not filter connections on the schedule of the loadbalanced grid. It would remove perforated cells from wells that cross the local domain's border. That would make it impossible to figure out the first connection. In addition we would not be able to check that the connections exist (as rank 0 would have the complete information -> inconsistency). --- ebos/eclcpgridvanguard.hh | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/ebos/eclcpgridvanguard.hh b/ebos/eclcpgridvanguard.hh index 69c400c22..b8b448dfc 100644 --- a/ebos/eclcpgridvanguard.hh +++ b/ebos/eclcpgridvanguard.hh @@ -234,15 +234,10 @@ public: cartesianIndexMapper_.reset(); - if ( ! equilGrid_ ) - { - // for processes that do not hold the global grid we filter here using the local grid. - // If we would filter in filterConnection_ our partition would be empty and the connections of all - // wells would be removed. - ActiveGridCells activeCells(grid().logicalCartesianSize(), - grid().globalCell().data(), grid().size(0)); - this->schedule().filterConnections(activeCells); - } + // Calling Schedule::filterConnectins would remove any perforated + // cells that exist on other ranks in the case of distributed wells + // But we need them to figure out the first cell of a well (e.g. for + // pressure). Hence this is now skipped. Rank 0 had everything even before. } #endif