nsamples check needed for zero-suppressed data, for DA classes and RawUtils

diff --git a/EMCAL/AliCaloCalibPedestal.cxx b/EMCAL/AliCaloCalibPedestal.cxx
index 264e427062c4971c8ecaeb98a80daaede660271d..d69d6506c8ff744980ae5c0f284ae4d3425dfe00 100644 (file)
--- a/EMCAL/AliCaloCalibPedestal.cxx
+++ b/EMCAL/AliCaloCalibPedestal.cxx
@@ -352,7 +352,8 @@ Bool_t AliCaloCalibPedestal::ProcessEvent(AliCaloRawStreamV3 *in)
 
       // counters
       int max = AliEMCALGeoParams::fgkSampleMin, min = AliEMCALGeoParams::fgkSampleMax; // min and max sample values
-      
+      int nsamples = 0;
+
       // for the pedestal calculation
       int sampleSum = 0; // sum of samples
       int squaredSampleSum = 0; // sum of samples squared
@@ -363,6 +364,7 @@ Bool_t AliCaloCalibPedestal::ProcessEvent(AliCaloRawStreamV3 *in)
       while (in->NextBunch()) {
        const UShort_t *sig = in->GetSignals();
        startBin = in->GetStartTimeBin();
+       nsamples += in->GetBunchLength();
        for (i = 0; i < in->GetBunchLength(); i++) {
          sample = sig[i];
          time = startBin--;
@@ -382,6 +384,8 @@ Bool_t AliCaloCalibPedestal::ProcessEvent(AliCaloRawStreamV3 *in)
        } // loop over samples in bunch
       } // loop over bunches
 
+      if (nsamples > 0) { // this check is needed for when we have zero-supp. on, but not sparse readout
+
       // calculate pedesstal estimate: mean of possibly selected samples
       if (nSum > 0) {
        mean = sampleSum / (1.0 * nSum);
@@ -434,7 +438,7 @@ Bool_t AliCaloCalibPedestal::ProcessEvent(AliCaloRawStreamV3 *in)
        }
       }//end if valid gain
 
-    
+      } // nsamples>0 check, some data found for this channel; not only trailer/header
     }// end while over channel   
   }//end while over DDL's, of input stream 
 

diff --git a/EMCAL/AliCaloCalibSignal.cxx b/EMCAL/AliCaloCalibSignal.cxx
index 446733cead2eacef051d5e7a7579f93cd0c28bed..e83ee0c4d775aa6a81c7ba1577f507c23119f08a 100644 (file)
--- a/EMCAL/AliCaloCalibSignal.cxx
+++ b/EMCAL/AliCaloCalibSignal.cxx
@@ -364,9 +364,11 @@ Bool_t AliCaloCalibSignal::ProcessEvent(AliCaloRawStreamV3 *in, AliRawEventHeade
 
       // counters
       int max = AliEMCALGeoParams::fgkSampleMin, min = AliEMCALGeoParams::fgkSampleMax; // min and max sample values
-      
+      int nsamples = 0;
+
       while (in->NextBunch()) {
        const UShort_t *sig = in->GetSignals();
+       nsamples += in->GetBunchLength();
        for (Int_t i = 0; i < in->GetBunchLength(); i++) {
          sample = sig[i];
 
@@ -377,6 +379,8 @@ Bool_t AliCaloCalibSignal::ProcessEvent(AliCaloRawStreamV3 *in, AliRawEventHeade
        } // loop over samples in bunch
       } // loop over bunches
 
+      if (nsamples > 0) { // this check is needed for when we have zero-supp. on, but not sparse readout
+
       gain = -1; // init to not valid value
       //If we're here then we're done with this tower
       if ( in->IsLowGain() ) {
@@ -417,7 +421,8 @@ Bool_t AliCaloCalibSignal::ProcessEvent(AliCaloRawStreamV3 *in, AliRawEventHeade
        RefNum = GetRefNum(arrayPos, in->GetColumn(), gain); 
        LEDAmpVal[RefNum] = max - min;
       } // end of LED ref
-      
+
+      } // nsamples>0 check, some data found for this channel; not only trailer/header      
     } // end while over channel 
    
   }//end while over DDL's, of input stream

diff --git a/EMCAL/AliEMCALCalibHistoProducer.cxx b/EMCAL/AliEMCALCalibHistoProducer.cxx
index e779b2f5e9c578b7fea4cd3545ee072c41b823d6..4522fb4b8729f33174f18ffed885e16a9101af33 100644 (file)
--- a/EMCAL/AliEMCALCalibHistoProducer.cxx
+++ b/EMCAL/AliEMCALCalibHistoProducer.cxx
@@ -201,6 +201,19 @@ void AliEMCALCalibHistoProducer::Run()
 
        if(fSMInstalled[in.GetModule()]==kFALSE) continue;
 
+       // loop over samples
+       int nsamples = 0;
+       Int_t maxSample = 0;
+       while (in.NextBunch()) {
+         const UShort_t *sig = in.GetSignals();
+         nsamples += in.GetBunchLength();
+         for (Int_t i = 0; i < in.GetBunchLength(); i++) {
+           if (sig[i] > maxSample) maxSample = sig[i];
+         }
+       } // bunches
+
+       if (nsamples > 0) { // this check is needed for when we have zero-supp. on, but not sparse readout
+
        // indices
        Int_t mod = in.GetModule();
        Int_t col = in.GetColumn();
@@ -217,21 +230,15 @@ void AliEMCALCalibHistoProducer::Run()
          sprintf(hname,"mod%d",mod);
          fAmpProf[mod] = new TProfile(hname,hname,nEvtBins,0.,nEvtBins);
        }
-       
-       // loop over samples
-       Int_t maxSample = 0;
-       while (in.NextBunch()) {
-         const UShort_t *sig = in.GetSignals();
-         for (Int_t i = 0; i < in.GetBunchLength(); i++) {
-           if (sig[i] > maxSample) maxSample = sig[i];
-         }
-       } // bunches
-       
+               
        //Fill histogram/profile 
        if(HighGainFlag) {
          fAmpHisto[mod][col][row]->Fill(maxSample);
          fAmpProf[mod]->Fill(evtbin, maxSample);
        }
+
+      } // nsamples>0 check, some data found for this channel; not only trailer/header
+
       } // channels
     } // DDL's
 

diff --git a/EMCAL/AliEMCALRawUtils.cxx b/EMCAL/AliEMCALRawUtils.cxx
index d93ff4afa9ef8e39d0bcb0926f59fa4c42d087ac..23fb4dd45f9ca60fc43be8a3630ce417e1a1d15c 100644 (file)
--- a/EMCAL/AliEMCALRawUtils.cxx
+++ b/EMCAL/AliEMCALRawUtils.cxx
@@ -331,10 +331,6 @@ void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
   while (in.NextDDL()) {
     while (in.NextChannel()) {
       
-      id =  fGeom->GetAbsCellIdFromCellIndexes(in.GetModule(), in.GetRow(), in.GetColumn()) ;
-      caloFlag = in.GetCaloFlag();
-      lowGain = in.IsLowGain();
-
       // There can be zero-suppression in the raw data, 
       // so set up the TGraph in advance
       for (i=0; i < GetRawFormatTimeBins(); i++) {
@@ -342,7 +338,7 @@ void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
       }
                
       Int_t maxTime = 0;
-
+      int nsamples = 0;
       while (in.NextBunch()) {
        const UShort_t *sig = in.GetSignals();
        startBin = in.GetStartTimeBin();
@@ -355,13 +351,19 @@ void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
          AliWarning(Form("Invalid time bin %d",maxTime));
          maxTime = GetRawFormatTimeBins();
        }
-
+       nsamples += in.GetBunchLength();
        for (i = 0; i < in.GetBunchLength(); i++) {
          time = startBin--;
          gSig->SetPoint(time, time, sig[i]) ;
        }
       } // loop over bunches
     
+      if (nsamples > 0) { // this check is needed for when we have zero-supp. on, but not sparse readout
+
+      id =  fGeom->GetAbsCellIdFromCellIndexes(in.GetModule(), in.GetRow(), in.GetColumn()) ;
+      caloFlag = in.GetCaloFlag();
+      lowGain = in.IsLowGain();
+
       gSig->Set(maxTime+1);
       FitRaw(gSig, signalF, amp, time) ; 
     
@@ -382,6 +384,7 @@ void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
       // Reset starting parameters for fit function
       signalF->SetParameters(10.,0.,fTau,fOrder,5.); //reset all defaults just to be safe
 
+      } // nsamples>0 check, some data found for this channel; not only trailer/header
    } // end while over channel   
   } //end while over DDL's, of input stream