Merge pull request #5820 from lisajulia/feature/ms-wells-fix-communication-mistake

Fix really stupid communication mistake
This commit is contained in:
Arne Morten Kvarving 2024-12-20 08:19:22 +01:00 committed by GitHub
commit 572da1ca62
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 2 additions and 2 deletions

View File

@ -152,7 +152,7 @@ apply(const BVector& x, BVector& Ax) const
duneB_.mv(x, Bx);
if (this->pw_info_.communication().size() == 1) {
if (this->pw_info_.communication().size() > 1) {
// We need to communicate here to get the contributions from all segments
this->pw_info_.communication().sum(Bx.data(), Bx.size());
}

View File

@ -44,7 +44,7 @@ add_test_compare_parallel_simulation(CASENAME msw-simple
ABS_TOL 1e4 # the absolute tolerance is pretty high here, yet in this case, we are only interested in the relative tolerance
REL_TOL 1e-5
MPI_PROCS 4
TEST_ARGS --solver-max-time-step-in-days=10 --allow-distributed-wells=true)
TEST_ARGS --solver-max-time-step-in-days=15 --allow-distributed-wells=true)
add_test_compare_parallel_simulation(CASENAME msw-3d
FILENAME MSW-3D # this file contains one Multisegment well with branches that is distributed across several processes