Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report Connection Level Fracturing Statistics to I/O Layer #5811

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

bska
Copy link
Member

@bska bska commented Dec 18, 2024

Populates the new data::Connection::fract data member from PR OPM/opm-common#4388.

To this end, introduce two helper classes to compute statistics of a running sample:

  • RunningStatistics calculates basic statics measures of a running sample
  • ConnFracStatistics aggregates those measures for connection level fracture pressure, fracture width, and flow rate

@bska
Copy link
Member Author

bska commented Dec 18, 2024

I am creating this PR in draft mode for two reasons

  1. It depends on, and contains, the earlier PR Add Infrastructure for Geomechanical Analysis #5801
  2. It needs the new data structure introduced in Add Connection Fracturing Statistics Summary Vectors opm-common#4388. The two PRs must therefore be reviewed and merged in concert.

@bska bska force-pushed the report-conn-level-fracture-statistics branch from 72baa03 to 2a0c294 Compare December 18, 2024 14:10
@bska bska force-pushed the report-conn-level-fracture-statistics branch 12 times, most recently from 7764476 to e03ad2d Compare December 20, 2024 14:47
@bska bska force-pushed the report-conn-level-fracture-statistics branch 9 times, most recently from 9dc2f96 to 20c4f36 Compare January 10, 2025 14:47
@bska bska force-pushed the report-conn-level-fracture-statistics branch 5 times, most recently from b6e1bbf to a074404 Compare January 14, 2025 08:51
@bska bska force-pushed the report-conn-level-fracture-statistics branch 23 times, most recently from 7a720ed to 12b9574 Compare February 6, 2025 12:28
@bska bska force-pushed the report-conn-level-fracture-statistics branch 3 times, most recently from 5f3a913 to ffb1d17 Compare February 7, 2025 13:33
bska added 3 commits February 7, 2025 15:45
In particular, split this function into distinct steps:

  1. Computing the linearised IJK index of each connection.
  2. Reporting multiplicative factors such as the CTF, the D factor,
     and the transmissibility multiplier due to rock compaction.
  3. Reporting pressures, flow rates, and productivity indices.
  4. Reporting filter cake values for injectors

While here, also reverse a condition in init() to enable "early
continue" and reduce levels of nesting and to reduce scope of some
objects.
In particular,

  * RunningStatistics calculates basic statics measures of a running
    sample
  * ConnFracStatistics aggregates those measures for connection
    level fracture pressure, fracture width, and flow rate
Populates the new data::Connection::fract data member.
@bska bska force-pushed the report-conn-level-fracture-statistics branch from ffb1d17 to 6d87714 Compare February 7, 2025 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant