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

[fix] reflect inactive cells due to MINPV/PINCH in Schedule::completed_cells

Open blattms opened this issue 1 year ago • 7 comments

Previously only cells were marked as inactive that were inactive before MINPV/PINCH was applied and those inactive due to MINPV/PINCH might still have been active. This presented a problem whenever ACTIONX was run as this would recreate the schedule using the information in Schedule::completed_cells. We experienced problems because wells suddenly had completions in active cells.

With this commit we use filterConnections (called after MINPV/PINCH processing and creation of the grid in opm-simulators) to also mark cells in Schedule::completed_cells as inactive that became inactive due to MINPV/PINCH or small volumes.

blattms avatar Mar 07 '24 16:03 blattms

jenkins build this please

blattms avatar Mar 07 '24 16:03 blattms

And all the failure messages are related to

flow: /var/lib/jenkins/workspace/opm-common-PR-builder/opm/input/eclipse/Schedule/Schedule.cpp:1327: void Opm::Schedule::filterConnections(const Opm::ActiveGridCells&): Assertion `cell.props' failed.

GitPaean avatar Mar 07 '24 20:03 GitPaean

Sorry for the delay. Should be fixed now.

blattms avatar Mar 15 '24 13:03 blattms

jenkins build this please

blattms avatar Mar 15 '24 13:03 blattms

Thanks. I will perform a test on the field case before reviewing the PR.

GitPaean avatar Mar 15 '24 22:03 GitPaean

I made a serial run and it did not look like it fixed the problem. I might made mistake, we should confirm that it did not fix first.

GitPaean avatar Mar 17 '24 12:03 GitPaean