wells: split injector connection flow between matrix and fracture - #7354
Draft
hnil wants to merge 2 commits into
Draft
wells: split injector connection flow between matrix and fracture#7354hnil wants to merge 2 commits into
hnil wants to merge 2 commits into
Conversation
New WellIndexFracture (own header): the extra connection factor a dynamically created fracture contributes to an existing perforation, with reference values reserved for pressure interpolation between fracture solves. WellInterfaceGeneric::addFracturePerforations() records/updates the contributions in place (storage allocated on first use; at most one fracture per cell assumed; unknown cells warned once per well/cell via OpmLog). closeCompletions() clears the fracture contribution together with the regular well index. getTw() adds the fracture factor to every conservation quantity before the D-factor adjustment. Wells without registered fracture perforations carry no extra state and take a single empty() check -- no behavior change. The pre-existing addPerforations() extension point is left untouched. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
New WellFractureFlowSplit.hpp: for injectors with registered fracture contributions, split each connection's water rate between matrix and fracture from the ratio of the regular connection transmissibility to the fracture contribution, filling the filtrate flow factor, the per-connection fracture water rate and the well's total fracture-routed rate. BlackoilWellModel gains a two-line, EnableMech-gated call before the filtration-model post-step update; initializeWellPerfData now receives the report step there as well. Wells without fracture contributions are skipped, non-mech builds compile the call out. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
hnil
added
the
manual:enhancement
hnil
marked this pull request as ready for review
August 28, 2026 13:57
hnil
marked this pull request as draft
August 28, 2026 14:50
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
1 participant
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.
For injectors with registered fracture contributions, splits each connection's water rate between matrix and fracture from the ratio of the regular connection transmissibility to the fracture contribution, filling the filtrate flow factor, the per-connection fracture water rate, and the well's total fracture-routed rate.
BlackoilWellModelgains a two-line call, gated onEnableMech, before the filtration-model post-step update;initializeWellPerfDatatakes the report step there as well. Wells without fracture contributions are skipped, and a non-mech build compiles the call out entirely.Stacked on #7353, which adds
WellIndexFractureand the contributions this reads. Until that merges the diff here includes it. The containers it fills came in with #7311.