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

Added generate_petro_jobs_for_field_update #259

Merged
merged 1 commit into from
Dec 10, 2024

Conversation

oddvarlia
Copy link
Contributor

@oddvarlia oddvarlia commented Nov 19, 2024

Purpose: RMS script to generate petrophysical property jobs for RMS.
Application: A preparation step to setup a workflow in FMU where both field parameters related to petrophysical variables and gaussian random fields for APS is to be updated as field parameters in ERT simultaneously,

Simplified description of a workflow in RMS where this is relevant:
Initial ensemble:
The typical workflow in RMS will be to simulate initial realizations of the petrophysical variable separately for each individual facies and conditioned to only blocked well observations of the petrophysical variables belonging to only one facies. To do this we want to setup RMS petrophysical jobs not using facies grid realization as input but use only one facies for all grid cells in each zone. The petrophysical variables should have identical distribution as was specified in the original petrophysical job for the given facies.

Updated ensemble: The updated petrophysical variables are imported from files updated by ERT.

A python job in RMS will use updated facies realization as filter to put the petrophysical variables for each facies into one grid parameter. The rest of the workflow in RMS will be unchanged.

The RMS python job defined by the script generate_petro_jobs_for_field_update.py will be used as a preparation script and generate RMS jobs for the petrophysical modelling of each individual facies. The input will be an existing petrophysical job using facies realization. The output will be a set of RMS petrophysical jobs, one per facies simulating the petrophysical variables conditioned to only one facies each. This will simplify and reduce manual work needed to setup the necessary jobs. Manual work will then be reduced to put the generated petrophysical jobs into the workflow instead of manually generating each of these jobs.

Closes: #263

@oddvarlia oddvarlia marked this pull request as draft November 19, 2024 11:59
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 3 times, most recently from 3e74ce1 to e75e2c6 Compare November 25, 2024 19:34
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 4 times, most recently from 3ecdf58 to b9e4507 Compare November 28, 2024 12:27
@oddvarlia oddvarlia marked this pull request as ready for review November 28, 2024 12:38
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 2 times, most recently from 8c96025 to 92e2274 Compare November 29, 2024 10:22
@oddvarlia oddvarlia marked this pull request as draft December 2, 2024 09:17
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 8 times, most recently from 52de7b9 to d185062 Compare December 2, 2024 11:21
@oddvarlia oddvarlia marked this pull request as ready for review December 2, 2024 11:24
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 2 times, most recently from 38433a9 to 797bd43 Compare December 4, 2024 14:15
pyproject.toml Outdated Show resolved Hide resolved
Copy link
Collaborator

@jcrivenaes jcrivenaes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some changes; do that first and I can take a second look afterwards

@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 6 times, most recently from fb21bf6 to 239ce9e Compare December 7, 2024 10:19
Copy link
Collaborator

@jcrivenaes jcrivenaes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In general OK. If you you have time, add docstrings to functions to explain short what it will do, and we now have a practice by using type-annotations and mypy checks. If not time now, you can add this as an issue for later work.

@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch 6 times, most recently from 9f0d547 to 875fed1 Compare December 10, 2024 09:50
Add testdata for generate_petro_jobs_for_field_update.
Add documentation
Update doc index
Added description of use
@oddvarlia oddvarlia force-pushed the create_rms_petrosim_jobs branch from 875fed1 to 883c269 Compare December 10, 2024 15:30
@oddvarlia oddvarlia merged commit 21780d4 into equinor:main Dec 10, 2024
16 checks passed
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.

Need a script to generate RMS petro jobs per facies from an original petro job
2 participants