]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - MUON/AliMUONDigitizerV3.cxx
Updated list of MUON libraries
[u/mrichter/AliRoot.git] / MUON / AliMUONDigitizerV3.cxx
index a5b0caf2d33cc38a953453da18fd50b7dd91e255..6df5a2df058a4152403c80f51c980e53bf14f6f3 100644 (file)
 
 // $Id$
 
+
 #include "AliMUONDigitizerV3.h"
 
-#include "AliLog.h"
+#include "AliMUON.h"
 #include "AliMUONCalibrationData.h"
-#include "AliMUONChamber.h"
+#include "AliCDBManager.h"
 #include "AliMUONConstants.h"
-#include "AliMUONData.h"
+#include "AliMUONSimData.h"
+#include "AliMUONDataIterator.h"
 #include "AliMUONDigit.h"
-#include "AliMUONTriggerDecisionV1.h"
+#include "AliMUONLogger.h"
+#include "AliMUONSegmentation.h"
+#include "AliMUONTriggerEfficiencyCells.h"
 #include "AliMUONTriggerElectronics.h"
-#include "AliMUONCalibParam.h"
+#include "AliMUONVCalibParam.h"
+
+#include "AliMpDEIterator.h"
 #include "AliMpDEManager.h"
+#include "AliMpIntPair.h"
+#include "AliMpPad.h"
 #include "AliMpStationType.h"
+#include "AliMpSegmentation.h"
+#include "AliMpVSegmentation.h"
+#include "AliMpDEManager.h"
+#include "AliMpCathodType.h"
+
 #include "AliRun.h"
 #include "AliRunDigitizer.h"
 #include "AliRunLoader.h"
-#include "Riostream.h"
-#include "TRandom.h"
-#include "TString.h"
+#include "AliLog.h"
+
+#include <Riostream.h>
+#include <TF1.h>
+#include <TMath.h>
+#include <TRandom.h>
+#include <TString.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"));
+  }
+}
 
+const Double_t AliMUONDigitizerV3::fgkNSigmas=3;
+
+/// \cond CLASSIMP
 ClassImp(AliMUONDigitizerV3)
+/// \endcond
 
 //_____________________________________________________________________________
 AliMUONDigitizerV3::AliMUONDigitizerV3(AliRunDigitizer* manager, 
-                                       ETriggerCodeVersion triggerCodeVersion)
+                                       Bool_t generateNoisyDigits)
 : AliDigitizer(manager),
-fZeroSuppression(6),
-fSaturation(3000),
 fIsInitialized(kFALSE),
 fOutputData(0x0),
 fCalibrationData(0x0),
 fTriggerProcessor(0x0),
-fTriggerCodeVersion(triggerCodeVersion)
+fTriggerEfficiency(0x0),
+fFindDigitIndexTimer(),
+fGenerateNoisyDigitsTimer(),
+fExecTimer(),
+fNoiseFunction(0x0),
+  fGenerateNoisyDigits(generateNoisyDigits),
+  fLogger(new AliMUONLogger(1000))
 {
+  /// Ctor.
+
   AliDebug(1,Form("AliRunDigitizer=%p",fManager));
+  fGenerateNoisyDigitsTimer.Start(kTRUE); fGenerateNoisyDigitsTimer.Stop();
+  fExecTimer.Start(kTRUE); fExecTimer.Stop();
+  fFindDigitIndexTimer.Start(kTRUE); fFindDigitIndexTimer.Stop();
 }
 
 //_____________________________________________________________________________
 AliMUONDigitizerV3::~AliMUONDigitizerV3()
 {
+  /// Dtor. Note we're the owner of some pointers.
+
   AliDebug(1,"dtor");
+
   delete fOutputData;
   delete fCalibrationData;
   delete fTriggerProcessor;
+  delete fNoiseFunction;
+  
+  AliDebug(1, Form("Execution time for FindDigitIndex() : R:%.2fs C:%.2fs",
+               fFindDigitIndexTimer.RealTime(),fFindDigitIndexTimer.CpuTime()));
+  if ( fGenerateNoisyDigits )
+  {
+    AliDebug(1, Form("Execution time for GenerateNoisyDigits() : R:%.2fs C:%.2fs",
+                 fGenerateNoisyDigitsTimer.RealTime(),
+                 fGenerateNoisyDigitsTimer.CpuTime()));
+  }
+  AliDebug(1, Form("Execution time for Exec() : R:%.2fs C:%.2fs",
+               fExecTimer.RealTime(),fExecTimer.CpuTime()));
+  AliInfo("Summary of messages");
+  fLogger->Print();
+  
+  delete fLogger;
 }
 
 //_____________________________________________________________________________
 void 
