Skip to content

Add support for experimental alternative of Dune::BCRSMatrix#7124

Open
SoilRos wants to merge 1 commit into
OPM:masterfrom
SoilRos:feature/ibcrsmatrix
Open

Add support for experimental alternative of Dune::BCRSMatrix#7124
SoilRos wants to merge 1 commit into
OPM:masterfrom
SoilRos:feature/ibcrsmatrix

Conversation

@SoilRos

@SoilRos SoilRos commented Jun 3, 2026

Copy link
Copy Markdown
Member

Update matrix type definitions across the codebase to use Dune::IBCRSMatrix instead of the classic Dune::BCRSMatrix when using experimental mode (see dune-istl !646). This transition affects well models, thermal and tracer models, and the linear solver backends, including specialized support for external solvers like UMFPack and SuperLU. The speedup from using the experimental mode is expected to be around 10% on distributed workloads (either MPI or multiple independent simulations working simultaneously).

  • For the experimental mode to work, dune-istl must use the branch containing the changes in dune-istl !646.
  • This is kept as experimental because the merging into dune may take quite some time until it's available on the releases.

@SoilRos SoilRos added enhancement manual:irrelevant This PR is a minor fix and should not appear in the manual labels Jun 3, 2026
@SoilRos

SoilRos commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

@akva2 I would like to eventually test the experimental mode in jenkins, but of course, ISTL would need to be pulled from the right branch. With markus, we thought that we could update the OPM branch from dune-istl (releases/opm) to make this feature available for OPM users. Do you have ideas on how could we make this work in the CI?

@SoilRos

SoilRos commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@akva2

akva2 commented Jun 3, 2026

Copy link
Copy Markdown
Member

you find the opm ci docker container here, https://github.com/OPM/opm-utilities/tree/master/opm-ci-builder

it currently pulls dune-istl from the ppa, we have to build it instead. we already build from source for the serial and debug-iter enabled configurations, so building from source for the "vanilla" is very doable.

if it becomes a "hard" dependency i can build dune packages in the ppa, but i'd prefer to not do that.

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from f5d49fc to 758e6f7 Compare June 4, 2026 16:11
@SoilRos

SoilRos commented Jun 4, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from c2502e5 to 98d6b2f Compare June 26, 2026 14:11
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from 98d6b2f to 6e9a578 Compare June 26, 2026 14:29
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

1 similar comment
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch 2 times, most recently from 3497317 to 1508f96 Compare June 26, 2026 14:52
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from 1508f96 to 154a152 Compare June 26, 2026 15:05
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from 154a152 to 7e97cb6 Compare June 26, 2026 15:34
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from 7e97cb6 to ccf9213 Compare June 26, 2026 15:39
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

(sorry for the noice with jenkins, I do not have a GPU locally to reproduce the erros)

@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from ccf9213 to d2c4911 Compare June 26, 2026 15:57
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

Update matrix type definitions across the codebase to use Dune::IBCRSMatrix instead of the older Dune::BCRSMatrix when using experimental mode. This transition affects well models, thermal and tracer models, and the linear solver backends, including specialized support for external solvers like UMFPack and SuperLU.
@SoilRos SoilRos force-pushed the feature/ibcrsmatrix branch from d2c4911 to 28d613e Compare June 26, 2026 17:14
@SoilRos

SoilRos commented Jun 26, 2026

Copy link
Copy Markdown
Member Author

jenkins build this please

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

Labels

enhancement manual:irrelevant This PR is a minor fix and should not appear in the manual

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants