OilfieldToolsNet/opm-simulators
4
0
Fork 0
mirror of https://github.com/OPM/opm-simulators.git synced 2025-02-13 19:55:35 -06:00
Code Issues Packages Projects Releases Wiki Activity

[fix] Do parallel sums with int instead of bool.

Browse Source 
Using bool here is at least frowned upon. To be honest, I have no idea
what happens underneath here if we pass a bool. In contrast to other
pod types we do not associate it with a builtin type of MPI (not even
sure what to use). Hence we probably create a custom type for sending
and receiving. That should work. But I have no idea what will be used
for summation.

BTW: I am debugging a case that previously crashed and now suddenly
works and this seems to be the only relevant change I made in the
meantime.
This commit is contained in:
Markus Blatt 2023-11-22 16:50:57 +01:00
parent c559de51cf
commit 6a866ac8d2
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
opm/simulators/wells/BlackoilWellModel_impl.hpp
View File

@ -1924,7 +1924,7 @@ namespace Opm {
}
}
changed_well_to_group = comm.sum(changed_well_to_group);
changed_well_to_group = comm.sum(static_cast<int>(changed_well_to_group));
if (changed_well_to_group) {
updateAndCommunicate(episodeIdx, iterationIdx, deferred_logger);
changed_well_group = true;
@ -1939,7 +1939,7 @@ namespace Opm {
changed_well_individual = changed_well || changed_well_individual;
}
}
changed_well_individual = comm.sum(changed_well_individual);
changed_well_individual = comm.sum(static_cast<int>(changed_well_individual));
if (changed_well_individual) {
updateAndCommunicate(episodeIdx, iterationIdx, deferred_logger);
changed_well_group = true;