diff options
author | Erik Schnetter <schnetter@gmail.com> | 2013-01-24 15:11:28 -0500 |
---|---|---|
committer | Erik Schnetter <schnetter@gmail.com> | 2013-01-24 15:11:28 -0500 |
commit | 87463f6c4d30ce3664c6d45fca1b73ce67a7bc4b (patch) | |
tree | 75aff006955681bb6eb13f7e6fda19d11ed3dd5c /Carpet | |
parent | 3146701470fbcc40afb7495065fd371bf3eec05d (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')
-rw-r--r-- | Carpet/CarpetRegrid2/src/property.cc | 5 |
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:: |