]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - MUON/AliMUONReconstructor.cxx
Temporary solution to set RecoParam for the reconstruction
[u/mrichter/AliRoot.git] / MUON / AliMUONReconstructor.cxx
index 9be5a6e3996eea47f686b2aee36471d32c4b5457..748e4341e07f1d5773a8ba4a095e496b9a1fd088 100644 (file)
  **************************************************************************/
 /* $Id$ */
 
-//-----------------------------
-// Class AliMUONReconstructor  
-//-----------------------------
-// Class for the 
-// MUON track reconstruction
+//-----------------------------------------------------------------------------
+/// \class AliMUONReconstructor
+///
+/// Implementation of AliReconstructor for MUON subsystem.
+///
+/// The clustering mode and the associated parameters can be changed by using
+/// AliMUONRecoParam *muonRecoParam = AliMUONRecoParam::GetLow(High)FluxParam();
+/// muonRecoParam->Set...(); // see methods in AliMUONRecoParam.h for details
+/// AliRecoParam::Instance()->RegisterRecoParam(muonRecoParam);
+///
+/// Valid modes are :
+///
+/// SIMPLEFIT : use the AliMUONClusterFinderSimpleFit clusterizer
+///
+/// SIMPLEFITV3 : SIMPLEFIT with preclustering=PRECLUSTERV3
+///
+/// MLEM : use AliMUONClusterFinderMLEM and AliMUONPreClusterFinder for preclustering (default)
+/// MLEMV2 : MLEM with preclustering=PRECLUSTERV2
+/// MLEMV3 : MLEM with preclustering=PRECLUSTERV3
+///
+/// PRECLUSTER : use only AliMUONPreClusterFinder. Only for debug as
+/// the produced clusters do not have a position, hence the tracking will not
+/// work
+/// PRECLUSTERV2 : another version of the preclustering
+/// PRECLUSTERV3 : yet another version of the preclustering
+///
+/// COG : use AliMUONClusterFinderCOG clusterizer. Not really a production
+/// option either, as center-of-gravity is generally not a good estimate
+/// of the cluster position...
+///
+/// PEAKCOG : COG cluster finder around local maxima
+/// PEAKFIT : fit around local maxima with up to 3 peaks, COG otherwise
+///
+/// NOCLUSTERING : bypass completely the clustering stage
+///
+/// ------
+///
+/// The behavior of the MUON reconstruction can also be changed, besides
+/// the usual methods found in AliReconstruction (e.g. to disable tracking)
+/// by using AliReconstruction::SetOption("MUON",options)
+/// where options should be a space separated string.
+///
+/// Valid options are :
+///
+/// SAVEDIGITS : if you want to save in the TreeD the *calibrated* digits
+///     that are used for the clustering
+///
+/// DIGITSTOREV1 : use the V1 implementation of the digitstore 
+/// DIGITSTOREV2R : use the V2R implementation of the digitstore 
+///
+/// NOLOCALRECONSTRUCTION : for debug, to disable local reconstruction (and hence
+/// "recover" old behavior)
+///
+/// TRIGGERDISABLE : disable the treatment of MUON trigger
+///
+/// NOFASTTRKDECODER : makes the digit maker class use the non-high performance decoder
+///                  AliMUONPayloadTracker instead of  AliMUONTrackerDDLDecoder.
+///
+/// NOFASTTRGDECODER : makes the digit maker class use the non-high performance decoder
+///                  AliMUONPayloadTrigger instead of AliMUONTriggerDDLDecoder.
+///
+/// NOFASTDECODERS : makes the digit maker class use the non-high performance decoders
+///                  AliMUONPayloadTracker and AliMUONPayloadTrigger.
+///
+/// \author Laurent Aphecetche, Subatech
+//-----------------------------------------------------------------------------
 
 #include "AliMUONReconstructor.h"
 
-#include "AliESD.h"
-#include "AliESDMuonTrack.h"
-#include "AliLog.h"
-#include "AliMUON.h"
-#include "AliMUONConstants.h"
 #include "AliMUONCalibrationData.h"
-#include "AliMUONClusterFinderAZ.h"
-#include "AliMUONClusterReconstructor.h"
-#include "AliMUONData.h"
+#include "AliMUONClusterFinderCOG.h"
+#include "AliMUONClusterFinderMLEM.h"
+#include "AliMUONClusterFinderSimpleFit.h"
+#include "AliMUONClusterFinderPeakCOG.h"
+#include "AliMUONClusterFinderPeakFit.h"
+#include "AliMUONClusterStoreV1.h"
+#include "AliMUONClusterStoreV2.h"
+#include "AliMUONConstants.h"
 #include "AliMUONDigitCalibrator.h"
-#include "AliMUONEventRecoCombi.h" 
 #include "AliMUONDigitMaker.h"
-#include "AliMUONTrack.h"
-#include "AliMUONTrackParam.h"
-#include "AliMUONTrackExtrap.h"
-#include "AliMUONVTrackReconstructor.h"
-#include "AliMUONTrackReconstructor.h"
-#include "AliMUONTrackReconstructorK.h"
-#include "AliMUONTriggerTrack.h"
+#include "AliMUONDigitStoreV1.h"
+#include "AliMUONDigitStoreV2R.h"
+#include "AliMUONGeometryTransformer.h"
+#include "AliMUONPreClusterFinder.h"
+#include "AliMUONPreClusterFinderV2.h"
+#include "AliMUONPreClusterFinderV3.h"
+#include "AliMUONRecoParam.h"
+#include "AliMUONSimpleClusterServer.h"
+#include "AliMUONTracker.h"
 #include "AliMUONTriggerCircuit.h"
-#include "AliMUONTriggerCrateStore.h"
+#include "AliMUONTriggerStoreV1.h"
+#include "AliMUONVClusterFinder.h"
+#include "AliMUONVClusterServer.h"
+#include "AliMUONVTrackStore.h"
 
-#include "AliMpSegmentation.h"
+#include "AliMpArea.h"
+#include "AliMpCDB.h"
+#include "AliMpConstants.h"
 
+#include "AliRecoParam.h"
 #include "AliRawReader.h"
-#include "AliRun.h"
-#include "AliRunLoader.h"
-#include "TTask.h"
-#include "TStopwatch.h"
+#include "AliCDBManager.h"
+#include "AliCodeTimer.h"
+#include "AliLog.h"
+
+#include <Riostream.h>
+#include <TObjArray.h>
+#include <TClonesArray.h>
+#include <TString.h>
+#include <TTree.h>
 
 /// \cond CLASSIMP
 ClassImp(AliMUONReconstructor)
-/// \endcond
+/// \endcond 
 
-//_____________________________________________________________________________
-AliMUONReconstructor::AliMUONReconstructor()
-  : AliReconstructor(), 
-    fRunLoader(0x0),
-    fDigitMaker(new AliMUONDigitMaker()), 
-    fCalibrationData(0x0),
-    fCrateManager(new AliMUONTriggerCrateStore()),
-    fTriggerCircuit(new TClonesArray("AliMUONTriggerCircuit", 234)),
-    fTransformer(new AliMUONGeometryTransformer(kTRUE))
+AliMUONRecoParam* AliMUONReconstructor::fgRecoParam = 0x0; // reconstruction parameters
 
+//_____________________________________________________________________________
+AliMUONReconstructor::AliMUONReconstructor() : 
+AliReconstructor(),
+fDigitMaker(0x0),
+fTransformer(new AliMUONGeometryTransformer()),
+fDigitStore(0x0),
+fTriggerCircuit(0x0),
+fCalibrationData(0x0),
+fDigitCalibrator(0x0),
+fClusterServer(0x0),
+fTriggerStore(0x0),
+fTrackStore(0x0),
+fClusterStore(0x0)
 {
-/// Default constructor
-
-    AliDebug(1,"");
-    // Crate manager
-    fCrateManager->ReadFromFile();
-
-    // set to digit maker
-    fDigitMaker->SetCrateManager(fCrateManager);
-
-    // transformater
-    fTransformer->ReadGeometryData("volpath.dat", "geometry.root");
-
-    // trigger circuit
-    for (Int_t i = 0; i < AliMUONConstants::NTriggerCircuit(); i++)  {
-      AliMUONTriggerCircuit* c = new AliMUONTriggerCircuit();
-      c->SetTransformer(fTransformer);
-      c->Init(i,*fCrateManager);
-      TClonesArray& circuit = *fTriggerCircuit;
-      new(circuit[circuit.GetEntriesFast()])AliMUONTriggerCircuit(*c);
-      delete c;
-    }
+  /// normal ctor
 
+  // Load mapping
+  if ( ! AliMpCDB::LoadDDLStore() ) {
+    AliFatal("Could not access mapping from OCDB !");
+  }
+  
+  // Load geometry data
+  fTransformer->LoadGeometryData();
   
 }
 
 //_____________________________________________________________________________
 AliMUONReconstructor::~AliMUONReconstructor()
 {
-/// Destructor
-
-  AliDebug(1,"");
-  delete fCalibrationData;
+  /// dtor
   delete fDigitMaker;
-  delete fCrateManager;
-  delete fTriggerCircuit;
+  delete fDigitStore;
   delete fTransformer;
+  delete fTriggerCircuit;
+  delete fCalibrationData;
+  delete fDigitCalibrator;
+  delete fClusterServer;
+  delete fTriggerStore;
+  delete fTrackStore;
+  delete fClusterStore;
 }
 
 //_____________________________________________________________________________
-TTask* 
-AliMUONReconstructor::GetCalibrationTask(AliMUONData* data) const
+const AliMUONRecoParam* AliMUONReconstructor::GetRecoParam()
 {
-/// Create the calibration task(s). 
+  /// get reconstruction parameters
+  
+  // initialize reconstruction parameters if not already done
+  if (!fgRecoParam) {
+    cout<<"W-AliMUONReconstructor::GetRecoParam: Reconstruction parameters not initialized - Use default one"<<endl;
+    fgRecoParam = AliMUONRecoParam::GetLowFluxParam();
+  }
   
-  const AliRun* run = fRunLoader->GetAliRun();
+  return fgRecoParam;
+}
 
-  AliInfo("Calibration will occur.");
-  Int_t runNumber = run->GetRunNumber();     
-  fCalibrationData = new AliMUONCalibrationData(runNumber);
-  if ( !fCalibrationData->IsValid() )
-    {
-      AliError("Could not retrieve calibrations !");
-      delete fCalibrationData;
-      fCalibrationData = 0x0;
-      return 0x0;
-    }    
-  TTask* calibration = new TTask("MUONCalibrator","MUON Digit calibrator");
-  calibration->Add(new AliMUONDigitCalibrator(data,fCalibrationData));
-  //FIXME: calibration->Add(something about dead channels should go here).
-  return calibration;
+//_____________________________________________________________________________
+void
+AliMUONReconstructor::Calibrate(AliMUONVDigitStore& digitStore) const
+{
+  /// Calibrate the digitStore
+  if (!fDigitCalibrator)
+  {
+    CreateCalibrator();
+  }
+  AliCodeTimerAuto(Form("%s::Calibrate(AliMUONVDigitStore*)",fDigitCalibrator->ClassName()))
+  fDigitCalibrator->Calibrate(digitStore);  
+}
 
+//_____________________________________________________________________________
+void
+AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, 
+                                    AliMUONVDigitStore* digitStore,
+                                    AliMUONVTriggerStore* triggerStore) const
+{
+  /// Convert raw data into digit and trigger stores
+  CreateDigitMaker();
+  
+  AliCodeTimerStart(Form("%s::Raw2Digits(AliRawReader*,AliMUONVDigitStore*,AliMUONVTriggerStore*)",
+                    fDigitMaker->ClassName()))
+  fDigitMaker->Raw2Digits(rawReader,digitStore,triggerStore);
+  AliCodeTimerStop(Form("%s::Raw2Digits(AliRawReader*,AliMUONVDigitStore*,AliMUONVTriggerStore*)",
+                         fDigitMaker->ClassName()))
+  Calibrate(*digitStore);
+}
+
+//_____________________________________________________________________________
+void 
+AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, TTree* digitsTree) const
+{
+   /// convert raw data into a digit tree
+  AliCodeTimerAuto("")
+
+  Bool_t alone = ( TriggerStore() == 0 );
+  
+  Bool_t ok = DigitStore()->Connect(*digitsTree,alone);
+  if ( TriggerStore() ) 
+  {
+    ok = ok && TriggerStore()->Connect(*digitsTree,kFALSE);
+  }
+  
+  if (!ok)
+  {
+    AliError("Could not make branches on TreeD");
+  }
+  else
+  {
+    ConvertDigits(rawReader,DigitStore(),TriggerStore());
+    AliCodeTimerStart("Fill digits")
+    digitsTree->Fill();
+    AliCodeTimerStop("Fill digits")
+    DigitStore()->Clear();
+  }
 }
 
 //_____________________________________________________________________________
 void
-AliMUONReconstructor::Init(AliRunLoader* runLoader)
+AliMUONReconstructor::CreateDigitMaker() const
 {
-/// Initialize
+  /// Create (and create if necessary) the digit maker
+  if (fDigitMaker) return;
+
+  AliCodeTimerAuto("")
 
-  fRunLoader = runLoader;
+  TString option = GetOption();
+  Bool_t enableErrorLogging = kTRUE;
+  Bool_t useFastTrackerDecoder = kTRUE;
+  Bool_t useFastTriggerDecoder = kTRUE;
+  if (option.Contains("NOFASTTRKDECODER"))
+  {
+    useFastTrackerDecoder = kFALSE;
+  }
+  if (option.Contains("NOFASTTRGDECODER"))
+  {
+    useFastTriggerDecoder = kFALSE;
+  }
+  if (option.Contains("NOFASTDECODERS"))
+  {
+    useFastTrackerDecoder = kFALSE;
+    useFastTriggerDecoder = kFALSE;
+  }
+  fDigitMaker = new AliMUONDigitMaker(
+      enableErrorLogging, useFastTrackerDecoder, useFastTriggerDecoder
+    );
+  option.ToUpper();
+  if ( option.Contains("SAVEDIGITS" ))
+    {
+      fDigitMaker->SetMakeTriggerDigits(kTRUE);
+    }
 }
 
 //_____________________________________________________________________________
-void AliMUONReconstructor::Reconstruct(AliRunLoader* runLoader) const
+void 
+AliMUONReconstructor::CreateTriggerCircuit() const
 {
-/// Reconstruct
-/// \todo add more
+  /// Return (and create if necessary) the trigger circuit object
+  if (fTriggerCircuit) return;
+
+  AliCodeTimerAuto("")
+
+  fTriggerCircuit = new AliMUONTriggerCircuit(fTransformer);
 
-  AliLoader* loader = runLoader->GetLoader("MUONLoader");
-  Int_t nEvents = runLoader->GetNumberOfEvents();
+}
 
-  AliMUONData* data = new AliMUONData(loader,"MUON","MUON");
+//_____________________________________________________________________________
+AliTracker* 
+AliMUONReconstructor::CreateTracker() const
+{
+  /// Create the MUONTracker object
+  
+  CreateTriggerCircuit();
+  CreateDigitMaker();
+  CreateClusterServer();
 
-// passing loader as argument.
-  AliMUONVTrackReconstructor* recoEvent;
-  if (strstr(GetOption(),"Original")) recoEvent = new AliMUONTrackReconstructor(data);
-  else if (strstr(GetOption(),"Combi")) recoEvent = new AliMUONTrackReconstructorK(data,"Combi");
-  else recoEvent = new AliMUONTrackReconstructorK(data,"Kalman");
+  AliMUONTracker* tracker(0x0);
   
-  recoEvent->SetTriggerCircuit(fTriggerCircuit);
+  if ( ! AliMUONReconstructor::GetRecoParam()->CombineClusterTrackReco() )
+  {
+    tracker = new AliMUONTracker(0x0,
+                                 *DigitStore(),
+                                 fDigitMaker,
+                                 fTransformer,
+                                 fTriggerCircuit);
+  }
+  else
+  {
+    tracker = new AliMUONTracker(fClusterServer,
+                                 *DigitStore(),
+                                 fDigitMaker,
+                                 fTransformer,
+                                 fTriggerCircuit);
+  }
+  
+  
+  return tracker;
+}
 
