]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - MUON/AliMUONDigitizerV3.cxx
- Disentangle masks effect from trigger chamber efficiency estimation.
[u/mrichter/AliRoot.git] / MUON / AliMUONDigitizerV3.cxx
index be55b17fc2d06f1a9c6bfc448ae0bee6009edc3b..172bdc9bb3c56b7ecd3a5d99dae8cf37113b0e9a 100644 (file)
 
 // $Id$
 
+
 #include "AliMUONDigitizerV3.h"
 
-#include "AliLog.h"
+#include "AliMUON.h"
 #include "AliMUONCalibrationData.h"
-#include "AliMUONChamber.h"
 #include "AliMUONConstants.h"
-#include "AliMUONData.h"
 #include "AliMUONDigit.h"
-#include "AliMUONTriggerDecisionV1.h"
+#include "AliMUONLogger.h"
 #include "AliMUONTriggerElectronics.h"
+#include "AliMUONTriggerStoreV1.h"
 #include "AliMUONVCalibParam.h"
+#include "AliMUONVDigitStore.h"
+#include "AliMUONGeometryTransformer.h" //ADDED for trigger noise
+#include "AliMUONTriggerChamberEfficiency.h"
+#include "AliMUONTriggerUtilities.h"
+
+#include "AliMpCDB.h"
+#include "AliMpSegmentation.h"
+#include "AliMpCathodType.h"
+#include "AliMpConstants.h"
+#include "AliMpDEIterator.h"
 #include "AliMpDEManager.h"
+#include "AliMpPad.h"
 #include "AliMpStationType.h"
+#include "AliMpVSegmentation.h"
+#include "AliMpDDLStore.h"
+
+#include "AliCDBManager.h"
+#include "AliCodeTimer.h"
+#include "AliLog.h"
 #include "AliRun.h"
 #include "AliRunDigitizer.h"
+#include "AliLoader.h"
 #include "AliRunLoader.h"
-#include "Riostream.h"
-#include "TRandom.h"
-#include "TString.h"
 
+#include <Riostream.h>
+#include <TF1.h>
+#include <TFile.h>
+#include <TMath.h>
+#include <TRandom.h>
+#include <TString.h>
+#include <TSystem.h>
+#include <TTree.h>
+
+//-----------------------------------------------------------------------------
+/// \class AliMUONDigitizerV3
+/// The digitizer is performing the transformation to go from SDigits (digits
+/// w/o any electronic noise) to Digits (w/ electronic noise, and decalibration)
+/// 
+/// The decalibration is performed by doing the reverse operation of the
+/// calibration, that is we do (Signal+pedestal)/gain -> ADC
+///
+/// Note also that the digitizer takes care of merging sdigits that belongs
+/// to the same pad, either because we're merging several input sdigit files
+/// or with a single file because the sdigitizer does not merge sdigits itself
+/// (for performance reason mainly, and because anyway we know we have to do it
+/// here, at the digitization level).
+///
+/// \author Laurent Aphecetche
+//-----------------------------------------------------------------------------
+
+namespace
+{
+  AliMUON* muon()
+  {
+    return static_cast<AliMUON*>(gAlice->GetModule("MUON"));
+  }
+
+  //ADDED for trigger noise
+  const AliMUONGeometryTransformer* GetTransformer()
+  {
+      return muon()->GetGeometryTransformer();
+  }
+}
+
+Double_t AliMUONDigitizerV3::fgNSigmas = 4.0;
+
+/// \cond CLASSIMP
 ClassImp(AliMUONDigitizerV3)
+/// \endcond
 
 //_____________________________________________________________________________
 AliMUONDigitizerV3::AliMUONDigitizerV3(AliRunDigitizer* manager, 
-                                       ETriggerCodeVersion triggerCodeVersion)
+                                       Int_t generateNoisyDigits)
 : AliDigitizer(manager),
 fIsInitialized(kFALSE),
-fOutputData(0x0),
 fCalibrationData(0x0),
 fTriggerProcessor(0x0),
-fTriggerCodeVersion(triggerCodeVersion)
+fNoiseFunctionTrig(0x0),
+fGenerateNoisyDigits(generateNoisyDigits),
+fLogger(new AliMUONLogger(1000)),
+fTriggerStore(new AliMUONTriggerStoreV1),
+fDigitStore(0x0),
+fOutputDigitStore(0x0),
+fInputDigitStores(0x0),
+fTriggerEfficiency(0x0),
+fTriggerUtilities(0x0),
+fEfficiencyResponse(2*AliMUONConstants::NTriggerCh()*AliMUONConstants::NTriggerCircuit())
 {
-  //
-  // Ctor.
-  //
-  AliDebug(1,Form("AliRunDigitizer=%p",fManager));
-}
+  /// Ctor.
 
-//______________________________________________________________________________
-AliMUONDigitizerV3::AliMUONDigitizerV3(const AliMUONDigitizerV3& right) 
-  : AliDigitizer(right) 
-{  
-/// Protected copy constructor (not implemented)
+  AliDebug(1,Form("AliRunDigitizer=%p",fManager));
 
-  AliFatal("Copy constructor not provided.");
 }
 
 //_____________________________________________________________________________
 AliMUONDigitizerV3::~AliMUONDigitizerV3()
 {
-  //
-  // Dtor. Note we're the owner of some pointers.
-  // 
+  /// Dtor. Note we're the owner of some pointers.
+
   AliDebug(1,"dtor");
-  delete fOutputData;
-  delete fCalibrationData;
+
// delete fCalibrationData;
   delete fTriggerProcessor;
+  delete fNoiseFunctionTrig;
+  delete fTriggerStore;
+  delete fDigitStore;
+  delete fOutputDigitStore;
+  delete fInputDigitStores;
+  delete fTriggerUtilities;
+  
+  AliInfo("Summary of messages");
+  fLogger->Print();
+  
+  delete fLogger;
 }
 
-//______________________________________________________________________________
-AliMUONDigitizerV3& 
-AliMUONDigitizerV3::operator=(const AliMUONDigitizerV3& right)
-{
-/// Protected assignement operator (not implemented)
-
-  // check assignement to self
-  if (this == &right) return *this;
-
-  AliFatal("Assignement operator not provided.");
-    
-  return *this;  
-}    
-
 //_____________________________________________________________________________
 void 
-AliMUONDigitizerV3::ApplyResponseToDigit(AliMUONDigit& digit)
+AliMUONDigitizerV3::ApplyResponseToTrackerDigit(AliMUONVDigit& digit, Bool_t addNoise)
 {
-  // For trigger digits, simply does nothing.
-  //
-  // For tracking digits, starting from an ideal digit's charge, we :
-  //
-  // - add some noise (thus leading to a realistic charge)
-  // - divide by a gain (thus decalibrating the digit)
-  // - add a pedestal (thus decalibrating the digit)
-  // - sets the signal to zero if below 3*sigma of the noise
-  //
-  
-  static const Int_t MAXADC = (1<<12)-1; // We code the charge on a 12 bits ADC.
+  /// For tracking digits, starting from an ideal digit's charge, we :
+  ///
+  /// - "divide" by a gain (thus decalibrating the digit)
+  /// - add a pedestal (thus decalibrating the digit)
+  /// - add some electronics noise (thus leading to a realistic adc), if requested to do so
+  /// - sets the signal to zero if below 3*sigma of the noise
+
+  Float_t charge = digit.IsChargeInFC() ? digit.Charge()*AliMUONConstants::FC2ADC() : digit.Charge();
   
+  // We set the charge to 0, as the only relevant piece of information
+  // after Digitization is the ADC value.  
+  digit.SetCharge(0);
+    
   Int_t detElemId = digit.DetElemId();
-  AliMpStationType stationType = AliMpDEManager::GetStationType(detElemId);
-  if ( stationType == kStationTrigger )
+  Int_t manuId = digit.ManuId();
+  
+  AliMUONVCalibParam* pedestal = fCalibrationData->Pedestals(detElemId,manuId);
+  if (!pedestal)
   {
+    fLogger->Log(Form("%s:%d:Could not get pedestal for DE=%4d manuId=%4d. Disabling.",
+                      __FILE__,__LINE__,
+                      detElemId,manuId));
+    digit.SetADC(0);
     return;    
   }
   
-  // The following is for tracking digits only.
-  
-  Int_t manuId = digit.ManuId();
   Int_t manuChannel = digit.ManuChannel();
   
-  AliMUONVCalibParam* pedestal = fCalibrationData->Pedestal(detElemId,manuId);
-  if (!pedestal)
+  if ( pedestal->ValueAsFloat(manuChannel,0) == AliMUONVCalibParam::InvalidFloatValue() ||
+      pedestal->ValueAsFloat(manuChannel,1) == AliMUONVCalibParam::InvalidFloatValue() )
   {
-    AliFatal(Form("Could not get pedestal for DE=%d manuId=%d",
-                  detElemId,manuId));    
+    // protection against invalid pedestal value
+    digit.SetADC(0);
+    return;
   }
-  Float_t pedestalMean = pedestal->ValueAsFloat(manuChannel,0);
-  Float_t pedestalSigma = pedestal->ValueAsFloat(manuChannel,1);
-  
-  Float_t adc_noise = gRandom->Gaus(0.0,pedestalSigma);
-  
-  AliMUONVCalibParam* gain = fCalibrationData->Gain(detElemId,manuId);
+      
+  AliMUONVCalibParam* gain = fCalibrationData->Gains(detElemId,manuId);
   if (!gain)
   {
-    AliFatal(Form("Could not get gain for DE=%d manuId=%d",
-                  detElemId,manuId));    
-  }
+    fLogger->Log(Form("%s:%d:Could not get gain for DE=%4d manuId=%4d. Disabling.",
+                      __FILE__,__LINE__,
+                      detElemId,manuId));
+    digit.SetADC(0);
+    return;        
+  }    
+
+  Int_t adc = DecalibrateTrackerDigit(*pedestal,*gain,manuChannel,charge,addNoise,
+                                      digit.IsNoiseOnly());
   
-  Float_t gainMean = gain->ValueAsFloat(manuChannel,0);
-  Float_t signal_noise = adc_noise*gainMean;
+  digit.SetADC(adc);
+}
+
+
+//_____________________________________________________________________________
+void 
+AliMUONDigitizerV3::ApplyResponseToTriggerDigit(AliMUONVDigit& digit)
+{
+  /// For trigger digits, starting from an ideal digit, we :
+  ///
+  /// - apply efficiency (on demand)
+  /// - apply trigger masks
+    
+  Int_t detElemId = digit.DetElemId();
+  Int_t localCircuit = digit.ManuId();
+  Int_t strip = digit.ManuChannel();
+  Int_t cathode = digit.Cathode();
+  Int_t trigCh = detElemId/100 - 11;
   
-  Float_t signal = digit.Signal() + signal_noise;
-  Int_t adc;
+  Int_t arrayIndex = GetArrayIndex(cathode, trigCh, localCircuit);
   
-  if ( gainMean < 1E-6 )
-  {
-    AliError(Form("Got a too small gain %e for DE=%d manuId=%d manuChannel=%d. "
-                  "Setting signal to 0.",
-                  gainMean,detElemId,manuId,manuChannel));
-    adc = 0;
+  // Trigger chamber efficiency
+  if ( fTriggerEfficiency ) {
+    if ( fEfficiencyResponse[arrayIndex] < 0 ) {
+      Bool_t isTrig[2] = {kTRUE, kTRUE};
+      fTriggerEfficiency->IsTriggered(detElemId, localCircuit, isTrig[0], isTrig[1]);
+      Int_t arrayIndexBend = GetArrayIndex(0, trigCh, localCircuit);
+      Int_t arrayIndexNonBend = GetArrayIndex(1, trigCh, localCircuit);
+      fEfficiencyResponse[arrayIndexBend] = isTrig[0];
+      fEfficiencyResponse[arrayIndexNonBend] = isTrig[1];
+    }
+    AliDebug(1,Form("ch %i  cath %i  board %i  strip %i  efficiency %i\n", trigCh, cathode, localCircuit, strip, fEfficiencyResponse[arrayIndex]));
+    if ( fEfficiencyResponse[arrayIndex] == 0 ) {
+      digit.SetCharge(0);
+      digit.SetADC(0);
+      //AliDebug(1,Form("ch %i  cath %i  board %i  strip %i  NOT efficient\n", trigCh, cathode, localCircuit, strip));
+      return;
+    }
+  }
+  
+  // Masked channels
+  Bool_t isMasked = fTriggerUtilities->IsMasked(digit);
+  AliDebug(1,Form("ch %i  cath %i  board %i  strip %i  mask %i\n", trigCh, cathode, localCircuit, strip, !isMasked));
+  if ( isMasked ) {
+    digit.SetCharge(0);
+    digit.SetADC(0);
+    //AliDebug(1,Form("ch %i  cath %i  board %i  strip %i  masked\n", trigCh, cathode, localCircuit, strip));
+    return;
   }
-  else
+}
+
+
+
+//_____________________________________________________________________________
+void
+AliMUONDigitizerV3::ApplyResponse(const AliMUONVDigitStore& store,
+                                  AliMUONVDigitStore& filteredStore)
+{
+  /// Loop over all chamber digits, and apply the response to them
+  /// Note that this method may remove digits.
+
+  filteredStore.Clear();
+  
+  const Bool_t kAddNoise = kTRUE;
+  
+  TIter next(store.CreateIterator());
+  AliMUONVDigit* digit;
+  
+  if ( fTriggerEfficiency ) fEfficiencyResponse.Reset(-1);
+  
+  while ( ( digit = static_cast<AliMUONVDigit*>(next()) ) )
   {
-    adc = TMath::Nint( signal / gainMean + pedestalMean );
+    AliMp::StationType stationType = AliMpDEManager::GetStationType(digit->DetElemId());
     
-    if ( adc <= pedestalMean + 3.0*pedestalSigma ) 
+    if ( stationType != AliMp::kStationTrigger )
     {
-      adc = 0;
+      Bool_t addNoise = kAddNoise;
+      if (digit->IsConverted()) addNoise = kFALSE; // No need to add extra noise to a converted real digit
+      ApplyResponseToTrackerDigit(*digit,addNoise);
+    }
+    else {
+      ApplyResponseToTriggerDigit(*digit);
+    }
+
+    if ( digit->ADC() > 0  || digit->Charge() > 0 )
+    {
+      filteredStore.Add(*digit,AliMUONVDigitStore::kIgnore);
     }
   }
-  // be sure we stick to 12 bits.
-  if ( adc > MAXADC )
-  {
-    adc = MAXADC;
-  }
-  digit.SetPhysicsSignal(TMath::Nint(signal));
-  digit.SetSignal(adc);
-  digit.SetADC(adc);
-}
+}    
 
 //_____________________________________________________________________________
-void
-AliMUONDigitizerV3::ApplyResponse()
+Int_t 
+AliMUONDigitizerV3::DecalibrateTrackerDigit(const AliMUONVCalibParam& pedestals,
+                                            const AliMUONVCalibParam& gains,
+                                            Int_t channel,
+                                            Float_t charge,
+                                            Bool_t addNoise,
+                                            Bool_t noiseOnly)
 {
-  //
-  // Loop over all chamber digits, and apply the response to them
-  // Note that this method may remove digits.
-  //
-  for ( Int_t ich = 0; ich < AliMUONConstants::NCh(); ++ich )
-       {
-    TClonesArray* digits = fOutputData->Digits(ich);
-    Int_t n = digits->GetEntriesFast();
-    for ( Int_t i = 0; i < n; ++i )
+  /// Decalibrate (i.e. go from charge to adc) a tracker digit, given its
+  /// pedestal and gain parameters.
+  /// Must insure before calling that channel is valid (i.e. between 0 and
+  /// pedestals or gains->GetSize()-1, but also corresponding to a valid channel
+  /// otherwise results are not predictible...)
+
+  static const Int_t kMaxADC = (1<<12)-1; // We code the charge on a 12 bits ADC.
+  
+  Float_t pedestalMean = pedestals.ValueAsFloat(channel,0);
+  Float_t pedestalSigma = pedestals.ValueAsFloat(channel,1);
+  
+  AliDebugClass(1,Form("DE %04d MANU %04d CH %02d PEDMEAN %7.2f PEDSIGMA %7.2f",
+                      pedestals.ID0(),pedestals.ID1(),channel,pedestalMean,pedestalSigma));
+  
+  Float_t a0 = gains.ValueAsFloat(channel,0);
+  Float_t a1 = gains.ValueAsFloat(channel,1);
+  Int_t thres = gains.ValueAsInt(channel,2);
+  Int_t qual = gains.ValueAsInt(channel,3);
+
+  if ( qual <= 0 ) return 0;
+  
+  Float_t chargeThres = a0*thres;
+  
+  Float_t padc(0); // (adc - ped) value
+  
+  if ( charge <= chargeThres || TMath::Abs(a1) < 1E-12 ) 
+  {
+    // linear part only
+    
+    if ( TMath::Abs(a0) > 1E-12 ) 
     {
-      AliMUONDigit* d = static_cast<AliMUONDigit*>(digits->UncheckedAt(i));
-      ApplyResponseToDigit(*d);
-      if ( d->Signal() <= 0 )
+      padc = charge/a0;    
+    }
+  }
+  else 
+  {
+    // linear + parabolic part
+    Double_t qt = chargeThres - charge;
+    Double_t delta = a0*a0-4*a1*qt;
+    if ( delta < 0 ) 
+    {
+      AliErrorClass(Form("delta=%e DE %d Manu %d Channel %d "
+                    " charge %e a0 %e a1 %e thres %d ped %e pedsig %e",
+                    delta,pedestals.ID0(),pedestals.ID1(),
+                    channel, charge, a0, a1, thres, pedestalMean, 
+                    pedestalSigma));      
+    }      
+    else
+    {
+      delta = TMath::Sqrt(delta);
+      
+      padc = ( ( -a0 + delta ) > 0 ? ( -a0 + delta ) : ( -a0 - delta ) );
+      
+      padc /= 2*a1;
+    
+      if ( padc < 0 )
       {
-        digits->RemoveAt(i);
+        if ( TMath::Abs(padc) > 1E-3) 
+        {
+          // this is more than a precision problem : let's signal it !
+          AliErrorClass(Form("padc=%e DE %d Manu %d Channel %d "
+                             " charge %e a0 %e a1 %e thres %d ped %e pedsig %e delta %e",
+                             padc,pedestals.ID0(),pedestals.ID1(),
+                             channel, charge, a0, a1, thres, pedestalMean, 
+                             pedestalSigma,delta));
+        }
+
+        // ok. consider we're just at thres, let it be zero.
+        padc = 0;
       }
+
+      padc += thres;
+
     }
-    digits->Compress();
-  }    
+  }
+  
+  Int_t adc(0);
+  
+  Float_t adcNoise = 0.0;
+    
+  if ( addNoise ) 
+  {
+    if ( noiseOnly )
+    {      
+      adcNoise = NoiseFunction()->GetRandom()*pedestalSigma;
+    }
+    else
+    {
+      adcNoise = gRandom->Gaus(0.0,pedestalSigma);
+    }
+  }
+    
+  adc = TMath::Nint(padc + pedestalMean + adcNoise + 0.5);
+  
+  if ( adc < TMath::Nint(pedestalMean + fgNSigmas*pedestalSigma + 0.5) ) 
+  {
+    // this is an error only in specific cases
+    if ( !addNoise || (addNoise && noiseOnly) ) 
+    {
+      AliErrorClass(Form(" DE %04d Manu %04d Channel %02d "
+                         " a0 %7.2f a1 %7.2f thres %04d ped %7.2f pedsig %7.2f adcNoise %7.2f "
+                         " charge=%7.2f padc=%7.2f adc=%04d ZS=%04d fgNSigmas=%e addNoise %d noiseOnly %d ",
+                         pedestals.ID0(),pedestals.ID1(),channel, 
+                         a0, a1, thres, pedestalMean, pedestalSigma, adcNoise,
+                         charge, padc, adc, 
+                         TMath::Nint(pedestalMean + fgNSigmas*pedestalSigma + 0.5),
+                         fgNSigmas,addNoise,noiseOnly));
+    }
+    
+    adc = 0;
+  }
+  
+  // be sure we stick to 12 bits.
+  if ( adc > kMaxADC )
+  {
+    adc = kMaxADC;
+  }
+  
+  return adc;
 }
 
 //_____________________________________________________________________________
 void
