]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - EMCAL/AliEMCALRawUtils.cxx
The total mult in V0 became float number.
[u/mrichter/AliRoot.git] / EMCAL / AliEMCALRawUtils.cxx
index 83204d2d37d3f12463a9c2ebdab57a2b97b84f39..d93ff4afa9ef8e39d0bcb0926f59fa4c42d087ac 100644 (file)
  **************************************************************************/
 
 /* $Id$ */
-/* History of cvs commits:
- *
- * $Log$
- * Revision 1.4  2007/10/31 17:15:24  mvl
- * Fixed bug in raw data unpacking; Added pedestal to signal fit; Added logic to deal with high/low gain
- *
- * Revision 1.3  2007/09/27 08:36:46  mvl
- * More robust setting of fit range in FitRawSignal (P. Hristov)
- *
- * Revision 1.2  2007/09/03 20:55:35  jklay
- * EMCAL e-by-e reconstruction methods from Cvetan
- *
- * Revision 1.1  2007/03/17 19:56:38  mvl
- * Moved signal shape routines from AliEMCAL to separate class AliEMCALRawUtils to streamline raw data reconstruction code.
- * */
 
+//_________________________________________________________________________
+//  Utility Class for handling Raw data
+//  Does all transitions from Digits to Raw and vice versa, 
+//  for simu and reconstruction
+//
+//  Note: the current version is still simplified. Only 
+//    one raw signal per digit is generated; either high-gain or low-gain
+//    Need to add concurrent high and low-gain info in the future
+//    No pedestal is added to the raw signal.
 //*-- Author: Marco van Leeuwen (LBL)
-#include "AliEMCALRawUtils.h"
 
+#include "AliEMCALRawUtils.h"
+  
 #include "TF1.h"
 #include "TGraph.h"
-#include "TSystem.h"
-
-#include "AliLog.h"
+class TSystem;
+  
+class AliLog;
+#include "AliRun.h"
 #include "AliRunLoader.h"
-#include "AliCaloAltroMapping.h"
+class AliCaloAltroMapping;
 #include "AliAltroBuffer.h"
 #include "AliRawReader.h"
-#include "AliCaloRawStream.h"
+#include "AliCaloRawStreamV3.h"
 #include "AliDAQ.h"
-
+  
+#include "AliEMCALRecParam.h"
 #include "AliEMCALLoader.h"
 #include "AliEMCALGeometry.h"
-#include "AliEMCALDigitizer.h"
+class AliEMCALDigitizer;
 #include "AliEMCALDigit.h"
-
-
+#include "AliEMCAL.h"
+  
 ClassImp(AliEMCALRawUtils)
-
+  
 // Signal shape parameters
-Int_t    AliEMCALRawUtils::fgOrder         = 2 ;      // Order of gamma function 
 Double_t AliEMCALRawUtils::fgTimeBinWidth  = 100E-9 ; // each sample is 100 ns
-Double_t AliEMCALRawUtils::fgTau         = 235E-9 ;   // 235 ns (from CERN testbeam; not very accurate)
 Double_t AliEMCALRawUtils::fgTimeTrigger = 1.5E-6 ;   // 15 time bins ~ 1.5 musec
 
 // some digitization constants
 Int_t    AliEMCALRawUtils::fgThreshold = 1;
 Int_t    AliEMCALRawUtils::fgDDLPerSuperModule = 2;  // 2 ddls per SuperModule
+Int_t    AliEMCALRawUtils::fgPedestalValue = 32;     // pedestal value for digits2raw
+Double_t AliEMCALRawUtils::fgFEENoise = 3.;          // 3 ADC channels of noise (sampled)
+
+AliEMCALRawUtils::AliEMCALRawUtils()
+  : fHighLowGainFactor(0.), fOrder(0), fTau(0.), fNoiseThreshold(0),
+    fNPedSamples(0), fGeom(0), fOption("")
+{
 
-AliEMCALRawUtils::AliEMCALRawUtils(): fHighLowGainFactor(0.) {
+  //These are default parameters.  
+  //Can be re-set from without with setter functions
   fHighLowGainFactor = 16. ;          // adjusted for a low gain range of 82 GeV (10 bits) 
+  fOrder = 2;                         // order of gamma fn
+  fTau = 2.35;                        // in units of timebin, from CERN 2007 testbeam
+  fNoiseThreshold = 3;
+  fNPedSamples = 5;
+
+  //Get Mapping RCU files from the AliEMCALRecParam                                 
+  const TObjArray* maps = AliEMCALRecParam::GetMappings();
+  if(!maps) AliFatal("Cannot retrieve ALTRO mappings!!");
+
+  for(Int_t i = 0; i < 4; i++) {
+    fMapping[i] = (AliAltroMapping*)maps->At(i);
+  }
+
+  //To make sure we match with the geometry in a simulation file,
+  //let's try to get it first.  If not, take the default geometry
+  AliRunLoader *rl = AliRunLoader::Instance();
+  if(!rl) AliError("Cannot find RunLoader!");
+  if (rl->GetAliRun() && rl->GetAliRun()->GetDetector("EMCAL")) {
+    fGeom = dynamic_cast<AliEMCAL*>(rl->GetAliRun()->GetDetector("EMCAL"))->GetGeometry();
+  } else {
+    AliInfo(Form("Using default geometry in raw reco"));
+    fGeom =  AliEMCALGeometry::GetInstance(AliEMCALGeometry::GetDefaultGeometryName());
+  }
+
+  if(!fGeom) AliFatal(Form("Could not get geometry!"));
+
+}
+
+//____________________________________________________________________________
+AliEMCALRawUtils::AliEMCALRawUtils(AliEMCALGeometry *pGeometry)
+  : fHighLowGainFactor(0.), fOrder(0), fTau(0.), fNoiseThreshold(0),
+    fNPedSamples(0), fGeom(pGeometry), fOption("")
+{
+  //
+  // Initialize with the given geometry - constructor required by HLT
+  // HLT does not use/support AliRunLoader(s) instances
+  // This is a minimum intervention solution
+  // Comment by MPloskon@lbl.gov
+  //
+
+  //These are default parameters. 
+  //Can be re-set from without with setter functions 
+  fHighLowGainFactor = 16. ;          // adjusted for a low gain range of 82 GeV (10 bits)
+  fOrder = 2;                         // order of gamma fn
+  fTau = 2.35;                        // in units of timebin, from CERN 2007 testbeam
+  fNoiseThreshold = 3;
+  fNPedSamples = 5;
+
+  //Get Mapping RCU files from the AliEMCALRecParam
+  const TObjArray* maps = AliEMCALRecParam::GetMappings();
+  if(!maps) AliFatal("Cannot retrieve ALTRO mappings!!");
+
+  for(Int_t i = 0; i < 4; i++) {
+    fMapping[i] = (AliAltroMapping*)maps->At(i);
+  }
+
+  if(!fGeom) AliFatal(Form("Could not get geometry!"));
+
+}
+
+//____________________________________________________________________________
+AliEMCALRawUtils::AliEMCALRawUtils(const AliEMCALRawUtils& rawU)
+  : TObject(),
+    fHighLowGainFactor(rawU.fHighLowGainFactor), 
+    fOrder(rawU.fOrder),
+    fTau(rawU.fTau),
+    fNoiseThreshold(rawU.fNoiseThreshold),
+    fNPedSamples(rawU.fNPedSamples),
+    fGeom(rawU.fGeom), 
+    fOption(rawU.fOption)
+{
+  //copy ctor
+  fMapping[0] = rawU.fMapping[0];
+  fMapping[1] = rawU.fMapping[1];
+  fMapping[2] = rawU.fMapping[2];
+  fMapping[3] = rawU.fMapping[3];
+}
+
+//____________________________________________________________________________
+AliEMCALRawUtils& AliEMCALRawUtils::operator =(const AliEMCALRawUtils &rawU)
+{
+  //assignment operator
+
+  if(this != &rawU) {
+    fHighLowGainFactor = rawU.fHighLowGainFactor;
+    fOrder = rawU.fOrder;
+    fTau = rawU.fTau;
+    fNoiseThreshold = rawU.fNoiseThreshold;
+    fNPedSamples = rawU.fNPedSamples;
+    fGeom = rawU.fGeom;
+    fOption = rawU.fOption;
+    fMapping[0] = rawU.fMapping[0];
+    fMapping[1] = rawU.fMapping[1];
+    fMapping[2] = rawU.fMapping[2];
+    fMapping[3] = rawU.fMapping[3];
+  }
+
+  return *this;
+
 }
+
 //____________________________________________________________________________
 AliEMCALRawUtils::~AliEMCALRawUtils() {
+  //dtor
+
 }
+
 //____________________________________________________________________________
 void AliEMCALRawUtils::Digits2Raw()
 {
   // convert digits of the current event to raw data
   
-  AliRunLoader *rl = AliRunLoader::GetRunLoader();
+  AliRunLoader *rl = AliRunLoader::Instance();
   AliEMCALLoader *loader = dynamic_cast<AliEMCALLoader*>(rl->GetDetectorLoader("EMCAL"));
 
   // get the digits
@@ -86,14 +192,7 @@ void AliEMCALRawUtils::Digits2Raw()
     Warning("Digits2Raw", "no digits found !");
     return;
   }
-    
-  // get the geometry
-  AliEMCALGeometry* geom = AliEMCALGeometry::GetInstance();
-  if (!geom) {
-    AliError(Form("No geometry found !"));
-    return;
-  }
-  
+
   static const Int_t nDDL = 12*2; // 12 SM hardcoded for now. Buffers allocated dynamically, when needed, so just need an upper limit here
   AliAltroBuffer* buffers[nDDL];
   for (Int_t i=0; i < nDDL; i++)
@@ -102,15 +201,6 @@ void AliEMCALRawUtils::Digits2Raw()
   Int_t adcValuesLow[fgkTimeBins];
   Int_t adcValuesHigh[fgkTimeBins];
 
-  //Load Mapping RCU files once
-  TString path = gSystem->Getenv("ALICE_ROOT");
-  path += "/EMCAL/mapping/RCU";
-  TString path0 = path+"0.data";//This file will change in future
-  TString path1 = path+"1.data";//This file will change in future
-  AliAltroMapping * mapping[2] ; // For the moment only 2
-  mapping[0] = new AliCaloAltroMapping(path0.Data());
-  mapping[1] = new AliCaloAltroMapping(path1.Data());
-
   // loop over digits (assume ordered digits)
   for (Int_t iDigit = 0; iDigit < digits->GetEntries(); iDigit++) {
     AliEMCALDigit* digit = dynamic_cast<AliEMCALDigit *>(digits->At(iDigit)) ;
@@ -124,10 +214,10 @@ void AliEMCALRawUtils::Digits2Raw()
     Int_t iphi = 0;
     Int_t ieta = 0;
     Int_t nModule = 0;
-    geom->GetCellIndex(digit->GetId(), nSM, nModule, nIphi, nIeta);
-    geom->GetCellPhiEtaIndexInSModule(nSM, nModule, nIphi, nIeta,iphi, ieta) ;
+    fGeom->GetCellIndex(digit->GetId(), nSM, nModule, nIphi, nIeta);
+    fGeom->GetCellPhiEtaIndexInSModule(nSM, nModule, nIphi, nIeta,iphi, ieta) ;
     
-    //Check which is the RCU of the cell.
+    //Check which is the RCU, 0 or 1, of the cell.
     Int_t iRCU = -111;
     //RCU0
     if (0<=iphi&&iphi<8) iRCU=0; // first cable row
@@ -137,6 +227,11 @@ void AliEMCALRawUtils::Digits2Raw()
     else if(8<=iphi&&iphi<16 && 24<=ieta&&ieta<48) iRCU=1; // second half; 
     //second cable row
     else if(16<=iphi&&iphi<24) iRCU=1; // third cable row
+
+    if (nSM%2==1) iRCU = 1 - iRCU; // swap for odd=C side, to allow us to cable both sides the same
+
+    if (iRCU<0) 
+      Fatal("Digits2Raw()","Non-existent RCU number: %d", iRCU);
     
     //Which DDL?
     Int_t iDDL = fgDDLPerSuperModule* nSM + iRCU;
@@ -146,7 +241,14 @@ void AliEMCALRawUtils::Digits2Raw()
     if (buffers[iDDL] == 0) {      
       // open new file and write dummy header
       TString fileName = AliDAQ::DdlFileName("EMCAL",iDDL);
-      buffers[iDDL] = new AliAltroBuffer(fileName.Data(),mapping[iRCU]);
+      //Select mapping file RCU0A, RCU0C, RCU1A, RCU1C
+      Int_t iRCUside=iRCU+(nSM%2)*2;
+      //iRCU=0 and even (0) SM -> RCU0A.data   0
+      //iRCU=1 and even (0) SM -> RCU1A.data   1
+      //iRCU=0 and odd  (1) SM -> RCU0C.data   2
+      //iRCU=1 and odd  (1) SM -> RCU1C.data   3
+      //cout<<" nSM "<<nSM<<"; iRCU "<<iRCU<<"; iRCUside "<<iRCUside<<endl;
+      buffers[iDDL] = new AliAltroBuffer(fileName.Data(),fMapping[iRCUside]);
       buffers[iDDL]->WriteDataHeader(kTRUE, kFALSE);  //Dummy;
     }
     
@@ -175,20 +277,14 @@ void AliEMCALRawUtils::Digits2Raw()
       delete buffers[i];
     }
   }
-  mapping[0]->Delete();
-  mapping[1]->Delete();
+
   loader->UnloadDigits();
 }
 
 //____________________________________________________________________________
 void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
 {
-  // convert raw data of the current event to digits
-  AliEMCALGeometry * geom = AliEMCALGeometry::GetInstance();
-  if (!geom) {
-    AliError(Form("No geometry found !"));
-    return;
-  }
+  // convert raw data of the current event to digits                                                                                     
 
   digitsArr->Clear(); 
 
@@ -201,73 +297,93 @@ void AliEMCALRawUtils::Raw2Digits(AliRawReader* reader,TClonesArray *digitsArr)
     return;
   }
 
-  // Use AliAltroRawStream to read the ALTRO format.  No need to
-  // reinvent the wheel :-) 
-  AliCaloRawStream in(reader,"EMCAL");
+  AliCaloRawStreamV3 in(reader,"EMCAL",fMapping);
   // Select EMCAL DDL's;
   reader->Select("EMCAL");
-  in.SetOldRCUFormat(kTRUE); // Needed for testbeam data
-  
-  cout << "Stream set up" << endl;
 
-  // reading is from previously existing AliEMCALGetter.cxx
-  // ReadRaw method
-  TF1 * signalF = new TF1("signal", RawResponseFunction, 0, GetRawFormatTimeMax(), 4);
+  //Updated fitting routine from 2007 beam test takes into account
+  //possibility of two peaks in data and selects first one for fitting
+  //Also sets some of the starting parameters based on the shape of the
+  //given raw signal being fit
+
+  TF1 * signalF = new TF1("signal", RawResponseFunction, 0, GetRawFormatTimeBins(), 5);
+  signalF->SetParameters(10.,0.,fTau,fOrder,5.); //set all defaults once, just to be safe
+  signalF->SetParNames("amp","t0","tau","N","ped");
+  signalF->SetParameter(2,fTau); // tau in units of time bin
+  signalF->SetParLimits(2,2,-1);
+  signalF->SetParameter(3,fOrder); // order
+  signalF->SetParLimits(3,2,-1);
   
   Int_t id =  -1;
   Float_t time = 0. ; 
   Float_t amp = 0. ; 
+  Int_t i = 0;
+  Int_t startBin = 0;
 
-  TGraph * gSig = new TGraph(GetRawFormatTimeBins()) ; 
+  //Graph to hold data we will fit (should be converted to an array
+  //later to speed up processing
+  TGraph * gSig = new TGraph(GetRawFormatTimeBins()); 
 
-  Int_t readOk = 1;
   Int_t lowGain = 0;
-
-  while (readOk && in.GetModule() < 0) 
-    readOk = in.Next();  // Go to first digit
-
-  while (readOk) { 
-    id =  geom->GetAbsCellIdFromCellIndexes(in.GetModule(), in.GetRow(), in.GetColumn()) ;
-    lowGain = in.IsLowGain();
-    Int_t maxTime = in.GetTime();  // timebins come in reverse order
-    if (maxTime < 0 || maxTime >= GetRawFormatTimeBins()) {
-      AliWarning(Form("Invalid time bin %d",maxTime));
-      maxTime = GetRawFormatTimeBins();
-    }
-    gSig->Set(maxTime+1);
-    // There is some kind of zero-suppression in the raw data, 
-    // so set up the TGraph in advance
-    for (Int_t i=0; i < maxTime; i++) {
-      gSig->SetPoint(i, i * GetRawFormatTimeBinWidth(), 0);
-    }
-
-    Int_t iTime = 0;
-    do {
-      if (in.GetTime() >= gSig->GetN()) {
-         AliWarning("Too many time bins");
-         gSig->Set(in.GetTime());
+  Int_t caloFlag = 0; // low, high gain, or TRU, or LED ref.
+
+  // start loop over input stream 
+  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++) {
+       gSig->SetPoint(i, i , 0);
       }
-      gSig->SetPoint(in.GetTime(), 
-                  in.GetTime() * GetRawFormatTimeBinWidth(), 
-                  in.GetSignal()) ;
-      if (in.GetTime() > maxTime)
-        maxTime = in.GetTime();
-      iTime++;
-    } while ((readOk = in.Next()) && !in.IsNewHWAddress());
-    signalF->SetRange(0,(Float_t)maxTime*GetRawFormatTimeBinWidth());
-
-    FitRaw(gSig, signalF, amp, time) ; 
+               
+      Int_t maxTime = 0;
+
+      while (in.NextBunch()) {
+       const UShort_t *sig = in.GetSignals();
+       startBin = in.GetStartTimeBin();
+
+       if (((UInt_t) maxTime) < in.GetStartTimeBin()) {
+         maxTime = in.GetStartTimeBin(); // timebins come in reverse order
+       }
+
+       if (maxTime < 0 || maxTime >= GetRawFormatTimeBins()) {
+         AliWarning(Form("Invalid time bin %d",maxTime));
+         maxTime = GetRawFormatTimeBins();
+       }
+
+       for (i = 0; i < in.GetBunchLength(); i++) {
+         time = startBin--;
+         gSig->SetPoint(time, time, sig[i]) ;
+       }
+      } // loop over bunches
     
-    if (amp > 0) {
-      AliDebug(2,Form("id %d lowGain %d amp %g", id, lowGain, amp));
-      AddDigit(digitsArr, id, lowGain, (Int_t)amp, time);
-    }
+      gSig->Set(maxTime+1);
+      FitRaw(gSig, signalF, amp, time) ; 
+    
+      if (caloFlag == 0 || caloFlag == 1) { // low gain or high gain 
+       if (amp > 0 && amp < 2000) {  //check both high and low end of
+       //result, 2000 is somewhat arbitrary - not nice with magic numbers in the code..
+         AliDebug(2,Form("id %d lowGain %d amp %g", id, lowGain, amp));
        
-    // Reset graph
-    for (Int_t index = 0; index < gSig->GetN(); index++) {
-      gSig->SetPoint(index, index * GetRawFormatTimeBinWidth(), 0) ;  
-    } 
-  }; // EMCAL entries loop
+         AddDigit(digitsArr, id, lowGain, (Int_t)amp, time);
+       }
+       
+      }
+
+      // Reset graph
+      for (Int_t index = 0; index < gSig->GetN(); index++) {
+       gSig->SetPoint(index, index, 0) ;  
+      } 
+      // Reset starting parameters for fit function
+      signalF->SetParameters(10.,0.,fTau,fOrder,5.); //reset all defaults just to be safe
+
+   } // end while over channel   
+  } //end while over DDL's, of input stream 
   
   delete signalF ; 
   delete gSig;
@@ -299,14 +415,14 @@ void AliEMCALRawUtils::AddDigit(TClonesArray *digitsArr, Int_t id, Int_t lowGain
     new((*digitsArr)[idigit]) AliEMCALDigit( -1, -1, id, amp, time, idigit) ;  
   }
   else { // a digit already exists, check range 
-         // (use high gain if signal < 800, otherwise low gain)
+         // (use high gain if signal < cut value, otherwise low gain)
     if (lowGain) { // new digit is low gain
-      if (digit->GetAmp() > 800) {  // use if stored digit is out of range
+      if (digit->GetAmp() > fgkOverflowCut) {  // use if stored digit is out of range
        digit->SetAmp(Int_t(fHighLowGainFactor * amp));
        digit->SetTime(time);
       }
     }
-    else if (amp < 800) { // new digit is high gain; use if not out of range
+    else if (amp < fgkOverflowCut) { // new digit is high gain; use if not out of range
       digit->SetAmp(amp);
       digit->SetTime(time);
     }
@@ -314,17 +430,15 @@ void AliEMCALRawUtils::AddDigit(TClonesArray *digitsArr, Int_t id, Int_t lowGain
 }
 
 //____________________________________________________________________________ 
-void AliEMCALRawUtils::FitRaw(TGraph * gSig, TF1* signalF, Float_t & amp, Float_t & time)
+void AliEMCALRawUtils::FitRaw(TGraph * gSig, TF1* signalF, Float_t & amp, Float_t & time) const 
 {
   // Fits the raw signal time distribution; from AliEMCALGetter 
 
-  const Int_t kNoiseThreshold = 5;
-  const Int_t kNPedSamples = 10;
   amp = time = 0. ; 
   Double_t ped = 0;
   Int_t nPed = 0;
 
-  for (Int_t index = 0; index < kNPedSamples; index++) {
+  for (Int_t index = 0; index < fNPedSamples; index++) {
     Double_t ttime, signal;
     gSig->GetPoint(index, ttime, signal) ; 
     if (signal > 0) {
@@ -336,68 +450,90 @@ void AliEMCALRawUtils::FitRaw(TGraph * gSig, TF1* signalF, Float_t & amp, Float_
   if (nPed > 0)
     ped /= nPed;
   else {
-    AliWarning("Could determine pedestal");      
+    AliWarning("Could not determine pedestal");          
     ped = 10; // put some small value as first guess
   }
 
-  Int_t max_found = 0;
-  Int_t i_max = 0;
+  Int_t maxFound = 0;
+  Int_t iMax = 0;
   Float_t max = -1;
-  Float_t tmax = 0;
-  Float_t max_fit = gSig->GetN()*GetRawFormatTimeBinWidth();
-  Float_t min_after_sig = 9999;
-  Int_t imin_after_sig = gSig->GetN();
-  Float_t tmin_after_sig = gSig->GetN()*GetRawFormatTimeBinWidth();
-  Int_t n_ped_after_sig = 0;
-
-  for (Int_t i=kNPedSamples; i < gSig->GetN(); i++) {
+  Float_t maxFit = gSig->GetN();
+  Float_t minAfterSig = 9999;
+  Int_t tminAfterSig = gSig->GetN();
+  Int_t nPedAfterSig = 0;
+  Int_t plateauWidth = 0;
+  Int_t plateauStart = 9999;
+  Float_t cut = 0.3;
+
+  for (Int_t i=fNPedSamples; i < gSig->GetN(); i++) {
     Double_t ttime, signal;
     gSig->GetPoint(i, ttime, signal) ; 
-    if (!max_found && signal > max) {
-      i_max = i;
-      tmax = ttime;
+    if (!maxFound && signal > max) {
+      iMax = i;
       max = signal;
     }
-    else if ( max > ped + kNoiseThreshold ) {
-      max_found = 1;
-      min_after_sig = signal;
-      imin_after_sig = i;
-      tmin_after_sig = ttime;
+    else if ( max > ped + fNoiseThreshold ) {
+      maxFound = 1;
+      minAfterSig = signal;
+      tminAfterSig = i;
     }
-    if (max_found) {
-      if ( signal < min_after_sig) {
-        min_after_sig = signal;
-       imin_after_sig = i;
-        tmin_after_sig = ttime;
+    if (maxFound) {
+      if ( signal < minAfterSig) {
+        minAfterSig = signal;
+       tminAfterSig = i;
+      }
+      if (i > tminAfterSig + 5) {  // Two close peaks; end fit at minimum
+        maxFit = tminAfterSig;
+        break;
       }
-      if (i > tmin_after_sig + 5) {  // Two close peaks; end fit at minimum
-        max_fit = tmin_after_sig;
+      if ( signal < cut*max){   //stop fit at 30% amplitude(avoid the pulse shape falling edge)
+        maxFit = i;
         break;
       }
-      if ( signal < ped + kNoiseThreshold)
-        n_ped_after_sig++;
-      if (n_ped_after_sig >= 5) {  // include 5 pedestal bins after peak
-        max_fit = ttime;
+      if ( signal < ped + fNoiseThreshold)
+        nPedAfterSig++;
+      if (nPedAfterSig >= 5) {  // include 5 pedestal bins after peak
+        maxFit = i;
         break;
       }
     }
+    //Add check on plateau
+    if (signal >= fgkRawSignalOverflow - fNoiseThreshold) {
+      if(plateauWidth == 0) plateauStart = i;
+      plateauWidth++;
+    }
   }
 
-  if ( max - ped > kNoiseThreshold ) { // else its noise 
+  if(plateauWidth > 0) {
+    for(int j = 0; j < plateauWidth; j++) {
+      //Note, have to remove the same point N times because after each
+      //remove, the positions of all subsequent points have shifted down
+      gSig->RemovePoint(plateauStart);
+    }
+  }
+
+  if ( max - ped > fNoiseThreshold ) { // else its noise 
     AliDebug(2,Form("Fitting max %d ped %d", max, ped));
-    signalF->SetParameter(0, ped) ; 
-    signalF->SetParameter(1, max - ped) ; 
-    signalF->SetParameter(2, tmax) ; 
-    signalF->SetParLimits(2, 0, max_fit) ; 
-    gSig->Fit(signalF, "QRON", "", 0., max_fit); //, "QRON") ; 
-    amp = signalF->GetParameter(1); 
-    time = signalF->GetParameter(2) - fgTimeTrigger;
+    signalF->SetRange(0,maxFit);
+
+    if(max-ped > 50) 
+      signalF->SetParLimits(2,1,3);
+
+    signalF->SetParameter(4, ped) ; 
+    signalF->SetParameter(1, iMax);
+    signalF->SetParameter(0, max);
+    
+    gSig->Fit(signalF, "QROW"); // Note option 'W': equal errors on all points
+    amp = signalF->GetParameter(0); 
+    time = signalF->GetParameter(1)*GetRawFormatTimeBinWidth() - fgTimeTrigger;
   }
   return;
 }
 //__________________________________________________________________
 Double_t AliEMCALRawUtils::RawResponseFunction(Double_t *x, Double_t *par)
 {
+  // Matches version used in 2007 beam test
+  //
   // Shape of the electronics raw reponse:
   // It is a semi-gaussian, 2nd order Gamma function of the general form
   //
@@ -406,20 +542,22 @@ Double_t AliEMCALRawUtils::RawResponseFunction(Double_t *x, Double_t *par)
   // F = 0                                for t < 0 
   //
   // parameters:
-  // ped: par[0]
-  // A:   par[1]   // Amplitude = peak value
-  // t0:  par[2]
-  // tau: fgTau
-  // N:   fgOrder
+  // A:   par[0]   // Amplitude = peak value
+  // t0:  par[1]
+  // tau: par[2]
+  // N:   par[3]
+  // ped: par[4]
   //
   Double_t signal ;
-  Double_t xx = ( x[0] - par[2] + fgTau ) / fgTau ; 
+  Double_t tau =par[2];
+  Double_t n =par[3];
+  Double_t ped = par[4];
+  Double_t xx = ( x[0] - par[1] + tau ) / tau ;
 
   if (xx <= 0) 
-    signal = par[0] ;  
+    signal = ped ;  
   else {  
-    signal = par[0] + par[1] * TMath::Power(xx , fgOrder) * TMath::Exp(fgOrder * (1 - xx )) ; 
-
+    signal = ped + par[0] * TMath::Power(xx , n) * TMath::Exp(n * (1 - xx )) ; 
   }
   return signal ;  
 }
@@ -431,29 +569,40 @@ const Double_t dtime, const Double_t damp, Int_t * adcH, Int_t * adcL) const
   // for a start time dtime and an amplitude damp given by digit, 
   // calculates the raw sampled response AliEMCAL::RawResponseFunction
 
-  const Int_t kRawSignalOverflow = 0x3FF ; 
-  const Int_t pedVal = 32;
   Bool_t lowGain = kFALSE ; 
 
-  TF1 signalF("signal", RawResponseFunction, 0, GetRawFormatTimeMax(), 4);
-  signalF.SetParameter(0, pedVal) ; 
-  signalF.SetParameter(1, damp) ; 
-  signalF.SetParameter(2, dtime + fgTimeTrigger) ; 
+  // A:   par[0]   // Amplitude = peak value
+  // t0:  par[1]                            
+  // tau: par[2]                            
+  // N:   par[3]                            
+  // ped: par[4]
+
+  TF1 signalF("signal", RawResponseFunction, 0, GetRawFormatTimeBins(), 5);
+  signalF.SetParameter(0, damp) ; 
+  signalF.SetParameter(1, (dtime + fgTimeTrigger)/fgTimeBinWidth) ; 
+  signalF.SetParameter(2, fTau) ; 
+  signalF.SetParameter(3, fOrder);
+  signalF.SetParameter(4, fgPedestalValue);
 
   for (Int_t iTime = 0; iTime < GetRawFormatTimeBins(); iTime++) {
-    Double_t time = iTime * GetRawFormatTimeBinWidth() ;
-    Double_t signal = signalF.Eval(time) ;     
+    Double_t signal = signalF.Eval(iTime) ;     
+
+    //According to Terry Awes, 13-Apr-2008
+    //add gaussian noise in quadrature to each sample
+    //Double_t noise = gRandom->Gaus(0.,fgFEENoise);
+    //signal = sqrt(signal*signal + noise*noise);
+
     adcH[iTime] =  static_cast<Int_t>(signal + 0.5) ;
-    if ( adcH[iTime] > kRawSignalOverflow ){  // larger than 10 bits 
-      adcH[iTime] = kRawSignalOverflow ;
+    if ( adcH[iTime] > fgkRawSignalOverflow ){  // larger than 10 bits 
+      adcH[iTime] = fgkRawSignalOverflow ;
       lowGain = kTRUE ; 
     }
 
     signal /= fHighLowGainFactor;
 
     adcL[iTime] =  static_cast<Int_t>(signal + 0.5) ;
-    if ( adcL[iTime] > kRawSignalOverflow)  // larger than 10 bits 
-      adcL[iTime] = kRawSignalOverflow ;
+    if ( adcL[iTime] > fgkRawSignalOverflow)  // larger than 10 bits 
+      adcL[iTime] = fgkRawSignalOverflow ;
   }
   return lowGain ; 
 }