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

Compositional well #6042

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

Compositional well #6042

wants to merge 43 commits into from

Conversation

GitPaean
Copy link
Member

This is the beginning of the well modeling for the compositional simulation.

The current version only supports BHP control, single perforation, and two phase, while the general facility is there.

Will try to cleaning up and add some more functionality before it goes in.

It depends on the upstream PR OPM/opm-common#4334 .

it is not clear what are needed there yet.
and we can not have zero mole fractions, so a mimium 1.e-8 is used to
make the phase stability goes through.
based on the flash calculation results.
it looks like some re-organization of the variables and functions will
be needed.
It looks like most of pieces are there.
so far, the values look correct, not sure why it does not converge.
it looks like might converged for the injector.
following the new way in the master branch.
…lWellModel

not sure about the correctness yet.
but it looks like flowexp framework does not check the well convergence.
only outputting the bhp for the moment.
by adding extra well model fucntions to CompositionalWellModel
removing the hacking for testing.
also helps to jenkins compile
@GitPaean GitPaean force-pushed the compositional_well branch 2 times, most recently from 562c784 to af95308 Compare March 2, 2025 11:06
@GitPaean GitPaean force-pushed the compositional_well branch 2 times, most recently from 05c2e72 to 6c43502 Compare March 4, 2025 09:18
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