Changeset 17850

Timestamp:

05/14/14 09:24:46 (11 years ago)

Author:

tbretz

Message:

To avoid unification with each and every single pixel around the main Island, in case it has a long time spread, only idlands with more than one pixel are considered.

File:

• trunk/Mars/mimage/MImgCleanTime.cc (modified) (1 diff)

trunk/Mars/mimage/MImgCleanTime.cc

@@ -176 +176 @@
             continue;
 
+        const Island &I1 = fIslands[idx1];
+        const Island &I2 = fIslands[idx2];
+
         // FIXME: Put a limit on count? Put a limit on size?
-        if ((fIslands[idx1].min<fIslands[idx2].min || fIslands[idx1].min>fIslands[idx2].max) &&
-            (fIslands[idx1].max<fIslands[idx2].min || fIslands[idx1].max>fIslands[idx2].max) &&
-            (fIslands[idx2].min<fIslands[idx1].min || fIslands[idx2].min>fIslands[idx1].max) &&
-            (fIslands[idx2].max<fIslands[idx1].min || fIslands[idx2].max>fIslands[idx1].max))
+        // The ideal cut would be to allow that a single
+        // pixel still can connect two clusters
+        if (I1.count==1 || I1.count==1)
+            continue;
+
+        if (I1.max<I2.min || I2.max<I1.min)
             continue;
 
         // Combine both islands
-        fIslands[idx1] += fIslands[idx2];
+        I1 += I2;
 
         // Remove idx2 from the list