-  AliMUONClusterReconstructor* recoCluster = new AliMUONClusterReconstructor(data);
+//_____________________________________________________________________________
+AliMUONVClusterFinder*
+AliMUONReconstructor::CreateClusterFinder(const char* clusterFinderType)
+{
+  /// Create a given cluster finder instance
   
-  AliMUONClusterFinderVS *recModel = recoCluster->GetRecoModel();
+  AliCodeTimerAutoGeneral("")
 
-  if (!strstr(GetOption(),"VS")) {
-    recModel = (AliMUONClusterFinderVS*) new AliMUONClusterFinderAZ();
-    recoCluster->SetRecoModel(recModel);
+  AliMUONVClusterFinder* clusterFinder(0x0);
+  
+  TString opt(clusterFinderType);
+  opt.ToUpper();
+  
+  if ( strstr(opt,"PRECLUSTERV2") )
+  {
+    clusterFinder = new AliMUONPreClusterFinderV2;
+  }    
+  else if ( strstr(opt,"PRECLUSTERV3") )
+  {
+    clusterFinder = new AliMUONPreClusterFinderV3;
+  }  
+  else if ( strstr(opt,"PRECLUSTER") )
+  {
+    clusterFinder = new AliMUONPreClusterFinder;
+  }  
+  else if ( strstr(opt,"PEAKCOG") )
+  {
+    clusterFinder = new AliMUONClusterFinderPeakCOG(kFALSE,new AliMUONPreClusterFinder);
+  }
+  else if ( strstr(opt,"PEAKFIT") )
+  {
+    clusterFinder = new AliMUONClusterFinderPeakFit(kFALSE,new AliMUONPreClusterFinder);
+  }
+  else if ( strstr(opt,"COG") )
+  {
+    clusterFinder = new AliMUONClusterFinderCOG(new AliMUONPreClusterFinder);
+  }  
+  else if ( strstr(opt,"SIMPLEFITV3") )
+  {
+    clusterFinder = new AliMUONClusterFinderSimpleFit(new AliMUONClusterFinderCOG(new AliMUONPreClusterFinderV3));
+  }
+  else if ( strstr(opt,"SIMPLEFIT") )
+  {
+    clusterFinder = new AliMUONClusterFinderSimpleFit(new AliMUONClusterFinderCOG(new AliMUONPreClusterFinder));
   }
-  recModel->SetGhostChi2Cut(10);
+  else if ( strstr(opt,"MLEM:DRAW") )
+  {
+    clusterFinder = new AliMUONClusterFinderMLEM(kTRUE,new AliMUONPreClusterFinder);
+  }
+  else if ( strstr(opt,"MLEMV3") )
+  {
+    clusterFinder = new AliMUONClusterFinderMLEM(kFALSE,new AliMUONPreClusterFinderV3);
+  } 
+  else if ( strstr(opt,"MLEMV2") )
+  {
+    clusterFinder = new AliMUONClusterFinderMLEM(kFALSE,new AliMUONPreClusterFinderV2);
+  } 
+  else if ( strstr(opt,"MLEM") )
+  {
+    clusterFinder = new AliMUONClusterFinderMLEM(kFALSE,new AliMUONPreClusterFinder);
+  } 
+  else
+  {
+    AliErrorClass(Form("clustering mode \"%s\" does not exist",opt.Data()));
+    return 0x0;
+  }
+  
+  return clusterFinder;
+}
 
-  loader->LoadDigits("READ");
-  loader->LoadRecPoints("RECREATE");
-  loader->LoadTracks("RECREATE");
+//_____________________________________________________________________________
+void
+AliMUONReconstructor::CreateClusterServer() const
+{
+  /// Create cluster server
   
-  TTask* calibration = GetCalibrationTask(data);
+  if ( fClusterServer ) return;
+  
+  AliCodeTimerAuto("");
+    
+  AliMUONVClusterFinder* clusterFinder = CreateClusterFinder(GetRecoParam()->GetClusteringMode());
   
-  Int_t chBeg = (strstr(GetOption(),"Combi") ? 6 : 0);
+  if ( !clusterFinder ) return;
+  
+  AliInfo(Form("Will use %s for clusterizing",clusterFinder->ClassName()));
+  
+  fClusterServer = new AliMUONSimpleClusterServer(clusterFinder,*fTransformer);
+}
 
-  //   Loop over events              
-  for(Int_t ievent = 0; ievent < nEvents; ievent++) {
+//_____________________________________________________________________________
+void
+AliMUONReconstructor::CreateCalibrator() const
+{
+  /// Create the calibrator
+  
+  AliCodeTimerAuto("")
+  
+  Int_t runNumber = AliCDBManager::Instance()->GetRun();
 
-    AliDebug(1,Form("Event %d",ievent));
-    
-    runLoader->GetEvent(ievent);
-
-    //----------------------- digit2cluster & Trigger2Trigger -------------------
-    if (!loader->TreeR()) loader->MakeRecPointsContainer();
-     
-    // tracking branch
-    if (!strstr(GetOption(),"Combi")) {
-      data->MakeBranch("RC");
-      data->SetTreeAddress("D,RC");
-    } else {
-      data->SetTreeAddress("D");
-      data->SetTreeAddress("RCC");
-    }
-    // Important for avoiding a memory leak when reading digits ( to be investigated more in detail)
-    // In any case the reading of GLT is needed for the Trigger2Tigger method below
-    data->SetTreeAddress("GLT");
+  AliInfo("Calibration will occur.");
+  
+  fCalibrationData = new AliMUONCalibrationData(runNumber);
+  if ( !fCalibrationData->IsValid() )
+  {
+    AliError("Could not retrieve calibrations !");
+    delete fCalibrationData;
+    fCalibrationData = 0x0;
+    return;
+  }    
+  
+  // Check that we get all the calibrations we'll need
+  if ( !fCalibrationData->Pedestals() ||
+       !fCalibrationData->Gains() ||
+       !fCalibrationData->HV() )
+  {
+    AliFatal("Could not access all required calibration data");
+  }
+  
+  TString opt(GetOption());
+  opt.ToUpper();
+  
+  if ( strstr(opt,"NOSTATUSMAP") )
+  {
+    AliWarning("NOSTATUSMAP is obsolete");
+  }
+
+  TString calibMode = GetRecoParam()->GetCalibrationMode();
+
+  fDigitCalibrator = new AliMUONDigitCalibrator(*fCalibrationData,calibMode.Data());
+}
 
-    data->GetDigits();
+//_____________________________________________________________________________
+AliMUONVDigitStore*
+AliMUONReconstructor::DigitStore() const
+{
+  /// Return (and create if necessary) the digit container
+  if (!fDigitStore) 
+  {
+    TString sopt(GetOption());
+    sopt.ToUpper();
+    
+    AliInfo(Form("Options=%s",sopt.Data()));
     
-    if ( calibration ) 
+    if ( sopt.Contains("DIGITSTOREV1") )
     {
-      calibration->ExecuteTask();
+      fDigitStore = AliMUONVDigitStore::Create("AliMUONDigitStoreV1");
     }
-    
-    recoCluster->Digits2Clusters(chBeg); 
-    
-    if (strstr(GetOption(),"Combi")) {
-      // Combined cluster / track finder
-      AliMUONEventRecoCombi::Instance()->FillEvent(data, (AliMUONClusterFinderAZ*)recModel);
-      ((AliMUONClusterFinderAZ*) recModel)->SetReco(2); 
+    else if ( sopt.Contains("DIGITSTOREV2R") ) 
+    {
+      fDigitStore = AliMUONVDigitStore::Create("AliMUONDigitStoreV2R");
     }
-    else data->Fill("RC"); 
-
-    // trigger branch
-    data->MakeBranch("TC");
-    data->SetTreeAddress("TC");
-    recoCluster->Trigger2Trigger(); 
-    data->Fill("TC");
-
-    //AZ loader->WriteRecPoints("OVERWRITE");
-
-    //---------------------------- Track & TriggerTrack ---------------------
-    if (!loader->TreeT()) loader->MakeTracksContainer();
-
-    // trigger branch
-    data->MakeBranch("RL"); //trigger track
-    data->SetTreeAddress("RL");
-    recoEvent->EventReconstructTrigger();
-    data->Fill("RL");
-
-    // tracking branch
-    data->MakeBranch("RT"); //track
-    data->SetTreeAddress("RT");
-    recoEvent->EventReconstruct();
-    data->Fill("RT");
-
-    loader->WriteTracks("OVERWRITE"); 
-  
-    if (strstr(GetOption(),"Combi")) { 
-      // Combined cluster / track
-      ((AliMUONClusterFinderAZ*) recModel)->SetReco(1);
-      data->MakeBranch("RC");
-      data->SetTreeAddress("RC");
-      AliMUONEventRecoCombi::Instance()->FillRecP(data, (AliMUONTrackReconstructorK*)recoEvent); 
-      data->Fill("RC"); 
+    else if ( sopt.Contains("DIGITSTOREV2S") ) 
+    {
+      fDigitStore = AliMUONVDigitStore::Create("AliMUONDigitStoreV2S");
     }
-    loader->WriteRecPoints("OVERWRITE"); 
-
-    //--------------------------- Resetting branches -----------------------
-    data->ResetDigits();
-    data->ResetRawClusters();
-    data->ResetTrigger();
-    data->ResetRecTracks();  
-    data->ResetRecTriggerTracks();
-
+    
+    if (!fDigitStore) fDigitStore = AliMUONVDigitStore::Create("AliMUONDigitStoreV2R");
+    
+    AliInfo(Form("Will use %s to store digits during reconstruction",fDigitStore->ClassName()));
   }
-  loader->UnloadDigits();
-  loader->UnloadRecPoints();
-  loader->UnloadTracks();
-
-  delete recoCluster;
-  delete recoEvent;
-  delete data;
-  delete calibration;
+  return fDigitStore;
 }
 
 //_____________________________________________________________________________
-void AliMUONReconstructor::Reconstruct(AliRunLoader* runLoader, AliRawReader* rawReader) const
+void
+AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore,
+                                TTree& clustersTree) const
 {
-/// Recontruct
-/// \todo add more
-
-  //  AliLoader
-  AliLoader* loader = runLoader->GetLoader("MUONLoader");
-  AliMUONData data(loader,"MUON","MUON");
-
-  // passing loader as argument.
-  fDigitMaker->SetMUONData(&data);
-
-  AliMUONClusterReconstructor recoCluster(&data);
-
-  AliMUONVTrackReconstructor *recoEvent;
-  if (strstr(GetOption(),"Original")) recoEvent = new AliMUONTrackReconstructor(&data);
-  else if (strstr(GetOption(),"Combi")) recoEvent = new AliMUONTrackReconstructorK(&data,"Combi");
-  else recoEvent = new AliMUONTrackReconstructorK(&data,"Kalman");
+  /// Write the trigger and cluster information into TreeR
   
-  recoEvent->SetTriggerCircuit(fTriggerCircuit);
-
-  AliMUONClusterFinderVS *recModel = recoCluster.GetRecoModel();
-  if (!strstr(GetOption(),"VS")) 
-  {
-    recModel = (AliMUONClusterFinderVS*) new AliMUONClusterFinderAZ();
-    recoCluster.SetRecoModel(recModel);
-  }
-  recModel->SetGhostChi2Cut(10);
+  AliCodeTimerAuto("")
 
-  TTask* calibration = GetCalibrationTask(&data);
-  
-  loader->LoadRecPoints("RECREATE");
-  loader->LoadTracks("RECREATE");
-  loader->LoadDigits("READ");
-  
-  //   Loop over events  
-  Int_t iEvent = 0;
-           
-  TStopwatch totalTimer;
-  TStopwatch rawTimer;
-  TStopwatch calibTimer;
-  TStopwatch clusterTimer;
-  TStopwatch trackingTimer;
+  AliDebug(1,"");
   
-  rawTimer.Start(kTRUE); rawTimer.Stop();
-  calibTimer.Start(kTRUE); calibTimer.Stop();
-  clusterTimer.Start(kTRUE); clusterTimer.Stop();
-  trackingTimer.Start(kTRUE); trackingTimer.Stop();
+  Bool_t ok(kFALSE);
+  Bool_t alone(kTRUE); // is trigger the only info in TreeR ?
   
-  totalTimer.Start(kTRUE);
+  if ( ! AliMUONReconstructor::GetRecoParam()->CombineClusterTrackReco() )
+  {
+    alone = kFALSE; // we'll get both tracker and trigger information in TreeR
+  }
   
-  while (rawReader->NextEvent()
+  if ( triggerStore 
   {
-    AliDebug(1,Form("Event %d",iEvent));
-    
-    runLoader->GetEvent(iEvent++);
+    ok = triggerStore->Connect(clustersTree,alone);
+    if (!ok)
+    {
+      AliError("Could not create triggerStore branches in TreeR");
+    }
+  }
 
-    //----------------------- raw2digits & raw2trigger-------------------
-    if (!loader->TreeD()) 
+  if ( !alone )
+  {
+    if (!fClusterStore)
     {
-      AliDebug(1,Form("Making Digit Container for event %d",iEvent));
-      loader->MakeDigitsContainer();
+      fClusterStore = new AliMUONClusterStoreV2;
     }
     
-    data.SetTreeAddress("D,GLT");
-    rawTimer.Start(kFALSE);
-    fDigitMaker->Raw2Digits(rawReader);
-    rawTimer.Stop();
+    CreateClusterServer();
     
-    if ( calibration )
+    TIter next(DigitStore()->CreateIterator());
+    fClusterServer->UseDigits(next);
+
+    AliMpArea area;
+    
+    AliDebug(1,Form("Doing full clusterization in local reconstruction using %s ",fClusterServer->ClassName()));
+    
+    for ( Int_t i = 0; i < AliMpConstants::NofTrackingChambers(); ++i ) 
     {
-      calibTimer.Start(kFALSE);
-      calibration->ExecuteTask();
-      calibTimer.Stop();
+      if (AliMUONReconstructor::GetRecoParam()->UseChamber(i))
+      {
+        if ( i >= 6 && AliMUONReconstructor::GetRecoParam()->BypassSt45() ) continue;
+        
+        fClusterServer->Clusterize(i,*fClusterStore,area);
+      }
     }
-  
-    //----------------------- digit2cluster & Trigger2Trigger -------------------
-    clusterTimer.Start(kFALSE);
-
-    if (!loader->TreeR()) loader->MakeRecPointsContainer();
-     
-    // tracking branch
-    data.MakeBranch("RC");
-    data.SetTreeAddress("RC");
-    recoCluster.Digits2Clusters(); 
-    data.Fill("RC"); 
-
-    // trigger branch
-    data.MakeBranch("TC");
-    data.SetTreeAddress("TC");
-    recoCluster.Trigger2Trigger();
-    data.Fill("TC");
     
-    loader->WriteRecPoints("OVERWRITE");
+    Bool_t cok = fClusterStore->Connect(clustersTree,alone);
+    
+    if (!cok) AliError("Could not connect clusterStore to clusterTree");
+    
+    AliDebug(1,Form("Number of clusters found = %d",fClusterStore->GetSize()));
+    
+    StdoutToAliDebug(1,fClusterStore->Print());
+  }
+         
+  if (ok) // at least one type of branches created successfully
+  {
+    clustersTree.Fill();
+  }
+  
+  if (fClusterStore) fClusterStore->Clear();
+}
 
-    clusterTimer.Stop();
+//_____________________________________________________________________________
+Bool_t 
+AliMUONReconstructor::HasDigitConversion() const
+{
+  /// We *do* have digit conversion, but we might advertise it only 
+  /// if we want to save the digits.
+  
+  TString opt(GetOption());
+  opt.ToUpper();
+  if ( opt.Contains("SAVEDIGITS" ) && !opt.Contains("NOLOCALRECONSTRUCTION") )
+  {
+    return kTRUE;
+  }
+  else
+  {
+    return kFALSE;
+  }
+}
 
-    //---------------------------- Track & TriggerTrack ---------------------
-    trackingTimer.Start(kFALSE);
-    if (!loader->TreeT()) loader->MakeTracksContainer();
+//_____________________________________________________________________________
+void 
+AliMUONReconstructor::Reconstruct(AliRawReader* rawReader, TTree* clustersTree) const
+{
+  /// This method is called by AliReconstruction if HasLocalReconstruction()==kTRUE AND
+  /// HasDigitConversion()==kFALSE
+  
+  if ( !clustersTree ) 
+  {
+    AliError("clustersTree is 0x0 !");
+    return;
+  }
+  
+  ConvertDigits(rawReader,DigitStore(),TriggerStore());
 
-    // trigger branch
-    data.MakeBranch("RL"); //trigger track
-    data.SetTreeAddress("RL");
-    recoEvent->EventReconstructTrigger();
-    data.Fill("RL");
+  FillTreeR(TriggerStore(),*clustersTree);
+}
 
-    // tracking branch
-    data.MakeBranch("RT"); //track
-    data.SetTreeAddress("RT");
-    recoEvent->EventReconstruct();
-    data.Fill("RT");
+//_____________________________________________________________________________
+void 
+AliMUONReconstructor::Reconstruct(TTree* digitsTree, TTree* clustersTree) const
+{
+  /// This method is called by AliReconstruction if HasLocalReconstruction()==kTRUE
+  /// AND HasDigitConversion()==kTRUE
+  
+  AliCodeTimerAuto("")
+  
+  AliDebug(1,"");
+  
+  if (!digitsTree || !clustersTree) 
+  {
+    AliError(Form("Tree is null : digitsTree=%p clustersTree=%p",
+                  digitsTree,clustersTree));
+    return;
+  }
 
-    loader->WriteTracks("OVERWRITE");  
-    trackingTimer.Stop();
-    
-    //--------------------------- Resetting branches -----------------------
-    data.ResetDigits();
-    data.ResetRawClusters();
-    data.ResetTrigger();
-    data.ResetRecTracks();
-    data.ResetRecTriggerTracks();
+  if (!fDigitStore)
+  {
+    fDigitStore = AliMUONVDigitStore::Create(*digitsTree);
+    if (!fDigitStore)
+    {
+      AliError(Form("Could not get DigitStore from %s",digitsTree->GetName()));
+    }
+    else
+    {
+      AliInfo(Form("Created %s from %s",fDigitStore->ClassName(),digitsTree->GetName()));
+    }
+  }
+  if (!fTriggerStore)
+  {
+    fTriggerStore = AliMUONVTriggerStore::Create(*digitsTree);
+    if (!fTriggerStore)
+    {
+      AliError(Form("Could not get TriggerStore from %s",digitsTree->GetName()));
+    }
+    else
+    {
+      AliInfo(Form("Created %s from %s",fTriggerStore->ClassName(),digitsTree->GetName()));
+    }
+  }
   
+  if (!fTriggerStore && !fDigitStore)
+  {
+    AliError("No store at all. Nothing to do.");
+    return;
   }
   
-  totalTimer.Stop();
+  // insure we start with empty stores
+  if ( fDigitStore ) 
+  {
+    fDigitStore->Clear(); 
+    Bool_t alone = ( fTriggerStore ? kFALSE : kTRUE );
+    Bool_t ok = fDigitStore->Connect(*digitsTree,alone);
+    if (!ok)
+    {
+      AliError("Could not connect digitStore to digitsTree");
+      return;
+    }
+  }
+  if ( fTriggerStore ) 
+  {
+    fTriggerStore->Clear();
+    Bool_t alone = ( fDigitStore ? kFALSE : kTRUE );
+    Bool_t ok = fTriggerStore->Connect(*digitsTree,alone);
+    if (!ok)
+    {
+      AliError("Could not connect triggerStore to digitsTree");
+      return;
+    }
+  }
   
-  loader->UnloadRecPoints();
-  loader->UnloadTracks();
-  loader->UnloadDigits();
+  digitsTree->GetEvent(0);
   
-  delete recoEvent;
-
-  AliInfo(Form("Execution time for converting RAW data to digits in MUON : R:%.2fs C:%.2fs",
-               rawTimer.RealTime(),rawTimer.CpuTime()));
-  AliInfo(Form("Execution time for calibrating MUON : R:%.2fs C:%.2fs",
-               calibTimer.RealTime(),calibTimer.CpuTime()));
-  AliInfo(Form("Execution time for clusterizing MUON : R:%.2fs C:%.2fs",
-               clusterTimer.RealTime(),clusterTimer.CpuTime()));
-  AliInfo(Form("Execution time for tracking MUON : R:%.2fs C:%.2fs",
-               trackingTimer.RealTime(),trackingTimer.CpuTime()));
-  AliInfo(Form("Total Execution time for Reconstruct(from raw) MUON : R:%.2fs C:%.2fs",
-               totalTimer.RealTime(),totalTimer.CpuTime()));
+  if ( fDigitStore ) 
+  {
+    // Insure we got calibrated digits (if we reconstruct from pure simulated,
+    // i.e. w/o going through raw data, this will be the case)
+    TIter next(fDigitStore->CreateIterator());
+    AliMUONVDigit* digit = static_cast<AliMUONVDigit*>(next());
+    if (digit && !digit->IsCalibrated())
+    {
+      Calibrate(*fDigitStore);
+    }
+  }
+    
+  FillTreeR(fTriggerStore,*clustersTree);
 }
 
 //_____________________________________________________________________________
-void AliMUONReconstructor::FillESD(AliRunLoader* runLoader, AliESD* esd) const
+AliMUONVTriggerStore*
+AliMUONReconstructor::TriggerStore() const
 {
-/// Fill ESD
-/// \todo add more
-
-  TClonesArray* recTracksArray = 0;
-  TClonesArray* recTrigTracksArray = 0;
-  
-  AliLoader* loader = runLoader->GetLoader("MUONLoader");
-  loader->LoadTracks("READ");
-  AliMUONData* muonData = new AliMUONData(loader,"MUON","MUON");
-
-   // declaration  
-  Int_t iEvent;// nPart;
-  Int_t nTrackHits;// nPrimary;
-  Double_t fitFmin;
-
-  Double_t bendingSlope, nonBendingSlope, inverseBendingMomentum;
-  Double_t xRec, yRec, zRec, chi2MatchTrigger;
-  Bool_t matchTrigger;
-
-  // setting pointer for tracks, triggertracks & trackparam at vertex
-  AliMUONTrack* recTrack = 0;
-  AliMUONTrackParam* trackParam = 0;
-  AliMUONTriggerTrack* recTriggerTrack = 0;
-
-  iEvent = runLoader->GetEventNumber(); 
-  runLoader->GetEvent(iEvent);
-
-  // Get vertex 
-  Double_t vertex[3] = {0};
-  const AliESDVertex *esdVert = esd->GetVertex(); 
-  if (esdVert->GetNContributors()) {
-    esdVert->GetXYZ(vertex);
-    printf("find vertex\n");
-  }
-  // setting ESD MUON class
-  AliESDMuonTrack* theESDTrack = new  AliESDMuonTrack() ;
-
-  //-------------------- trigger tracks-------------
-  Long_t trigPat = 0;
-  muonData->SetTreeAddress("RL");
-  muonData->GetRecTriggerTracks();
-  recTrigTracksArray = muonData->RecTriggerTracks();
-
-  // ready global trigger pattern from first track
-  if (recTrigTracksArray) 
-    recTriggerTrack = (AliMUONTriggerTrack*) recTrigTracksArray->First();
-  if (recTriggerTrack) trigPat = recTriggerTrack->GetGTPattern();
-
-  //printf(">>> Event %d Number of Recconstructed tracks %d \n",iEvent, nrectracks);
-  // -------------------- tracks-------------
-  muonData->SetTreeAddress("RT");
-  muonData->GetRecTracks();
-  recTracksArray = muonData->RecTracks();
-        
-  Int_t nRecTracks = 0;
-  if (recTracksArray)
-    nRecTracks = (Int_t) recTracksArray->GetEntriesFast(); //
-  
-  // loop over tracks
-  for (Int_t iRecTracks = 0; iRecTracks <  nRecTracks;  iRecTracks++) {
-
-    // reading info from tracks
-    recTrack = (AliMUONTrack*) recTracksArray->At(iRecTracks);
-
-    trackParam = (AliMUONTrackParam*) (recTrack->GetTrackParamAtHit())->First();
-   
-    if (esdVert->GetNContributors())
-      AliMUONTrackExtrap::ExtrapToVertex(trackParam, vertex[0],vertex[1],vertex[2]);
-
-    bendingSlope            = trackParam->GetBendingSlope();
-    nonBendingSlope         = trackParam->GetNonBendingSlope();
-    inverseBendingMomentum  = trackParam->GetInverseBendingMomentum();
-    xRec  = trackParam->GetNonBendingCoor();
-    yRec  = trackParam->GetBendingCoor();
-    zRec  = trackParam->GetZ();
-
-    nTrackHits       = recTrack->GetNTrackHits();
-    fitFmin          = recTrack->GetFitFMin();
-    matchTrigger     = recTrack->GetMatchTrigger();
-    chi2MatchTrigger = recTrack->GetChi2MatchTrigger();
-
-    // setting data member of ESD MUON
-    theESDTrack->SetInverseBendingMomentum(inverseBendingMomentum);
-    theESDTrack->SetThetaX(TMath::ATan(nonBendingSlope));
-    theESDTrack->SetThetaY(TMath::ATan(bendingSlope));
-    theESDTrack->SetZ(zRec);
-    theESDTrack->SetBendingCoor(yRec); // calculate vertex at ESD or Tracking level ?
-    theESDTrack->SetNonBendingCoor(xRec);
-    theESDTrack->SetChi2(fitFmin);
-    theESDTrack->SetNHit(nTrackHits);
-    theESDTrack->SetMatchTrigger(matchTrigger);
-    theESDTrack->SetChi2MatchTrigger(chi2MatchTrigger);
-
-    // storing ESD MUON Track into ESD Event 
-    if (nRecTracks != 0)  
-      esd->AddMuonTrack(theESDTrack);
-  } // end loop tracks
-
-  // reset muondata
-  muonData->ResetRecTracks();
-  muonData->ResetRecTriggerTracks();
-
-  //} // end loop on event  
-  loader->UnloadTracks(); 
-
-  delete theESDTrack;
-  delete muonData;
-}//_____________________________________________________________________________
-void AliMUONReconstructor::FillESD(AliRunLoader* runLoader, AliRawReader* /*rawReader*/, AliESD* esd) const
-{
-/// Fill ESD
-/// \todo add more
-
-  // don't need rawReader ???
-  FillESD(runLoader, esd);
+  /// Return (and create if necessary and allowed) the trigger container
+  TString sopt(GetOption());
+  sopt.ToUpper();
+  
+  if (sopt.Contains("TRIGGERDISABLE"))
+  {
+    delete fTriggerStore;
+    fTriggerStore = 0x0;
+  }
+  else
+  {
+    if (!fTriggerStore)
+    {
+      fTriggerStore = new AliMUONTriggerStoreV1;
+    }
+  }
+  return fTriggerStore;
 }