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

Fix RFT output for shut distributed wells #5824

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vkip
Copy link
Member

@vkip vkip commented Dec 20, 2024

and also for permanently inactive wells that are split across processors (partition method 3 issue only).

To see the issue for shut distributed wells, run attached example (
shut-distributed-wells-rft-issue.tar.gz) as:

mpirun -np 4 flow --external-partiton=part4.txt --allow-distributed-wells=true TEST-RFT-SHUT-DISTRIBUTED-WELL.DATA

and for inactive wells split across processors with partition method 3 (where root does not appear to contain all cells):

mpirun -np 3 flow --partition-method=3 --allow-distributed-wells=true TEST-RFT-INACTIVE-SPLIT-WELL-PARTMETHOD3.DATA

In these cases current master outputs 0 for RFT data for some connections of well INJE01/INJE02 respectively, whereas this PR ensures that all connections get actual values.

The solution here is a quick fix within the current framework, and should at some point be revised to avoid some global communication and adding the well data multiple times for shut distributed and split permanently inactive wells.

…lls split across processors (partition method 3 issue only).
@vkip
Copy link
Member Author

vkip commented Dec 20, 2024

jenkins build this please

@vkip vkip marked this pull request as ready for review December 20, 2024 11:33
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