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

RPTRST switch FLOWS #16

Open
dr-robertk opened this issue Apr 18, 2016 · 8 comments
Open

RPTRST switch FLOWS #16

dr-robertk opened this issue Apr 18, 2016 · 8 comments

Comments

@dr-robertk
Copy link
Member

Within the IOR Centre there is the desire to couple OPM with IORSim through restart files (as a first step). IORSim relies on the output of inter block flow fields. The output of these values is done by the switch FLOWS as part of the RPTRST keyword.

What is the best way to add this functionality? The flow fields are obviously available in OPM and the output of face values should also be available.

@joakim-hove
Copy link
Member

joakim-hove commented Apr 18, 2016

  1. Update the IOConfig object: https://github.com/OPM/opm-parser/blob/master/opm/parser/eclipse/EclipseState/IOConfig/IOConfig.hpp to internalize the FLOWS mnemonic from the RPTRST keyword.
  2. Then I would say the EclipseWriter class here in opm-output should be updated by dumping the relevant fields in the restart ouptut. If you send me an example of a Eclipse generated restart file with the necessary content I can probably advice a bit further.

@atgeirr
Copy link
Member

atgeirr commented Apr 18, 2016

My question is how are these data structured in the output file. While we have a face/connection (if we include NNCs) ordering, it would not be the same as that used by i.e. Eclipse. Are these data structured in a logical cartesian manner for example?

@joakim-hove
Copy link
Member

Are these data structured in a logical cartesian manner for example?

I don't know; but by staring at a file generated by Eclipse we might be able to infer it? I we have the source to the IORSim application we might be able to learn something from that as well.

@atgeirr
Copy link
Member

atgeirr commented Apr 18, 2016

I we have the source to the IORSim application we might be able to learn something from that as well

IORSim is closed source (correct?). However since it is made by the IOR center we should be able to discuss this with the authors.

@dr-robertk
Copy link
Member Author

I'm not sure about the open- or closedness of IORSim. But there is room for discussion next week at the IOR conference. For now I have no idea how the data is organized. But I'll double check.

@atgeirr
Copy link
Member

atgeirr commented May 25, 2016

For flow diagnostics purposes we also need FLOWRES (giving interblock reservoir volume flows), just adding it to this issue as a future reminder.

@EnricoScantamburlo
Copy link

EnricoScantamburlo commented Jun 22, 2017

Hello,
we use 3DSL to calculate streamlines from ECLIPSE output

3DSL needs one of these settings
RPTRST
BASIC=4
FLORES /

or

RPTRST
BASIC=6
FLOWS DENS DENO DENG RS /

or

RPTRST
BASIC=6
FLOOIL FLOGAS FLOWAT DENW DENO DENG RS
/

I already saw that BASIC=6 is not supported, but it is not a big problem, adding support to the other keywords would be really appreciated.

@atgeirr
Copy link
Member

atgeirr commented Jun 22, 2017

There has been some effort related to this (although not finished), see the discussion in OPM/opm-simulators#1039.

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

No branches or pull requests

4 participants