Print

Print


Author: [log in to unmask]
Date: Thu Mar  5 10:05:31 2015
New Revision: 2262

Log:
Fixed an issue where clustering in verbose mode was ordering the hits incorrectly for cluster reconstruction.

Modified:
    java/trunk/ecal-recon/src/main/java/org/hps/recon/ecal/cluster/GTPOnlineClusterer.java

Modified: java/trunk/ecal-recon/src/main/java/org/hps/recon/ecal/cluster/GTPOnlineClusterer.java
 =============================================================================
--- java/trunk/ecal-recon/src/main/java/org/hps/recon/ecal/cluster/GTPOnlineClusterer.java	(original)
+++ java/trunk/ecal-recon/src/main/java/org/hps/recon/ecal/cluster/GTPOnlineClusterer.java	Thu Mar  5 10:05:31 2015
@@ -94,21 +94,7 @@
     		System.out.println("======================================================================");
     		System.out.println("=== GTP Readout Clusterer ============================================");
     		System.out.println("======================================================================");
-    	}
-    	
-        // Track the valid clusters.
-        List<Cluster> clusterList = new ArrayList<Cluster>();
-        
-        // Sort the hits by time in reverse order.
-        Collections.sort(hitList, new Comparator<CalorimeterHit>() {
-            @Override
-            public int compare(CalorimeterHit firstHit, CalorimeterHit secondHit) {
-                return Double.compare(secondHit.getTime(), firstHit.getTime());
-            }
-        }); 
-        
-        // VERBOSE :: Print the hit information.
-        if(verbose) {
+    		
         	Collections.sort(hitList, new Comparator<CalorimeterHit>() {
 				@Override
 				public int compare(CalorimeterHit firstHit, CalorimeterHit secondHit) {
@@ -131,6 +117,17 @@
             }
             System.out.println();
         }
+    	
+        // Track the valid clusters.
+        List<Cluster> clusterList = new ArrayList<Cluster>();
+        
+        // Sort the hits by time in reverse order.
+        Collections.sort(hitList, new Comparator<CalorimeterHit>() {
+            @Override
+            public int compare(CalorimeterHit firstHit, CalorimeterHit secondHit) {
+                return Double.compare(secondHit.getTime(), firstHit.getTime());
+            }
+        });
         
         // A seed hit is a hit that is the largest both within its
         // spatial range (+/- 1 in the ix and iy direction) and