Repository navigation
Make the single-connection water cases easier and add ORAT and GRAT cases - #1640
Merged
Merged
Conversation
GitPaean
marked this pull request as draft
October 5, 2026 10:03
Keep the wells off a second limit, where the small cells make a control switch unstable: the LRAT injector runs at its pressure limit from the start, the WATERFLOOD and WATER_TO_GAS injectors get a pressure limit they do not reach, and WRAT ends at day 20, before its producer falls back to its pressure limit.
The water flood producer holds an oil rate until water breaks through, and the CO2 flood producer holds a gas rate while it lifts mobile water.
GitPaean
force-pushed
the
easier-single-connection-well-cases
branch
from
October 6, 2026 09:20
630c23a to
2c98ed1
Compare
Member
Author
|
None of the cases are in the regression running in the master branch yet, so the PR is okay to go in by itself. |
bska
approved these changes
Oct 6, 2026
bska
left a comment
bska
left a comment
Member
There was a problem hiding this comment.
Looks good to me and I'll merge into master.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up to #1621 for OPM/opm-simulators#7442.
The well model in that PR is at an early stage, so the cases should not be too hard to solve yet. The cells are small (1 m³ of pore volume each), and a well sitting on the boundary between a rate control and its bottom-hole pressure limit can switch back and forth within a time step. The first commit keeps the wells off a second limit:
The second commit adds two cases, so that the oil, water, gas and liquid rate controls of a producer are all covered:
SIMPLE_COMP_WATER_ORAT: the water flood producer holds an oil rate until water breaks through around day 75, when its bottom-hole pressure limit takes over;SIMPLE_COMP_WATER_GRAT: the CO2 flood producer holds a gas rate for all 50 days while it lifts mobile water.The simulator PR registers the two new cases next to the existing four. CI will generate their reference results.