search

equinor / flownet

FlowNet - Data-Driven Reservoir Predictions
GNU General Public License v3.0
63 stars 29 forks source link

adaptations to allow for production temperature history matching #443

Closed plgbrts closed 2 years ago

plgbrts commented 2 years ago

The purpose of this PR is to allow for history matching production fluid temperatures. Main adjustement: 1) keyword WTPCHEA (temperature of producers) added for parsing (_config_parser.py) 2) extended phases with additional element temp. Also brine is allowed now to be specified under phases item in the yml config file. Thus the items 'oil', 'gas', 'water', 'disgas', 'vapoil', 'brine' and 'temp' can all be specified now under the phases keyword. (Note the term phases is not so appropriate anymore and should may be renamed) (_config_parser.py) 3) added WTPCHEA and WTICHEA to dataframe (from_flow.py) 4) helper function adedd to create WTEMP for injection temperature specification. The has_brine method commented out as brine keyword should now be specified via phases keyword in the config yml file (_schedule.py) 5) further adjustments to template files to enable WTEMP writing to schedule include file and to allow for production temperature observations

todo: 1) Enable specifying grid property THCONR ( total thermal conductivity) as static property for a flownet simulation. See issue #442 2) add a brine +thermal test case to flownet-testdata repository 3) update documentation and changelog

olelod commented 2 years ago

Nice job! There is a few formatting issues, I see. I'll fix them and merge later today.

plgbrts commented 2 years ago

Thanks Ole Petter for fixing the format issues! I see that merging is blocked because of the requested change of @fahaddilib but that is resolved by @olelod. Let me know if there is still something to be done on my side.