X-Git-Url: http://git.uio.no/git/?a=blobdiff_plain;ds=sidebyside;f=MUON%2FAliMUONReconstructor.cxx;h=561c78ad9d767f83dfd4c47ed55ccc4f1b5dc035;hb=dfbacac8a06b1c2f1aa95db0aa29615065ede580;hp=935c2f9e6fe0051a690672c9bae3623fbea38045;hpb=a0dc65b45f53d95747053ece26219759360f5eb6;p=u%2Fmrichter%2FAliRoot.git diff --git a/MUON/AliMUONReconstructor.cxx b/MUON/AliMUONReconstructor.cxx index 935c2f9e6fe..561c78ad9d7 100644 --- a/MUON/AliMUONReconstructor.cxx +++ b/MUON/AliMUONReconstructor.cxx @@ -19,10 +19,9 @@ /// /// 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); +/// The clustering mode and the associated parameters can be changed through the +/// AliMUONRecoParam object set in the reconstruction macro or read from the CDB +/// (see methods in AliMUONRecoParam.h file for details) /// /// Valid modes are : /// @@ -69,14 +68,7 @@ /// /// 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. +/// ENABLEERRORLOGGING : enable error logging (this activates also warnings in RawStreamTracker) /// /// \author Laurent Aphecetche, Subatech //----------------------------------------------------------------------------- @@ -97,6 +89,7 @@ #include "AliMUONDigitStoreV1.h" #include "AliMUONDigitStoreV2R.h" #include "AliMUONGeometryTransformer.h" +#include "AliMUONPadStatusMaker.h" #include "AliMUONPreClusterFinder.h" #include "AliMUONPreClusterFinderV2.h" #include "AliMUONPreClusterFinderV3.h" @@ -107,15 +100,20 @@ #include "AliMUONVClusterFinder.h" #include "AliMUONVClusterServer.h" #include "AliMUONVTrackStore.h" +#include "AliMUONTriggerElectronics.h" +#include "AliMUONTriggerUtilities.h" #include "AliMpArea.h" #include "AliMpCDB.h" #include "AliMpConstants.h" +#include "AliMpDDLStore.h" +#include "AliMpSegmentation.h" #include "AliRawReader.h" #include "AliCDBManager.h" #include "AliCodeTimer.h" #include "AliLog.h" +#include "AliRunInfo.h" #include #include @@ -136,13 +134,31 @@ fDigitStore(0x0), fTriggerCircuit(0x0), fCalibrationData(0x0), fDigitCalibrator(0x0), -fClusterServer(0x0), fTriggerStore(0x0), fTrackStore(0x0), -fClusterStore(0x0) +fClusterStore(0x0), +fTriggerProcessor(0x0), +fTriggerUtilities(0x0), +fClusterServers(), +fTrackers() { /// normal ctor + AliDebug(1,""); + + // Unload mapping objects + // if they have been loaded from the obsolete OCDB mapping objects + + if ( AliMpDDLStore::Instance(false) ) { + AliCDBManager::Instance()->UnloadFromCache("MUON/Calib/DDLStore"); + delete AliMpDDLStore::Instance(); + } + + if ( AliMpSegmentation::Instance(false) ) { + AliCDBManager::Instance()->UnloadFromCache("MUON/Calib/Mapping"); + delete AliMpSegmentation::Instance(); + } + // Load mapping if ( ! AliMpCDB::LoadDDLStore() ) { AliFatal("Could not access mapping from OCDB !"); @@ -150,23 +166,34 @@ fClusterStore(0x0) // Load geometry data fTransformer->LoadGeometryData(); - + + fClusterServers.SetOwner(kTRUE); + fTrackers.SetOwner(kTRUE); } //_____________________________________________________________________________ AliMUONReconstructor::~AliMUONReconstructor() { /// dtor + + AliDebug(1,""); + + delete fDigitCalibrator; + delete fDigitMaker; delete fDigitStore; delete fTransformer; delete fTriggerCircuit; - delete fCalibrationData; - delete fDigitCalibrator; - delete fClusterServer; delete fTriggerStore; delete fTrackStore; delete fClusterStore; + delete fTriggerProcessor; + delete fTriggerUtilities; + + delete AliMpSegmentation::Instance(false); + delete AliMpDDLStore::Instance(false); + + delete fCalibrationData; } //_____________________________________________________________________________ @@ -178,8 +205,8 @@ AliMUONReconstructor::Calibrate(AliMUONVDigitStore& digitStore) const { CreateCalibrator(); } - AliCodeTimerAuto(Form("%s::Calibrate(AliMUONVDigitStore*)",fDigitCalibrator->ClassName())) - fDigitCalibrator->Calibrate(digitStore); + AliCodeTimerAuto(Form("%s::Calibrate(AliMUONVDigitStore*)",fDigitCalibrator->ClassName()),0) + fDigitCalibrator->Calibrate(digitStore); } //_____________________________________________________________________________ @@ -190,6 +217,14 @@ AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, { /// Convert raw data into digit and trigger stores CreateDigitMaker(); + + // Skip reconstruction if event is Calibration + if ( GetRecoParam()->GetEventSpecie() == AliRecoParam::kCalib ) { + digitStore->Clear(); // Remove possible digits from previous event + triggerStore->Clear(); // Remove possible triggers from previous event + AliInfo("Calibration event: do not convert digits"); + return; + } AliCodeTimerStart(Form("%s::Raw2Digits(AliRawReader*,AliMUONVDigitStore*,AliMUONVTriggerStore*)", fDigitMaker->ClassName())) @@ -204,7 +239,7 @@ void AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, TTree* digitsTree) const { /// convert raw data into a digit tree - AliCodeTimerAuto("") + AliCodeTimerAuto("",0) Bool_t alone = ( TriggerStore() == 0 ); @@ -235,33 +270,32 @@ AliMUONReconstructor::CreateDigitMaker() const /// Create (and create if necessary) the digit maker if (fDigitMaker) return; - AliCodeTimerAuto("") + AliCodeTimerAuto("",0) TString option = GetOption(); - Bool_t enableErrorLogging = kTRUE; - Bool_t useFastTrackerDecoder = kTRUE; - Bool_t useFastTriggerDecoder = kTRUE; - if (option.Contains("NOFASTTRKDECODER")) + + Bool_t enableErrorLogging = kFALSE; + + if (option.Contains("ENABLEERRORLOGGING")) { - useFastTrackerDecoder = kFALSE; + enableErrorLogging = kTRUE; } - if (option.Contains("NOFASTTRGDECODER")) + + fDigitMaker = new AliMUONDigitMaker(enableErrorLogging); + option.ToUpper(); + + // Always make trigger digits + // (needed when calculating trigger chamber efficiency) + fDigitMaker->SetMakeTriggerDigits(kTRUE); + + if ( GetRecoParam()->TryRecover() ) { - useFastTriggerDecoder = kFALSE; + fDigitMaker->SetTryRecover(kTRUE); } - if (option.Contains("NOFASTDECODERS")) + else { - useFastTrackerDecoder = kFALSE; - useFastTriggerDecoder = kFALSE; + fDigitMaker->SetTryRecover(kFALSE); } - fDigitMaker = new AliMUONDigitMaker( - enableErrorLogging, useFastTrackerDecoder, useFastTriggerDecoder - ); - option.ToUpper(); - if ( option.Contains("SAVEDIGITS" )) - { - fDigitMaker->SetMakeTriggerDigits(kTRUE); - } } //_____________________________________________________________________________ @@ -271,12 +305,26 @@ AliMUONReconstructor::CreateTriggerCircuit() const /// Return (and create if necessary) the trigger circuit object if (fTriggerCircuit) return; - AliCodeTimerAuto("") + AliCodeTimerAuto("",0) fTriggerCircuit = new AliMUONTriggerCircuit(fTransformer); } +//_____________________________________________________________________________ +void +AliMUONReconstructor::CreateTriggerUtilities() const +{ + /// Return (and create if necessary) the trigger utilities object + if ( fTriggerUtilities ) return; + + AliCodeTimerAuto("",0) + + if ( ! fCalibrationData ) CreateCalibrationData(); + + fTriggerUtilities = new AliMUONTriggerUtilities(fCalibrationData); +} + //_____________________________________________________________________________ AliTracker* AliMUONReconstructor::CreateTracker() const @@ -284,28 +332,45 @@ AliMUONReconstructor::CreateTracker() const /// Create the MUONTracker object CreateTriggerCircuit(); - CreateDigitMaker(); - CreateClusterServer(); - - AliMUONTracker* tracker(0x0); + CreateTriggerUtilities(); - if ( ! GetRecoParam()->CombineClusterTrackReco() ) - { - tracker = new AliMUONTracker(GetRecoParam(), - 0x0, - *DigitStore(), - fDigitMaker, - fTransformer, - fTriggerCircuit); - } - else + const AliMUONRecoParam* rp = GetRecoParam(); + + Int_t es = rp->GetEventSpecie(); + + AliTracker* tracker = static_cast(fTrackers.At(es)); + + if (!tracker ) { - tracker = new AliMUONTracker(GetRecoParam(), - fClusterServer, - *DigitStore(), - fDigitMaker, - fTransformer, - fTriggerCircuit); + if ( ! rp->CombineClusterTrackReco() ) + { + tracker = new AliMUONTracker(rp, + 0x0, + *DigitStore(), + fTransformer, + fTriggerCircuit, + fTriggerUtilities); + + AliInfo(Form("Created tracker %p for recoparam of type %s es=%d", + tracker, + AliRecoParam::GetEventSpecieName(AliRecoParam::Convert(rp->GetEventSpecie())),es)); + } + else + { + + tracker = new AliMUONTracker(rp, + CreateClusterServer(*rp), + *DigitStore(), + fTransformer, + fTriggerCircuit, + fTriggerUtilities); + + AliInfo(Form("Created (combined) tracker %p for recoparam of type %s es=%d", + tracker, + AliRecoParam::GetEventSpecieName(AliRecoParam::Convert(rp->GetEventSpecie())),es)); + } + + fTrackers.AddAtAndExpand(tracker,es); } @@ -318,7 +383,7 @@ AliMUONReconstructor::CreateClusterFinder(const char* clusterFinderType) { /// Create a given cluster finder instance - AliCodeTimerAutoGeneral("") + AliCodeTimerAutoGeneral("",0) AliMUONVClusterFinder* clusterFinder(0x0); @@ -383,36 +448,42 @@ AliMUONReconstructor::CreateClusterFinder(const char* clusterFinderType) } //_____________________________________________________________________________ -void -AliMUONReconstructor::CreateClusterServer() const +AliMUONVClusterServer* +AliMUONReconstructor::CreateClusterServer(const AliMUONRecoParam& rp) const { /// Create cluster server - if ( fClusterServer ) return; - - AliCodeTimerAuto(""); - - AliMUONVClusterFinder* clusterFinder = CreateClusterFinder(GetRecoParam()->GetClusteringMode()); + AliCodeTimerAuto("",0); - if ( !clusterFinder ) return; + AliMUONVClusterServer* clusterServer = static_cast(fClusterServers.At(rp.GetEventSpecie())); - AliInfo(Form("Will use %s for clusterizing",clusterFinder->ClassName())); + if (!clusterServer ) + { + AliMUONVClusterFinder* clusterFinder = CreateClusterFinder(rp.GetClusteringMode()); + + if ( !clusterFinder ) return 0x0; + + clusterServer = new AliMUONSimpleClusterServer(clusterFinder,*fTransformer); + + AliInfo(Form("Created AliMUONSimpleClusterServer (%p) for specie %d with clustering = %s (following requesting clustering mode %s)", + clusterServer,rp.GetEventSpecie(),clusterFinder->ClassName(),rp.GetClusteringMode())); + + fClusterServers.AddAtAndExpand(clusterServer,rp.GetEventSpecie()); + } - fClusterServer = new AliMUONSimpleClusterServer(clusterFinder,*fTransformer); + return clusterServer; } //_____________________________________________________________________________ void -AliMUONReconstructor::CreateCalibrator() const +AliMUONReconstructor::CreateCalibrationData() const { /// Create the calibrator - AliCodeTimerAuto("") + AliCodeTimerAuto("",0); Int_t runNumber = AliCDBManager::Instance()->GetRun(); - AliInfo("Calibration will occur."); - fCalibrationData = new AliMUONCalibrationData(runNumber); if ( !fCalibrationData->IsValid() ) { @@ -422,14 +493,67 @@ AliMUONReconstructor::CreateCalibrator() const return; } - // Check that we get all the calibrations we'll need - if ( !fCalibrationData->Pedestals() || - !fCalibrationData->Gains() || - !fCalibrationData->HV() ) + // It is now time to check whether we have everything to proceed. + // What we need depends on whether both tracker and trigger + // are in the readout chain, and what specific "bad channel policy" + // we use + + Bool_t kTracker(kFALSE); + Bool_t kTrigger(kFALSE); + + const AliRunInfo* runInfo = GetRunInfo(); + if (!runInfo) { - AliFatal("Could not access all required calibration data"); + AliError("Could not get runinfo ?"); } + else + { + TString detectors(runInfo->GetActiveDetectors()); + if (detectors.Contains("MUONTRK")) kTracker=kTRUE; + if (detectors.Contains("MUONTRG")) kTrigger=kTRUE; + } + + AliInfo(Form("Run with MUON TRIGGER : %s and MUON TRACKER : %s", + kTrigger ? "YES":"NO" , + kTracker ? "YES":"NO")); + if ( kTracker ) + { + // Check that we get all the calibrations we'll need + if ( !fCalibrationData->Pedestals() || + !fCalibrationData->Gains() ) + { + AliFatal(Form("Could not access all required calibration data (PED %p GAIN %p)", + fCalibrationData->Pedestals(),fCalibrationData->Gains())); + } + + if ( !fCalibrationData->HV() ) + { + // Special treatment of HV. We only break if the values + // are not there *AND* we cut on them. + UInt_t mask = GetRecoParam()->PadGoodnessMask(); + TString smask(AliMUONPadStatusMaker::AsCondition(mask)); + if ( smask.Contains("HV") ) + { + AliFatal("Could not access all required calibration data (HV)"); + } + } + } +} + +//_____________________________________________________________________________ +void +AliMUONReconstructor::CreateCalibrator() const +{ + /// Create the calibrator + + AliCodeTimerAuto("",0); + + if ( ! fCalibrationData ) + CreateCalibrationData(); + + AliInfo("Calibration will occur."); + TString opt(GetOption()); opt.ToUpper(); @@ -438,9 +562,24 @@ AliMUONReconstructor::CreateCalibrator() const AliWarning("NOSTATUSMAP is obsolete"); } - TString calibMode = GetRecoParam()->GetCalibrationMode(); + fDigitCalibrator = new AliMUONDigitCalibrator(*fCalibrationData,GetRecoParam()); +} + +//_____________________________________________________________________________ +void +AliMUONReconstructor::ResponseRemovingChambers(AliMUONVTriggerStore* triggerStore) const +{ + /// Update trigger information with informatins obtained after + /// re-calculation of trigger response + AliCodeTimerAuto("",0); + + if ( ! fCalibrationData ) + CreateCalibrationData(); + + if ( ! fTriggerProcessor ) + fTriggerProcessor = new AliMUONTriggerElectronics(fCalibrationData); - fDigitCalibrator = new AliMUONDigitCalibrator(*fCalibrationData,GetRecoParam(),calibMode.Data()); + fTriggerProcessor->ResponseRemovingChambers(*triggerStore); } //_____________________________________________________________________________ @@ -481,21 +620,22 @@ AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore, TTree& clustersTree) const { /// Write the trigger and cluster information into TreeR - - AliCodeTimerAuto("") - AliDebug(1,""); - + AliCodeTimerAuto("",0) + Bool_t ok(kFALSE); Bool_t alone(kTRUE); // is trigger the only info in TreeR ? - if ( ! GetRecoParam()->CombineClusterTrackReco() ) + const AliMUONRecoParam* rp = GetRecoParam(); + + if ( ! rp->CombineClusterTrackReco() ) { alone = kFALSE; // we'll get both tracker and trigger information in TreeR } if ( triggerStore ) { + ResponseRemovingChambers(triggerStore); ok = triggerStore->Connect(clustersTree,alone); if (!ok) { @@ -510,23 +650,23 @@ AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore, fClusterStore = new AliMUONClusterStoreV2; } - CreateClusterServer(); + AliMUONVClusterServer* clusterServer = CreateClusterServer(*rp); TIter next(DigitStore()->CreateIterator()); - fClusterServer->UseDigits(next); + clusterServer->UseDigits(next,DigitStore()); AliMpArea area; - AliDebug(1,Form("Doing full clusterization in local reconstruction using %s ",fClusterServer->ClassName())); + AliDebug(1,Form("Doing full clusterization in local reconstruction using %s ",clusterServer->ClassName())); for ( Int_t i = 0; i < AliMpConstants::NofTrackingChambers(); ++i ) { - if (GetRecoParam()->UseChamber(i)) + if (rp->UseChamber(i)) { - if ( ( i == 6 || i == 7 ) && GetRecoParam()->BypassSt4() ) continue; - if ( ( i == 8 || i == 9 ) && GetRecoParam()->BypassSt5() ) continue; + if ( ( i == 6 || i == 7 ) && rp->BypassSt4() ) continue; + if ( ( i == 8 || i == 9 ) && rp->BypassSt5() ) continue; - fClusterServer->Clusterize(i,*fClusterStore,area); + clusterServer->Clusterize(i,*fClusterStore,area,rp); } } @@ -547,6 +687,14 @@ AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore, if (fClusterStore) fClusterStore->Clear(); } +//_____________________________________________________________________________ +const AliMUONRecoParam* AliMUONReconstructor::GetRecoParam() +{ + /// Get the recoparam from reconstruction + return dynamic_cast(AliReconstructor::GetRecoParam(AliReconstruction::GetDetIndex("MUON"))); +} + + //_____________________________________________________________________________ Bool_t AliMUONReconstructor::HasDigitConversion() const @@ -591,7 +739,7 @@ AliMUONReconstructor::Reconstruct(TTree* digitsTree, TTree* clustersTree) const /// This method is called by AliReconstruction if HasLocalReconstruction()==kTRUE /// AND HasDigitConversion()==kTRUE - AliCodeTimerAuto("") + AliCodeTimerAuto("",0) AliDebug(1,"");