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.

Sign up for GitHub

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

Jump to bottom

Kriging using well logs to constrain permeability and porosity parameter distributions in flow tubes #155

Merged
merged 42 commits into from
Oct 30, 2020
Merged

Kriging using well logs to constrain permeability and porosity parameter distributions in flow tubes #155

merged 42 commits into from
Oct 30, 2020

Conversation

wouterjdb
Copy link
Collaborator

@wouterjdb wouterjdb commented Aug 28, 2020 •
edited
Loading

This pull request adds the following functionality:

  • Loading of permeability and porosity logs from simulation data
  • Kriging of measured data
  • Assignment of value + uncertainty to individual tubes/connections based on kriging results.
  • Many small formatting changes due to an update in black.

Kriging example of log10(k) on Norne data

image

Contributor checklist

  • 🎉 This PR closes Test: Include log data into FlowNet #154
  • 📜 I have broken down my PR into the following tasks:
    • Load permeability and porosity data
    • Perform kriging
    • Assign permeability to tubes/connections
    • Assign porosity to tubes/connections
    • Perform tests, compare conditioned and non-conditioned prior ensemble for Norne case
  • 🤖 I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR.
  • 📖 I have considered adding a new entry in CHANGELOG.md.
  • 📚 I have considered updating the documentation.

@wouterjdb wouterjdb added the enhancement New feature or request label Aug 28, 2020
@wouterjdb wouterjdb self-assigned this Aug 28, 2020
wouterjdb and others added 18 commits August 31, 2020 11:02
@wouterjdb
Incorporate kriging results for permeability in ert setup
b165601
@wouterjdb
Changes due to new black version
ac1c0a0
@wouterjdb
Added CHANGELOG entry
9904d4e
@wouterjdb
Implemented on porosity as well
c3d3e1c
@wouterjdb
Made mypy happy
d09d93c
@wouterjdb
Branch 'master' of github.com:equinor/flownet
bf743e8
@wouterjdb
Merge branch 'master' into i154-use-log-data
ee9c814
@wouterjdb
Minor updates in docstring and coding style
e012e6c
@wouterjdb
Updates in code style due to update of Black
f9c76c8
@wouterjdb
Merge branch 'master' into i154-use-log-data
cd4f39c
@wouterjdb
Updated test_one_dimensional black
a569809
@wouterjdb
Merge branch 'master' of github.com:equinor/flownet
136cc6a
@wouterjdb
Solved merge conflicts
b38356d
@wouterjdb
Fixed too many eqlnum/satum bug
6c1592e
@wouterjdb
Black
b935161
@wouterjdb
Merge branch 'master' into i154-use-log-data
83884a2
@wouterjdb
Merge branch 'master' of github.com:equinor/flownet
d65a3d5
@wouterjdb
Merge branch 'master' into i154-use-log-data
4cd3208
@wouterjdb wouterjdb marked this pull request as ready for review October 5, 2020 08:42
olelod
olelod reviewed Oct 14, 2020
View reviewed changes
src/flownet/ahm/_run_ahm.py Outdated Show resolved Hide resolved
anders-kiaer
anders-kiaer requested changes Oct 15, 2020
View reviewed changes
Copy link
Collaborator

@anders-kiaer anders-kiaer left a comment

Choose a reason for hiding this comment

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

Nice work 🚀 Some minor comments only.

src/flownet/ahm/_run_ahm.py Show resolved Hide resolved
src/flownet/config_parser/_config_parser.py Outdated Show resolved Hide resolved
src/flownet/utils/kriging.py Outdated Show resolved Hide resolved
olelod
olelod approved these changes Oct 30, 2020
View reviewed changes
Copy link
Collaborator

@olelod olelod left a comment

Choose a reason for hiding this comment

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

LGTM! Nice job!

anders-kiaer
anders-kiaer approved these changes Oct 30, 2020
View reviewed changes
.github/workflows/flownet.yml Outdated Show resolved Hide resolved
@wouterjdb wouterjdb merged commit e0e189e into equinor:master Oct 30, 2020
@wouterjdb wouterjdb deleted the i154-use-log-data branch October 30, 2020 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anders-kiaer anders-kiaer anders-kiaer approved these changes

@olelod olelod olelod approved these changes

Assignees

@wouterjdb wouterjdb

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Test: Include log data into FlowNet
3 participants
@wouterjdb @anders-kiaer @olelod