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;