Skip to content

Conversation

@michal-toth
Copy link
Contributor

The pressure system of the cprw solver does treats a distributed well as a collection of smaller wells - each rank has a smaller well. Just using a correct IDs for the well equations allowed us to get the equations to the correct row of the matrix on the coarsest level but the performance got worse. We hope that adding the overlap connections will improve the performance.

To treat a distributed well as one entity in the pressure system, we:

  1. Match IDs of well equations in the coarse system
  2. Extend the pattern of the matrix to accommodate the overlap
  3. Communicate the overlap entries from other ranks (this the the most difficult part)
  4. Change the diagonal term - consider weights of all cells, not just the local perforations

@michal-toth michal-toth added the manual:new-feature This is a new feature and should be described in the manual label Dec 19, 2025
@michal-toth
Copy link
Contributor Author

It needs some refactoring and rebasing to the current master. However, it will have to wait for the next year.

@michal-toth michal-toth force-pushed the bugfix/cprw-matchIDofDistributedWells branch from 791939c to 21c39ca Compare January 7, 2026 15:50
@michal-toth
Copy link
Contributor Author

jenkins build this serial please

@michal-toth
Copy link
Contributor Author

jenkins build this serial please

@akva2
Copy link
Member

akva2 commented Jan 8, 2026

cmake -DBUILD_FLOW_FLOAT_VARIANTS=1 to reproduce failure.

@michal-toth
Copy link
Contributor Author

jenkins build this serial please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

manual:new-feature This is a new feature and should be described in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants