Print

Print


Author: [log in to unmask]
Date: Thu Oct 20 09:53:19 2016
New Revision: 4518

Log:
removed a bug i had accidentally introduced.  under certain circumstances, Track.getCharge() returns 0.  Perhaps it is not being set somewhere?  use TrackUtils.getCharge(Track track) instead.  

Modified:
    java/trunk/recon/src/main/java/org/hps/recon/utils/TrackClusterMatcher.java

Modified: java/trunk/recon/src/main/java/org/hps/recon/utils/TrackClusterMatcher.java
 =============================================================================
--- java/trunk/recon/src/main/java/org/hps/recon/utils/TrackClusterMatcher.java	(original)
+++ java/trunk/recon/src/main/java/org/hps/recon/utils/TrackClusterMatcher.java	Thu Oct 20 09:53:19 2016
@@ -288,7 +288,8 @@
         
         // choose which parameterization of mean and sigma to use:
         double dxMean[],dyMean[],dxSigm[],dySigm[];
-        if (track.getCharge()>0) {
+        int charge = TrackUtils.getCharge(track);
+        if (charge>0) {
             if (isTopTrack) {
                 dxMean = isGBL ? dxMeanTopPosiGBL : dxMeanTopPosiSeed;
                 dxSigm = isGBL ? dxSigmTopPosiGBL : dxSigmTopPosiSeed;
@@ -302,7 +303,7 @@
                 dySigm = isGBL ? dySigmBotPosiGBL : dySigmBotPosiSeed;
             }
         }
-        else if (track.getCharge()<0) {
+        else if (charge<0) {
             if (isTopTrack) {
                 dxMean = isGBL ? dxMeanTopElecGBL : dxMeanTopElecSeed;
                 dxSigm = isGBL ? dxSigmTopElecGBL : dxSigmTopElecSeed;