aboutsummaryrefslogtreecommitdiff
path: root/Carpet/CarpetRegrid2
diff options
context:
space:
mode:
authorErik Schnetter <schnetter@gmail.com>2013-01-24 15:11:28 -0500
committerErik Schnetter <schnetter@gmail.com>2013-01-24 15:11:28 -0500
commit87463f6c4d30ce3664c6d45fca1b73ce67a7bc4b (patch)
tree75aff006955681bb6eb13f7e6fda19d11ed3dd5c /Carpet/CarpetRegrid2
parent3146701470fbcc40afb7495065fd371bf3eec05d (diff)
CarpetRegrid2: Correct min_fraction
A condition was reversed, inverting the meaning of min_fraction, possibly leading to quite inefficient grid structures.
Diffstat (limited to 'Carpet/CarpetRegrid2')
-rw-r--r--Carpet/CarpetRegrid2/src/property.cc5
1 files changed, 3 insertions, 2 deletions
diff --git a/Carpet/CarpetRegrid2/src/property.cc b/Carpet/CarpetRegrid2/src/property.cc
index a568e82d9..e7df4ea24 100644
--- a/Carpet/CarpetRegrid2/src/property.cc
+++ b/Carpet/CarpetRegrid2/src/property.cc
@@ -202,9 +202,10 @@ namespace CarpetRegrid2 {
CCTK_REAL const combined_size =
static_cast <CCTK_REAL> (combined.size());
- // Would a single bbox be efficient enough?
+ // Is the current setup simple enough? (It is "simple enough" if
+ // using a single bbox would be too inefficient.)
// TODO: Check this also for pairs of regions
- return min_fraction * combined_size <= regions_size;
+ return min_fraction * combined_size > regions_size;
}
void combine_regions::