-AliMUONDigitizerV3::ApplyResponseToDigit(AliMUONDigit& digit)
+AliMUONDigitizerV3::ApplyResponseToTrackerDigit(AliMUONDigit& digit, Bool_t addNoise)
 {
-  // For trigger digits, simply sets the digit's charge to 0 or 1.
-  //
-  // 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 ZeroSuppression() level (thus simulating
-  //   zero suppression).
-  //
+  /// For tracking digits, starting from an ideal digit's charge, we :
+  ///
+  /// - add some noise (thus leading to a realistic charge), if requested to do so
+  /// - 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 kMaxADC = (1<<12)-1; // We code the charge on a 12 bits ADC.
   
-  Int_t detElemId = digit.DetElemId();
-  AliMpStationType stationType = AliMpDEManager::GetStationType(detElemId);
-  if ( stationType == kStationTrigger )
+  Float_t signal = digit.Signal();
+  
+  if ( !addNoise )
   {
-    if ( digit.Signal() > 0  )
-    {
-      digit.SetSignal(1);
-    }
-    else
-    {
-      digit.SetSignal(0);
-    }
-    return;    
+    digit.SetADC(TMath::Nint(signal));
+    return;
   }
   
-  // The following is for tracking digits only.
+  Int_t detElemId = digit.DetElemId();
   
   Int_t manuId = digit.ManuId();
   Int_t manuChannel = digit.ManuChannel();
   
-  AliMUONCalibParam* pedestal = fCalibrationData->Pedestal(detElemId,manuId,manuChannel);
+  AliMUONVCalibParam* pedestal = fCalibrationData->Pedestals(detElemId,manuId);
   if (!pedestal)
   {
-    AliFatal(Form("Could not get pedestal for DE=%d manuId=%d manuChannel=%d",
-                  detElemId,manuId,manuChannel));    
+    fLogger->Log(Form("%s:%d:Could not get pedestal for DE=%4d manuId=%4d. Disabling.",
+                      __FILE__,__LINE__,
+                      detElemId,manuId));
+    digit.SetPhysicsSignal(0);
+    digit.SetSignal(0);
+    digit.SetADC(0);
+    return;    
   }
-  Float_t adc_noise = gRandom->Gaus(0.0,pedestal->Sigma());
+  Float_t pedestalMean = pedestal->ValueAsFloat(manuChannel,0);
+  Float_t pedestalSigma = pedestal->ValueAsFloat(manuChannel,1);
   
-  AliMUONCalibParam* gain = fCalibrationData->Gain(detElemId,manuId,manuChannel);
+  AliMUONVCalibParam* gain = fCalibrationData->Gains(detElemId,manuId);
   if (!gain)
   {
-    AliFatal(Form("Could not get gain for DE=%d manuId=%d manuChannel=%d",
-                  detElemId,manuId,manuChannel));    
-  }
+    fLogger->Log(Form("%s:%d:Could not get gain for DE=%4d manuId=%4d. Disabling.",
+                      __FILE__,__LINE__,
+                      detElemId,manuId));
+    digit.SetPhysicsSignal(0);
+    digit.SetSignal(0);
+    digit.SetADC(0);
+    return;        
+  }    
+  Float_t gainMean = gain->ValueAsFloat(manuChannel,0);
   