-AliMUONDigitizerV3::AddOrUpdateDigit(TClonesArray& array, 
-                                     const AliMUONDigit& digit)
+AliMUONDigitizerV3::CreateInputDigitStores()
 {
-  //
-  // Add or update a digit, depending on whether there's already a digit
-  // for the corresponding channel.
-  //
-  Int_t ix = FindDigitIndex(array,digit);
+  /// Create input digit stores
+  /// 
   
-  if (ix>=0)
+  if (fInputDigitStores)
   {
-    AliMUONDigit* d = static_cast<AliMUONDigit*>(array.UncheckedAt(ix));
-    Bool_t ok = MergeDigits(digit,*d);
-    if (!ok)
+    AliFatal("Should be called only once !");
+  }
+  
+  fInputDigitStores = new TObjArray;
+  
+  fInputDigitStores->SetOwner(kTRUE);
+  
+  for ( Int_t iFile = 0; iFile < fManager->GetNinputs(); ++iFile )
+  {    
+    AliLoader* inputLoader = GetLoader(fManager->GetInputFolderName(iFile));
+    
+    inputLoader->LoadSDigits("READ");
+    
+    TTree* iTreeS = inputLoader->TreeS();
+    if (!iTreeS)
     {
-      AliError("Digits are not mergeable !");
+      AliFatal(Form("Could not get access to input file #%d",iFile));
     }
+    
+    fInputDigitStores->AddAt(AliMUONVDigitStore::Create(*iTreeS),iFile);
   }
-  else
-  {
-    ix = array.GetLast() + 1;
-    new(array[ix]) AliMUONDigit(digit);
-  }
-  
 }
 
 //_____________________________________________________________________________
 void
 AliMUONDigitizerV3::Exec(Option_t*)
 {
-  //
-  // Main method.
-  // We first loop over input files, and merge the sdigits we found there.
-  // We then digitize all the resulting sdigits
-  // And we finally generate the trigger outputs.
-  //
-  AliDebug(1, "Running digitizer.");
+  /// Main method.
+  /// We first loop over input files, and merge the sdigits we found there.
+  /// Second, we digitize all the resulting sdigits
+  /// Then we generate noise-only digits (for tracker only)
+  /// And we finally generate the trigger outputs.
+    
+  AliCodeTimerAuto("",0)
   
   if ( fManager->GetNinputs() == 0 )
   {
@@ -245,122 +474,336 @@ AliMUONDigitizerV3::Exec(Option_t*)
   
   Int_t nInputFiles = fManager->GetNinputs();
   
-  if ( fOutputData->TreeD() == 0x0 )
+  AliLoader* outputLoader = GetLoader(fManager->GetOutputFolderName());
+  
+  outputLoader->MakeDigitsContainer();
+  
+  TTree* oTreeD = outputLoader->TreeD();
+  
+  if (!oTreeD) 
   {
-    AliDebug(1,"Calling MakeDigitsContainer");
-    fOutputData->GetLoader()->MakeDigitsContainer();
+    AliFatal("Cannot create output TreeD");
   }
-  fOutputData->MakeBranch("D,GLT");
-  fOutputData->SetTreeAddress("D,GLT");
-  
+
   // Loop over all the input files, and merge the sdigits found in those
   // files.
+  
   for ( Int_t iFile = 0; iFile < nInputFiles; ++iFile )
-  {    
-    AliMUONData* inputData = GetDataAccess(fManager->GetInputFolderName(iFile));
-    if (!inputData)
+  {  
+    AliLoader* inputLoader = GetLoader(fManager->GetInputFolderName(iFile));
+
+    inputLoader->LoadSDigits("READ");
+
+    TTree* iTreeS = inputLoader->TreeS();
+    if (!iTreeS)
     {
       AliFatal(Form("Could not get access to input file #%d",iFile));
     }
 
-    inputData->GetLoader()->LoadSDigits("READ");
-    inputData->SetTreeAddress("S");
-    inputData->GetSDigits();
+    if (!fInputDigitStores)
+    {
+      CreateInputDigitStores();      
+    }
+    
+    AliMUONVDigitStore* dstore = static_cast<AliMUONVDigitStore*>(fInputDigitStores->At(iFile));
+    
+    dstore->Connect(*iTreeS);
+    
+    iTreeS->GetEvent(0);
+
+    MergeWithSDigits(fDigitStore,*dstore,fManager->GetMask(iFile));
 
-    MergeWithSDigits(*fOutputData,*inputData,fManager->GetMask(iFile));
+    inputLoader->UnloadSDigits();
     
-    inputData->ResetSDigits();
-    inputData->GetLoader()->UnloadSDigits();
-    delete inputData;
+    dstore->Clear();
   }
+
   
   // At this point, we do have digit arrays (one per chamber) which contains 
   // the merging of all the sdigits of the input file(s).
   // We now massage them to apply the detector response, i.e. this
   // is here that we do the "digitization" work.
   
-  ApplyResponse();
+  if (!fOutputDigitStore)
+  {
+    fOutputDigitStore = fDigitStore->Create();
+  }
+  
+  if ( fGenerateNoisyDigits>=2 )
+  {
+    // Generate noise-only digits for trigger.
+    GenerateNoisyDigitsForTrigger(*fDigitStore);
+  }
+  ApplyResponse(*fDigitStore,*fOutputDigitStore);
 
+  if ( fGenerateNoisyDigits )
+  {
+    // Generate noise-only digits for tracker.
+    GenerateNoisyDigits(*fOutputDigitStore);
+  }
+  
   // We generate the global and local trigger decisions.
-  fTriggerProcessor->ExecuteTask();
+  fTriggerProcessor->Digits2Trigger(*fOutputDigitStore,*fTriggerStore);
+
+  // Prepare output tree
+  Bool_t okD = fOutputDigitStore->Connect(*oTreeD,kFALSE);
+  Bool_t okT = fTriggerStore->Connect(*oTreeD,kFALSE);
+  if (!okD || !okT)
+  {
+    AliError(Form("Could not make branch : Digit %d Trigger %d",okD,okT));
+    return;
+  }
   
   // Fill the output treeD
-  fOutputData->Fill("D,GLT");
+  oTreeD->Fill();
   
   // Write to the output tree(D).
   // Please note that as GlobalTrigger, LocalTrigger and Digits are in the same
   // tree (=TreeD) in different branches, this WriteDigits in fact writes all of 
   // the 3 branches.
-  fOutputData->GetLoader()->WriteDigits("OVERWRITE");
+  outputLoader->WriteDigits("OVERWRITE");  
+  
+  outputLoader->UnloadDigits();
   
   // Finally, we clean up after ourselves.
-  fOutputData->ResetDigits();
-  fOutputData->ResetTrigger();
-  fOutputData->GetLoader()->UnloadDigits();
+  fTriggerStore->Clear();
+  fDigitStore->Clear();
+  fOutputDigitStore->Clear();
 }
 
+
 //_____________________________________________________________________________
-Int_t
-AliMUONDigitizerV3::FindDigitIndex(TClonesArray& array, const AliMUONDigit& digit)
+void
+AliMUONDigitizerV3::GenerateNoisyDigits(AliMUONVDigitStore& digitStore)
 {
-  // 
-  // Return the index of digit within array, if that digit is there, 
-  // otherwise returns -1
-  //
-  // FIXME: this is of course not the best implementation you can think of.
-  // Reconsider the use of hit/digit map... ? (but be sure it's needed!)
-  //
-  Int_t n = array.GetEntriesFast();
-  for ( Int_t i = 0; i < n; ++i )
+  /// According to a given probability, generate digits that
+  /// have a signal above the noise cut (ped+n*sigma_ped), i.e. digits
+  /// that are "only noise".
+  
+  AliCodeTimerAuto("",0)
+  
+  for ( Int_t i = 0; i < AliMUONConstants::NTrackingCh(); ++i )
   {
-    AliMUONDigit* d = static_cast<AliMUONDigit*>(array.UncheckedAt(i));
-    if ( d->DetElemId() == digit.DetElemId() &&
-         d->PadX() == digit.PadX() &&
-         d->PadY() == digit.PadY() && 
-         d->Cathode() == digit.Cathode() )
+    AliMpDEIterator it;
+  
+    it.First(i);
+  
+    while ( !it.IsDone() )
     {
-      return i;
+      for ( Int_t cathode = 0; cathode < 2; ++cathode )
+      {
+        GenerateNoisyDigitsForOneCathode(digitStore,it.CurrentDEId(),cathode);
+      }
+      it.Next();
     }
   }
-  return -1;
 }
+//_____________________________________________________________________________
+void
+AliMUONDigitizerV3::GenerateNoisyDigitsForOneCathode(AliMUONVDigitStore& digitStore,
+                                                     Int_t detElemId, Int_t cathode)
+{
+  /// Generate noise-only digits for one cathode of one detection element.
+  /// Called by GenerateNoisyDigits()
+  
+  const AliMpVSegmentation* seg 
+    = AliMpSegmentation::Instance()->GetMpSegmentation(detElemId,AliMp::GetCathodType(cathode));
+  Int_t nofPads = seg->NofPads();
+  
+  Int_t maxIx = seg->MaxPadIndexX();
+  Int_t maxIy = seg->MaxPadIndexY();
+  
+  static const Double_t kProbToBeOutsideNsigmas = TMath::Erfc(fgNSigmas/TMath::Sqrt(2.0)) / 2. ;
+  
+  Int_t nofNoisyPads = TMath::Nint(kProbToBeOutsideNsigmas*nofPads);
+  if ( !nofNoisyPads ) return;
+  
+  nofNoisyPads = 
+    TMath::Nint(gRandom->Gaus(nofNoisyPads,
+                              nofNoisyPads/TMath::Sqrt(nofNoisyPads)));
+  
+  AliDebug(3,Form("DE %d cath %d nofNoisyPads %d",detElemId,cathode,nofNoisyPads));
+  
+  for ( Int_t i = 0; i < nofNoisyPads; ++i ) 
+  {
+    Int_t ix(-1);
+    Int_t iy(-1);
+    AliMpPad pad;
+    
+    do {
+      ix = gRandom->Integer(maxIx+1);
+      iy = gRandom->Integer(maxIy+1);
+      pad = seg->PadByIndices(ix,iy,kFALSE);
+    } while ( !pad.IsValid() );
+
+    Int_t manuId = pad.GetManuId();
+    Int_t manuChannel = pad.GetManuChannel();    
+
+    AliMUONVCalibParam* pedestals = fCalibrationData->Pedestals(detElemId,manuId);
+    
+    if (!pedestals) 
+    {
+      // no pedestal available for this channel, simply give up
+      continue;
+    }
+    
+    AliMUONVDigit* d = digitStore.CreateDigit(detElemId,manuId,manuChannel,cathode);
+    
+    d->SetPadXY(ix,iy);
+    
+    d->SetCharge(0.0); // charge is zero, the ApplyResponseToTrackerDigit will add the noise
+    d->NoiseOnly(kTRUE);
+    ApplyResponseToTrackerDigit(*d,kTRUE);
+    if ( d->ADC() > 0 )
+    {
+      Bool_t ok = digitStore.Add(*d,AliMUONVDigitStore::kDeny);
+      // this can happen (that we randomly chose a digit that is
+      // already there). We simply ignore this, but log the occurence
+      // to cross-check that it's not too frequent.
+      if (!ok)
+      {
+        fLogger->Log("Collision while adding noiseOnly digit");
+      }
+      else
+      {
+        fLogger->Log("Added noiseOnly digit");
+      }
+    }
+    delete d;
+  }
+}
+
 
 //_____________________________________________________________________________
-AliMUONData* 
-AliMUONDigitizerV3::GetDataAccess(const TString& folderName)
+void
+AliMUONDigitizerV3::GenerateNoisyDigitsForTrigger(AliMUONVDigitStore& digitStore)
 {
-  //
-  // Create an AliMUONData to deal with data found in folderName.
-  //
-  AliDebug(1,Form("Getting access to folder %s",folderName.Data()));
-  AliRunLoader* runLoader = AliRunLoader::GetRunLoader(folderName);
-  if (!runLoader)
+  /// Generate noise-only digits for one cathode of one detection element.
+  /// Called by GenerateNoisyDigits()
+
+  if ( !fNoiseFunctionTrig )
   {
-    AliError(Form("Could not get RunLoader from folder %s",folderName.Data()));
-    return 0x0;
+    fNoiseFunctionTrig = new TF1("AliMUONDigitizerV3::fNoiseFunctionTrig","landau",
+                                50.,270.);
+    
+    fNoiseFunctionTrig->SetParameters(3.91070e+02, 9.85026, 9.35881e-02);
   }
-  AliLoader* loader = static_cast<AliLoader*>(runLoader->GetLoader("MUONLoader"));
+
+  AliMpPad pad[2];
+  AliMUONVDigit *d[2]={0x0};
+
+  for ( Int_t chamberId = AliMUONConstants::NTrackingCh(); chamberId < AliMUONConstants::NCh(); ++chamberId )
+  {
+  
+    Int_t nofNoisyPads = 50;
+
+    Float_t r=-1, fi = 0., gx, gy, x, y, z, xg01, yg01, zg, xg02, yg02;
+    AliMpDEIterator it;
+  
+    AliDebug(3,Form("Chamber %d nofNoisyPads %d",chamberId,nofNoisyPads));
+
+    for ( Int_t i = 0; i < nofNoisyPads; ++i )
+    {
+      //printf("Generating noise %i\n",i);
+       Int_t ix(-1);
+       Int_t iy(-1);
+       Bool_t isOk = kFALSE;
+       Int_t detElemId = -1;
+       do {
+         //r = gRandom->Landau(9.85026, 9.35881e-02);
+           r = fNoiseFunctionTrig->GetRandom();
+           fi = 2. * TMath::Pi() * gRandom->Rndm();
+           //printf("r = %f\tfi = %f\n", r, fi);
+           gx = r * TMath::Cos(fi);
+           gy = r * TMath::Sin(fi);
+
+           for ( it.First(chamberId); ! it.IsDone(); it.Next() ){
+               Int_t currDetElemId = it.CurrentDEId();
+               const AliMpVSegmentation* seg
+                   = AliMpSegmentation::Instance()->GetMpSegmentation(currDetElemId,AliMp::GetCathodType(0));
+               if (!seg) continue;
+               Float_t deltax = seg->GetDimensionX();
+               Float_t deltay = seg->GetDimensionY();
+               GetTransformer()->Local2Global(currDetElemId, -deltax, -deltay, 0, xg01, yg01, zg);
+               GetTransformer()->Local2Global(currDetElemId,  deltax,  deltay, 0, xg02, yg02, zg);
+               Float_t xg1 = xg01, xg2 = xg02, yg1 = yg01, yg2 = yg02;
+               if(xg01>xg02){
+                   xg1 = xg02;
+                   xg2 = xg01;
+               }
+               if(yg01>yg02){
+                   yg1 = yg02;
+                   yg2 = yg01;
+               }
+               if(gx>=xg1 && gx<=xg2 && gy>=yg1 && gy<=yg2){
+                   detElemId = currDetElemId;
+                   GetTransformer()->Global2Local(detElemId, gx, gy, 0, x, y, z);
+                   pad[0] = seg->PadByPosition(x,y,kFALSE);
+                   if(!pad[0].IsValid()) continue;
+                   isOk = kTRUE;
+                   break;
+               }
+           } // loop on slats
+       } while ( !isOk );
+
+       const AliMpVSegmentation* seg1
+           = AliMpSegmentation::Instance()->GetMpSegmentation(detElemId,AliMp::GetCathodType(1));
+       pad[1] = seg1->PadByPosition(x,y,kFALSE);
+
+       for ( Int_t cathode = 0; cathode < 2; ++cathode ){
+         Int_t manuId = pad[cathode].GetLocalBoardId(0);
+         Int_t manuChannel = pad[cathode].GetLocalBoardChannel(0);    
+         d[cathode] = digitStore.CreateDigit(detElemId,manuId,manuChannel,cathode);
+         ix = pad[cathode].GetIx();
+         iy = pad[cathode].GetIy();
+         d[cathode]->SetPadXY(ix,iy);
+         //d[cathode].SetSignal(1);
+         //d[cathode].SetPhysicsSignal(0);
+         d[cathode]->SetCharge(1);
+         d[cathode]->NoiseOnly(kTRUE);
+         AliDebug(3,Form("Adding a pure noise digit :"));
+
+         Bool_t ok = digitStore.Add(*d[cathode],AliMUONVDigitStore::kDeny);
+         if (!ok)
+         {
+             fLogger->Log("Collision while adding TriggerNoise digit");
+         }
+         else
+         {
+             fLogger->Log("Added triggerNoise digit");
+         }
+       } //loop on cathodes
+    } // loop on noisy pads
+  } // loop on chambers
+}
+
+
+//_____________________________________________________________________________
+AliLoader*
+AliMUONDigitizerV3::GetLoader(const TString& folderName)
+{
+  /// Get a MUON loader
+
+  AliDebug(2,Form("Getting access to folder %s",folderName.Data()));
+  AliLoader* loader = AliRunLoader::GetDetectorLoader("MUON",folderName.Data());
   if (!loader)
   {
     AliError(Form("Could not get MuonLoader from folder %s",folderName.Data()));
     return 0x0;
   }
-  AliMUONData* data = new AliMUONData(loader,"MUON","MUONDataForDigitOutput");
-  AliDebug(1,Form("AliMUONData=%p loader=%p",data,loader));
-  return data;
+  return loader;
 }
 
 //_____________________________________________________________________________
 Bool_t
 AliMUONDigitizerV3::Init()
 {
-  //
-  // Initialization of the TTask :
-  // a) set the outputData pointer
-  // b) create the calibrationData, according to run number
-  // c) create the trigger processing task
-  //
-  AliDebug(1,"");
+  /// Initialization of the TTask :
+  /// a) create the calibrationData, according to run number
+  /// b) create the trigger processing task
+
+  AliDebug(2,"");
   
   if ( fIsInitialized )
   {
@@ -374,105 +817,93 @@ AliMUONDigitizerV3::Init()
     return kFALSE;
   }
   
-  fOutputData = GetDataAccess(fManager->GetOutputFolderName());
-  if (!fOutputData)
+  // Load mapping
+  if ( ! AliMpCDB::LoadDDLStore() ) {
+    AliFatal("Could not access mapping from OCDB !");
+  }
+  
+  if (!fCalibrationData)
+      AliFatal("Calibration data object not defined");
+
+  if ( !fCalibrationData->Pedestals() )
   {
-    AliError("Can not perform digitization. I'm sorry");
-    return kFALSE;
+    AliFatal("Could not access pedestals from OCDB !");
+  }
+  if ( !fCalibrationData->Gains() )
+  {
+    AliFatal("Could not access gains from OCDB !");
   }
-  AliDebug(1,Form("fOutputData=%p",fOutputData));
   
-  AliRunLoader* runLoader = fOutputData->GetLoader()->GetRunLoader();
-  AliRun* galice = runLoader->GetAliRun();  
-  Int_t runnumber = galice->GetRunNumber();
   
-  fCalibrationData = new AliMUONCalibrationData(runnumber);
+   AliInfo("Using trigger configuration from CDB");
   
-  switch (fTriggerCodeVersion)
-  {
-    case kTriggerDecision:
-      fTriggerProcessor = new AliMUONTriggerDecisionV1(fOutputData);
-      break;
-    case kTriggerElectronics:
-      fTriggerProcessor = new AliMUONTriggerElectronics(fOutputData);
-      break;
-    default:
-      AliFatal("Unknown trigger processor type");
-      break;
-  }
-  AliDebug(1,Form("Using the following trigger code %s - %s",
-                  fTriggerProcessor->GetName(),fTriggerProcessor->GetTitle()));
+  fTriggerProcessor = new AliMUONTriggerElectronics(fCalibrationData);
+  
+  AliDebug(1, Form("Will %s generate noise-only digits for tracker",
+                     (fGenerateNoisyDigits ? "":"NOT")));
+  
+  fTriggerUtilities = new AliMUONTriggerUtilities(fCalibrationData);
+  
+  if ( muon()->GetTriggerEffCells() ) {
+    // Apply trigger efficiency
+    AliDebug(1, "Will apply trigger efficiency");
+    fTriggerEfficiency = new AliMUONTriggerChamberEfficiency(fCalibrationData->TriggerEfficiency());
+  }
+
   fIsInitialized = kTRUE;
   return kTRUE;
 }
 
+
 //_____________________________________________________________________________
-Bool_t
-AliMUONDigitizerV3::MergeDigits(const AliMUONDigit& src, 
-                                AliMUONDigit& srcAndDest)
+Int_t AliMUONDigitizerV3::GetArrayIndex(Int_t cathode, Int_t trigCh, Int_t localCircuit)
 {
-  //
-  // Merge 2 digits (src and srcAndDest) into srcAndDest.
-  //
-  AliDebug(1,"Merging the following digits:");
-  StdoutToAliDebug(1,src.Print("tracks"););
-  StdoutToAliDebug(1,srcAndDest.Print("tracks"););
-  
-  Bool_t check = ( src.DetElemId() == srcAndDest.DetElemId() &&
-                   src.PadX() == srcAndDest.PadX() &&
-                   src.PadY() == srcAndDest.PadY() &&
-                   src.Cathode() == srcAndDest.Cathode() );
-  if (!check)
-  {
-    return kFALSE;
-  }
-  
-  srcAndDest.AddSignal(src.Signal());
-  srcAndDest.AddPhysicsSignal(src.Physics());
-  for ( Int_t i = 0; i < src.Ntracks(); ++i )
-  {
-    srcAndDest.AddTrack(src.Track(i),src.TrackCharge(i));
-  }
-  StdoutToAliDebug(1,cout << "result:"; srcAndDest.Print("tracks"););
-  return kTRUE;
+  /// Get index of array with trigger status map or efficiency
+  return
+    AliMUONConstants::NTriggerCircuit() * AliMUONConstants::NTriggerCh() * cathode +
+    AliMUONConstants::NTriggerCircuit() * trigCh + localCircuit-1;
 }
 
+
 //_____________________________________________________________________________
 void 
-AliMUONDigitizerV3::MergeWithSDigits(AliMUONData& outputData, 
-                                     const AliMUONData& inputData, Int_t mask)
+AliMUONDigitizerV3::MergeWithSDigits(AliMUONVDigitStore*& outputStore,
+                                     const AliMUONVDigitStore& input,
+                                     Int_t mask)
 {
-  //
-  // Merge the sdigits in inputData with the digits already present in outputData
-  //
-  AliDebug(1,"");
-  
-       for ( Int_t ich = 0; ich < AliMUONConstants::NCh(); ++ich )
-       {
-    TClonesArray* iDigits = inputData.SDigits(ich); 
-    TClonesArray* oDigits = outputData.Digits(ich);
-    if (!iDigits)
+  /// Merge the sdigits in inputData with the digits already present in outputData
+  
+  if ( !outputStore ) outputStore = input.Create();
+  
+  TIter next(input.CreateIterator());
+  AliMUONVDigit* sdigit;
+  
+  while ( ( sdigit = static_cast<AliMUONVDigit*>(next()) ) )
+  {
+    // Update the track references using the mask.
+    // FIXME: this is dirty, for backward compatibility only.
+    // Should re-design all this way of keeping track of MC information...
+    if ( mask ) sdigit->PatchTracks(mask);
+    // Then add or update the digit to the output.
+    AliMUONVDigit* added = outputStore->Add(*sdigit,AliMUONVDigitStore::kMerge);
+    if (!added)
     {
-      AliError(Form("Could not get sdigits for ich=%d",ich));
-      return;
+      AliError("Could not add digit in merge mode");
     }
-    Int_t nSDigits = iDigits->GetEntriesFast();
-    for ( Int_t k = 0; k < nSDigits; ++k )
-               {
-                       AliMUONDigit* sdigit = static_cast<AliMUONDigit*>(iDigits->UncheckedAt(k));
-      if (!sdigit)
-      {
-        AliError(Form("Could not get sdigit for ich=%d and k=%d",ich,k));
-      }
-      else
-      {
-        // Update the track references using the mask.
-        // FIXME: this is dirty, for backward compatibility only.
-        // Should re-design all this way of keeping track of MC information...
-        if ( mask ) sdigit->PatchTracks(mask);
-        // Then add or update the digit to the output.
-        AddOrUpdateDigit(*oDigits,*sdigit);
-      }
-               }   
   }
 }
+
+//_____________________________________________________________________________
+TF1*
+AliMUONDigitizerV3::NoiseFunction()
+{
+  /// Return noise function
+  static TF1* f = 0x0;
+  if (!f)
+  {
+    f = new TF1("AliMUONDigitizerV3::NoiseFunction","gaus",fgNSigmas,fgNSigmas*10);
+    f->SetParameters(1,0,1);
+  }
+  return f;
+}
+