-  Float_t signal_noise = adc_noise*gain->Mean();
+  Float_t adcNoise = gRandom->Gaus(0.0,pedestalSigma);
   
-  Float_t signal = digit.Signal() + signal_noise;
   Int_t adc;
   
-  if ( signal > fSaturation )
+  if ( gainMean < 1E-6 )
   {
-    adc = fSaturation;
-    digit.Saturated(kTRUE);
+    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;
   }
   else
   {
-    if ( gain->Mean() < 1E-6 )
-    {
-      AliFatal(Form("Got a too small gain %e for DE=%d manuId=%d manuChannel=%d",
-                    gain->Mean(),detElemId,manuId,manuChannel));
-    }
-    adc = TMath::Nint( signal / gain->Mean() + pedestal->Mean() );
+    adc = TMath::Nint( signal / gainMean + pedestalMean + adcNoise);///
     
-    if ( adc <= pedestal->Mean() + 3.0*pedestal->Sigma() ) 
+    if ( adc <= pedestalMean + fgkNSigmas*pedestalSigma ) 
     {
       adc = 0;
     }
   }
+  
+  // be sure we stick to 12 bits.
+  if ( adc > kMaxADC )
+  {
+    adc = kMaxADC;
+  }
+  
   digit.SetPhysicsSignal(TMath::Nint(signal));
   digit.SetSignal(adc);
   digit.SetADC(adc);
 }
 
+//_____________________________________________________________________________
+void 
+AliMUONDigitizerV3::ApplyResponseToTriggerDigit(AliMUONDigit& digit)
+{
+  /// \todo add comment
+
+  if ( !fTriggerEfficiency ) return;
+
+  if (digit.IsEfficiencyApplied()) return;
+
+  AliMUONDigit* correspondingDigit = FindCorrespondingDigit(digit);
+
+  if (!correspondingDigit) return; //reject bad correspondences
+
+  Int_t detElemId = digit.DetElemId();
+
+  AliMpSegmentation* segmentation = AliMpSegmentation::Instance();
+  const AliMpVSegmentation* segment[2] = 
+  {
+    segmentation->GetMpSegmentation(detElemId,AliMp::GetCathodType(digit.Cathode())), 
+    segmentation->GetMpSegmentation(detElemId,AliMp::GetCathodType(correspondingDigit->Cathode()))
+  };
+
+  AliMpPad pad[2] = 
+  {
+    segment[0]->PadByIndices(AliMpIntPair(digit.PadX(),digit.PadY()),kTRUE), 
+    segment[1]->PadByIndices(AliMpIntPair(correspondingDigit->PadX(),correspondingDigit->PadY()),kTRUE)
+  };
+
+  Int_t p0(1);
+  if (digit.Cathode()==0) p0=0;
+
+  AliMpIntPair location = pad[p0].GetLocation(0);
+  Int_t nboard = location.GetFirst();
+
+  Bool_t isTrig[2];
+
+  fTriggerEfficiency->IsTriggered(detElemId, nboard-1, 
+                                  isTrig[0], isTrig[1]);
+  digit.EfficiencyApplied(kTRUE);
+  correspondingDigit->EfficiencyApplied(kTRUE);
+
+  if (!isTrig[digit.Cathode()])
+  {
+         digit.SetSignal(0);
+  }
+  
+  if ( &digit != correspondingDigit )
+  {
+         if (!isTrig[correspondingDigit->Cathode()])
+    {
+      correspondingDigit->SetSignal(0);
+         }
+  }
+}
+
 //_____________________________________________________________________________
 void
 AliMUONDigitizerV3::ApplyResponse()
 {
+  /// Loop over all chamber digits, and apply the response to them
+  /// Note that this method may remove digits.
+
+  const Bool_t kAddNoise = kTRUE;
+  
   for ( Int_t ich = 0; ich < AliMUONConstants::NCh(); ++ich )
-       {
+  {
     TClonesArray* digits = fOutputData->Digits(ich);
     Int_t n = digits->GetEntriesFast();
+    Bool_t trackingChamber = ( ich < AliMUONConstants::NTrackingCh() );
     for ( Int_t i = 0; i < n; ++i )
     {
       AliMUONDigit* d = static_cast<AliMUONDigit*>(digits->UncheckedAt(i));
-      ApplyResponseToDigit(*d);
+      if ( !d ) continue; // that digit might have been removed
+      if ( trackingChamber )
+      {
+        ApplyResponseToTrackerDigit(*d,kAddNoise);
+      }
+      else
+      {
+        ApplyResponseToTriggerDigit(*d);
+      }
       if ( d->Signal() <= 0 )
       {
         digits->RemoveAt(i);
       }
     }
+    digits->Compress(); // only do the compress at the end in order not to
+    // change the n = digits->GetEntriesFast()
   }    
   
-  for ( Int_t ich = 0; ich < AliMUONConstants::NCh(); ++ich )
-       {
-    fOutputData->Digits(ich)->Compress();
-  }
+// The version below, using iterator, does not yet work (as the iterator
+// assumes it is reading digits from the tree, while in this case it's
+// writing...)
+//
+//  AliMUONDigit* digit(0x0);
+//
+//  // First loop on tracker digits
+//  AliMUONDataIterator tracker(fOutputData,"D",AliMUONDataIterator::kTrackingChambers);
+//  
+//  while ( ( digit = static_cast<AliMUONDigit*>(tracker.Next()) ) )
+//  {
+//    ApplyResponseToTrackerDigit(*digit);
+//    if ( digit->Signal() <= 0 )
+//    {
+//      tracker.Remove();
+//    }    
+//    
+//  }
+//
+//  // Then loop on trigger digits
+//  AliMUONDataIterator trigger(fOutputData,"D",AliMUONDataIterator::kTriggerChambers);
+//  
+//  while ( ( digit = static_cast<AliMUONDigit*>(trigger.Next()) ) )
+//  {
+//    ApplyResponseToTriggerDigit(*digit,fOutputData);
+//    if ( digit->Signal() <= 0 )
+//    {
+//      trigger.Remove();
+//    }    
+//  }
 }
 
 //_____________________________________________________________________________
@@ -170,6 +347,9 @@ void
 AliMUONDigitizerV3::AddOrUpdateDigit(TClonesArray& array, 
                                      const AliMUONDigit& digit)
 {
+  /// Add or update a digit, depending on whether there's already a digit
+  /// for the corresponding channel.
+
   Int_t ix = FindDigitIndex(array,digit);
   
   if (ix>=0)
@@ -180,10 +360,6 @@ AliMUONDigitizerV3::AddOrUpdateDigit(TClonesArray& array,
     {
       AliError("Digits are not mergeable !");
     }
-    else
-    {
-      new(array[ix]) AliMUONDigit(*d);
-    }
   }
   else
   {
@@ -197,6 +373,12 @@ AliMUONDigitizerV3::AddOrUpdateDigit(TClonesArray& array,
 void
 AliMUONDigitizerV3::Exec(Option_t*)
 {
+  /// 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.
+    
   AliDebug(1, "Running digitizer.");
   
   if ( fManager->GetNinputs() == 0 )
@@ -211,11 +393,13 @@ AliMUONDigitizerV3::Exec(Option_t*)
     return;
   }
   
+  fExecTimer.Start(kFALSE);
+
   Int_t nInputFiles = fManager->GetNinputs();
   
   if ( fOutputData->TreeD() == 0x0 )
   {
-    AliDebug(1,"Calling MakeDigitsContainer");
+    AliDebug(2,"Calling MakeDigitsContainer");
     fOutputData->GetLoader()->MakeDigitsContainer();
   }
   fOutputData->MakeBranch("D,GLT");
@@ -225,19 +409,18 @@ AliMUONDigitizerV3::Exec(Option_t*)
   // files.
   for ( Int_t iFile = 0; iFile < nInputFiles; ++iFile )
   {    
-    AliMUONData* inputData = GetDataAccess(fManager->GetInputFolderName(iFile));
+    AliMUONSimData* inputData = GetDataAccess(fManager->GetInputFolderName(iFile));
     if (!inputData)
     {
       AliFatal(Form("Could not get access to input file #%d",iFile));
     }
-    AliDebug(1,Form("inputData=%p",inputData));
-    //FIXME: what about the fMask ???
+
     inputData->GetLoader()->LoadSDigits("READ");
-    TTree* treeS = inputData->GetLoader()->TreeS();
-    AliDebug(1,Form("TreeS=%p",treeS));
     inputData->SetTreeAddress("S");
     inputData->GetSDigits();
-    MergeWithSDigits(*fOutputData,*inputData);
+
+    MergeWithSDigits(*fOutputData,*inputData,fManager->GetMask(iFile));
+    
     inputData->ResetSDigits();
     inputData->GetLoader()->UnloadSDigits();
     delete inputData;
@@ -250,7 +433,11 @@ AliMUONDigitizerV3::Exec(Option_t*)
   
   ApplyResponse();
   
-  // 
+  if ( fGenerateNoisyDigits )
+  {
+    // Generate noise-only digits for tracker.
+    GenerateNoisyDigits();
+  }
   
   // We generate the global and local trigger decisions.
   fTriggerProcessor->ExecuteTask();
@@ -268,14 +455,65 @@ AliMUONDigitizerV3::Exec(Option_t*)
   fOutputData->ResetDigits();
   fOutputData->ResetTrigger();
   fOutputData->GetLoader()->UnloadDigits();
+  
+  fExecTimer.Stop();
 }
 
+//_____________________________________________________________________________
+AliMUONDigit* 
+AliMUONDigitizerV3::FindCorrespondingDigit(AliMUONDigit& digit) const
+{                                                
+  /// Find, if it exists, the digit corresponding to digit.Hit(), in the 
+  /// other cathode
+
+// Iterator does not yet work when writing digits (only works when reading,
+// which is not the case here)
+//
+//  AliMUONDataIterator it(data,"D",AliMUONDataIterator::kTriggerChambers);
+//  AliMUONDigit* cd;
+//
+//  while ( ( cd = static_cast<AliMUONDigit*>(it.Next()) ) )
+//  {
+//    if ( cd->DetElemId() == digit.DetElemId() &&
+//         cd->Hit() == digit.Hit() &&
+//         cd->Cathode() != digit.Cathode() )
+//    {
+//      break;
+//    }
+//  }
+  
+  Int_t ich = AliMpDEManager::GetChamberId(digit.DetElemId());  
+  TClonesArray* digits = fOutputData->Digits(ich);
+  Int_t n = digits->GetEntriesFast();
+  for ( Int_t i = 0; i < n; ++i )
+  {
+    AliMUONDigit* d = static_cast<AliMUONDigit*>(digits->UncheckedAt(i));
+    if ( d &&
+         d->DetElemId() == digit.DetElemId() &&
+         d->Hit() == digit.Hit() &&
+         d->Cathode() != digit.Cathode() )
+    {
+      return d;
+    }      
+  }    
+
+  return 0x0;
+}
+
+
 //_____________________________________________________________________________
 Int_t
-AliMUONDigitizerV3::FindDigitIndex(TClonesArray& array, const AliMUONDigit& digit)
+AliMUONDigitizerV3::FindDigitIndex(TClonesArray& array, 
+                                   const AliMUONDigit& digit) const
 {
-  // FIXME: this is of course not the best implementation you can think of.
-  // Reconsider the use of hit/digit map...
+  /// Return the index of digit within array, if that digit is there, 
+  /// otherwise returns -1
+  ///
+  /// \todo 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!)
+  
+  fFindDigitIndexTimer.Start(kFALSE);
+  
   Int_t n = array.GetEntriesFast();
   for ( Int_t i = 0; i < n; ++i )
   {
@@ -285,17 +523,144 @@ AliMUONDigitizerV3::FindDigitIndex(TClonesArray& array, const AliMUONDigit& digi
          d->PadY() == digit.PadY() && 
          d->Cathode() == digit.Cathode() )
     {
+      fFindDigitIndexTimer.Stop();
       return i;
     }
   }
+  fFindDigitIndexTimer.Stop();
   return -1;
 }
 
 //_____________________________________________________________________________
-AliMUONData* 
+void
+AliMUONDigitizerV3::GenerateNoisyDigits()
+{
+  /// 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".
+  
+  if ( !fNoiseFunction )
+  {
+    fNoiseFunction = new TF1("AliMUONDigitizerV3::fNoiseFunction","gaus",
+                             fgkNSigmas,fgkNSigmas*10);
+    
+    fNoiseFunction->SetParameters(1,0,1);
+  }
+  
+  fGenerateNoisyDigitsTimer.Start(kFALSE);
+  
+  for ( Int_t i = 0; i < AliMUONConstants::NTrackingCh(); ++i )
+  {
+    AliMpDEIterator it;
+  
+    it.First(i);
+  
+    while ( !it.IsDone() )
+    {
+      for ( Int_t cathode = 0; cathode < 2; ++cathode )
+      {
+        GenerateNoisyDigitsForOneCathode(it.CurrentDEId(),cathode);
+      }
+      it.Next();
+    }
+  }
+  
+  fGenerateNoisyDigitsTimer.Stop();
+}
+//_____________________________________________________________________________
+void
+AliMUONDigitizerV3::GenerateNoisyDigitsForOneCathode(Int_t detElemId, Int_t cathode)
+{
+  /// Generate noise-only digits for one cathode of one detection element.
+  /// Called by GenerateNoisyDigits()
+  
+  Int_t chamberId = AliMpDEManager::GetChamberId(detElemId);
+  TClonesArray* digits = fOutputData->Digits(chamberId);
+  
+  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(fgkNSigmas/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);
+    do {
+      ix = gRandom->Integer(maxIx+1);
+      iy = gRandom->Integer(maxIy+1);
+    } while ( !seg->HasPad(AliMpIntPair(ix,iy)) );
+    AliMUONDigit d;
+    d.SetDetElemId(detElemId);
+    d.SetCathode(cathode);
+    d.SetPadX(ix);
+    d.SetPadY(iy);
+    if ( FindDigitIndex(*digits,d) >= 0 )
+    {
+      // this digit is already there, and not noise-only, we simply skip it
+      continue;
+    }
+    AliMpPad pad = seg->PadByIndices(AliMpIntPair(ix,iy));
+    Int_t manuId = pad.GetLocation().GetFirst();
+    Int_t manuChannel = pad.GetLocation().GetSecond();
+    
+    d.SetElectronics(manuId,manuChannel);
+    
+    AliMUONVCalibParam* pedestals = fCalibrationData->Pedestals(detElemId,manuId);
+    
+    if (!pedestals) 
+    {
+      // no pedestal available for this channel, simply give up
+      return;
+    }
+    
+    Float_t pedestalMean = pedestals->ValueAsFloat(manuChannel,0);
+    Float_t pedestalSigma = pedestals->ValueAsFloat(manuChannel,1);
+    
+    Double_t ped = fNoiseFunction->GetRandom()*pedestalSigma;
+
+    d.SetSignal(TMath::Nint(ped+pedestalMean+0.5));
+    d.SetPhysicsSignal(0);
+    d.NoiseOnly(kTRUE);
+    AliDebug(3,Form("Adding a pure noise digit :"));
+//    StdoutToAliDebug(3,cout << "Before Response: " << endl; 
+//                     d.Print(););
+    ApplyResponseToTrackerDigit(d,kFALSE);
+    if ( d.Signal() > 0 )
+    {
+      AddOrUpdateDigit(*digits,d);
+    }
+    else
+    {
+      AliError("Pure noise below threshold. This should not happen. Not adding "
+               "this digit.");
+    }
+//    StdoutToAliDebug(3,cout << "After Response: " << endl; 
+//                     d.Print(););
+  }
+}
+
+//_____________________________________________________________________________
+AliMUONSimData* 
 AliMUONDigitizerV3::GetDataAccess(const TString& folderName)
 {
-  AliDebug(1,Form("Getting access to folder %s",folderName.Data()));
+  /// Create an AliMUONSimData to deal with data found in folderName.
+
+  AliDebug(2,Form("Getting access to folder %s",folderName.Data()));
   AliRunLoader* runLoader = AliRunLoader::GetRunLoader(folderName);
   if (!runLoader)
   {
@@ -308,8 +673,8 @@ AliMUONDigitizerV3::GetDataAccess(const TString& folderName)
     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));
+  AliMUONSimData* data = new AliMUONSimData(loader,"MUON","MUONDataForDigitOutput");
+  AliDebug(2,Form("AliMUONSimData=%p loader=%p",data,loader));
   return data;
 }
 
@@ -317,7 +682,12 @@ AliMUONDigitizerV3::GetDataAccess(const TString& folderName)
 Bool_t
 AliMUONDigitizerV3::Init()
 {
-  AliDebug(1,"");
+  /// Initialization of the TTask :
+  /// a) set the outputData pointer
+  /// b) create the calibrationData, according to run number
+  /// c) create the trigger processing task
+
+  AliDebug(2,"");
   
   if ( fIsInitialized )
   {
@@ -337,39 +707,52 @@ AliMUONDigitizerV3::Init()
     AliError("Can not perform digitization. I'm sorry");
     return kFALSE;
   }
-  AliDebug(1,Form("fOutputData=%p",fOutputData));
+  AliDebug(2,Form("fOutputData=%p",fOutputData));
   
-  AliRunLoader* runLoader = fOutputData->GetLoader()->GetRunLoader();
-  AliRun* galice = runLoader->GetAliRun();  
-  Int_t runnumber = galice->GetRunNumber();
+  Int_t runnumber = AliCDBManager::Instance()->GetRun();
   
   fCalibrationData = new AliMUONCalibrationData(runnumber);
+  if ( !fCalibrationData->Pedestals() )
+  {
+    AliFatal("Could not access pedestals from OCDB !");
+  }
+  if ( !fCalibrationData->Gains() )
+  {
+    AliFatal("Could not access gains from OCDB !");
+  }
+  fTriggerProcessor = new AliMUONTriggerElectronics(fOutputData,fCalibrationData);
   
-  switch (fTriggerCodeVersion)
-  {
-    case kTriggerDecision:
-      fTriggerProcessor = new AliMUONTriggerDecisionV1(fOutputData);
-      break;
-    case kTriggerElectronics:
-      fTriggerProcessor = new AliMUONTriggerElectronics(fOutputData);
-      break;
-    default:
-      AliFatal("Unknown trigger processor type");
-      break;
+  if ( muon()->GetTriggerEffCells() )
+  {
+    fTriggerEfficiency = fCalibrationData->TriggerEfficiency();
+    if ( fTriggerEfficiency )
+    {
+      AliDebug(1, "Will apply trigger efficiency");
+    }
+    else
+    {
+      AliFatal("I was requested to apply trigger efficiency, but I could "
+               "not get it !");
+    }
   }
-  AliDebug(1,Form("Using the following trigger code %s - %s",
-                  fTriggerProcessor->GetName(),fTriggerProcessor->GetTitle()));
+  
+  AliDebug(1, Form("Will %s generate noise-only digits for tracker",
+                     (fGenerateNoisyDigits ? "":"NOT")));
+
   fIsInitialized = kTRUE;
   return kTRUE;
 }
 
 //_____________________________________________________________________________
 Bool_t
-AliMUONDigitizerV3::MergeDigits(const AliMUONDigit& src, AliMUONDigit& srcAndDest)
+AliMUONDigitizerV3::MergeDigits(const AliMUONDigit& src, 
+                                AliMUONDigit& srcAndDest)
 {
-  AliDebug(1,"Merging the following digits:");
-  StdoutToAliDebug(1,src.Print(););
-  StdoutToAliDebug(1,srcAndDest.Print(););
+  /// Merge 2 digits (src and srcAndDest) into srcAndDest.
+
+  AliDebug(2,"Merging the following digits:");
+//  StdoutToAliDebug(2,src.Print("tracks"););
+//  StdoutToAliDebug(2,srcAndDest.Print("tracks"););
   
   Bool_t check = ( src.DetElemId() == srcAndDest.DetElemId() &&
                    src.PadX() == srcAndDest.PadX() &&
@@ -380,19 +763,24 @@ AliMUONDigitizerV3::MergeDigits(const AliMUONDigit& src, AliMUONDigit& srcAndDes
     return kFALSE;
   }
   
   srcAndDest.AddSignal(src.Signal());
   srcAndDest.AddPhysicsSignal(src.Physics());
-  StdoutToAliDebug(1,cout << "result:"; srcAndDest.Print(););
+  for ( Int_t i = 0; i < src.Ntracks(); ++i )
+  {
+    srcAndDest.AddTrack(src.Track(i),src.TrackCharge(i));
+  }
+//  StdoutToAliDebug(2,cout << "result:"; srcAndDest.Print("tracks"););
   return kTRUE;
 }
 
 //_____________________________________________________________________________
 void 
-AliMUONDigitizerV3::MergeWithSDigits(AliMUONData& outputData, 
-                                     const AliMUONData& inputData)
+AliMUONDigitizerV3::MergeWithSDigits(AliMUONSimData& outputData, 
+                                     const AliMUONSimData& inputData, Int_t mask)
 {
-  AliDebug(1,"");
+  /// Merge the sdigits in inputData with the digits already present in outputData
+
+  AliDebug(2,"");
   
        for ( Int_t ich = 0; ich < AliMUONConstants::NCh(); ++ich )
        {
@@ -407,37 +795,19 @@ AliMUONDigitizerV3::MergeWithSDigits(AliMUONData& outputData,
     for ( Int_t k = 0; k < nSDigits; ++k )
                {
                        AliMUONDigit* sdigit = static_cast<AliMUONDigit*>(iDigits->UncheckedAt(k));
-      // FIXME: Merging logic should go here.
-      // For testing, only put the digit in the output.
       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);
       }
-               }   
+    }   
   }
 }
-
-//------------------------------------------------------------------------
-//void AliMUONDigitizerv2::FillTriggerOutput()
-//{
-//  // Derived to fill TreeD and resets the trigger array in fMUONData.
-//  
-//     AliDebug(3,"Filling trees with trigger.");
-//     fMUONData->Fill("GLT");
-//     fMUONData->ResetTrigger();
-//}
-//
-//------------------------------------------------------------------------
-//void AliMUONDigitizerv2::CreateTrigger()
-//{
-//  fMUONData->MakeBranch("GLT");
-//  fMUONData->SetTreeAddress("GLT");
-//  fTrigDec->Digits2Trigger(); 
-//  FillTriggerOutput();       
-//  
-//}
-