X-Git-Url: http://git.uio.no/git/?a=blobdiff_plain;f=MUON%2FAliMUONReconstructor.cxx;h=beaf1f5fa9f68ba23af4489748e9e93a276dbc69;hb=95153273156d2401573870c81190c9bff2bdeeec;hp=5ad4de38e18ab820a81ab1e7bc3c86113906e5dd;hpb=5f3519d9041eaa23e2546ee444e4251e773e18ab;p=u%2Fmrichter%2FAliRoot.git diff --git a/MUON/AliMUONReconstructor.cxx b/MUON/AliMUONReconstructor.cxx index 5ad4de38e18..beaf1f5fa9f 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,28 +89,31 @@ #include "AliMUONDigitStoreV1.h" #include "AliMUONDigitStoreV2R.h" #include "AliMUONGeometryTransformer.h" +#include "AliMUONPadStatusMaker.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 "AliMUONTriggerElectronics.h" +#include "AliMUONTriggerUtilities.h" #include "AliMpArea.h" #include "AliMpCDB.h" #include "AliMpConstants.h" +#include "AliMpDDLStore.h" +#include "AliMpSegmentation.h" -#include "AliRecoParam.h" #include "AliRawReader.h" #include "AliCDBManager.h" #include "AliCodeTimer.h" #include "AliLog.h" +#include "AliRunInfo.h" #include #include @@ -130,24 +125,41 @@ ClassImp(AliMUONReconstructor) /// \endcond -AliMUONRecoParam* AliMUONReconstructor::fgRecoParam = 0x0; // reconstruction parameters - //_____________________________________________________________________________ AliMUONReconstructor::AliMUONReconstructor() : AliReconstructor(), -fCrateManager(0x0), fDigitMaker(0x0), fTransformer(new AliMUONGeometryTransformer()), fDigitStore(0x0), fTriggerCircuit(0x0), fCalibrationData(0x0), fDigitCalibrator(0x0), -fClusterServer(0x0), fTriggerStore(0x0), -fTrackStore(0x0) +fTrackStore(0x0), +fClusterStore(0x0), +fTriggerProcessor(0x0), +fTriggerUtilities(0x0), +fClusterServers(), +fTrackers(), +fShouldCalibrate(kTRUE) { /// 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 !"); @@ -155,51 +167,34 @@ fTrackStore(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 fCrateManager; delete fTriggerCircuit; - delete fCalibrationData; - delete fDigitCalibrator; - delete fClusterServer; delete fTriggerStore; delete fTrackStore; -} + delete fClusterStore; + delete fTriggerProcessor; + delete fTriggerUtilities; -//_____________________________________________________________________________ -const AliMUONRecoParam* AliMUONReconstructor::GetRecoParam() -{ - /// get reconstruction parameters - - if (!fgRecoParam) { - - // get reconstruction parameters from AliRecoParam if any - TObjArray *recoParams = AliRecoParam::Instance()->GetRecoParam("MUON"); - - if (recoParams) { - - fgRecoParam = (AliMUONRecoParam*) recoParams->Last(); - - } else { - - // initialize reconstruction parameters if not already done - cout<<"W-AliMUONReconstructor::GetRecoParam: Reconstruction parameters not initialized - Use default one"<RegisterRecoParam(fgRecoParam); - - } - - } - - return fgRecoParam; + delete AliMpSegmentation::Instance(false); + delete AliMpDDLStore::Instance(false); + + delete fCalibrationData; } //_____________________________________________________________________________ @@ -211,8 +206,12 @@ AliMUONReconstructor::Calibrate(AliMUONVDigitStore& digitStore) const { CreateCalibrator(); } - AliCodeTimerAuto(Form("%s::Calibrate(AliMUONVDigitStore*)",fDigitCalibrator->ClassName())) - fDigitCalibrator->Calibrate(digitStore); + + if ( fShouldCalibrate ) + { + AliCodeTimerAuto(Form("%s::Calibrate(AliMUONVDigitStore*)",fDigitCalibrator->ClassName()),0) + fDigitCalibrator->Calibrate(digitStore); + } } //_____________________________________________________________________________ @@ -223,6 +222,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())) @@ -237,7 +244,7 @@ void AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, TTree* digitsTree) const { /// convert raw data into a digit tree - AliCodeTimerAuto("") + AliCodeTimerAuto("",0) Bool_t alone = ( TriggerStore() == 0 ); @@ -261,17 +268,6 @@ AliMUONReconstructor::ConvertDigits(AliRawReader* rawReader, TTree* digitsTree) } } -//_____________________________________________________________________________ -AliMUONTriggerCrateStore* -AliMUONReconstructor::CrateManager() const -{ - /// Return (and create if necessary) the trigger crate store - if (fCrateManager) return fCrateManager; - fCrateManager = new AliMUONTriggerCrateStore; - fCrateManager->ReadFromFile(); - return fCrateManager; -} - //_____________________________________________________________________________ void AliMUONReconstructor::CreateDigitMaker() const @@ -279,33 +275,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); - } } //_____________________________________________________________________________ @@ -315,12 +310,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 @@ -328,26 +337,45 @@ AliMUONReconstructor::CreateTracker() const /// Create the MUONTracker object CreateTriggerCircuit(); - CreateDigitMaker(); - CreateClusterServer(); - - AliMUONTracker* tracker(0x0); + CreateTriggerUtilities(); - if ( ! AliMUONReconstructor::GetRecoParam()->CombineClusterTrackReco() ) - { - tracker = new AliMUONTracker(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(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); } @@ -360,7 +388,7 @@ AliMUONReconstructor::CreateClusterFinder(const char* clusterFinderType) { /// Create a given cluster finder instance - AliCodeTimerAutoGeneral("") + AliCodeTimerAutoGeneral("",0) AliMUONVClusterFinder* clusterFinder(0x0); @@ -425,36 +453,42 @@ AliMUONReconstructor::CreateClusterFinder(const char* clusterFinderType) } //_____________________________________________________________________________ -void -AliMUONReconstructor::CreateClusterServer() const +AliMUONVClusterServer* +AliMUONReconstructor::CreateClusterServer(const AliMUONRecoParam& rp) const { /// Create cluster server - if ( fClusterServer ) return; + AliCodeTimerAuto("",0); - AliCodeTimerAuto(""); - - AliMUONVClusterFinder* clusterFinder = CreateClusterFinder(GetRecoParam()->GetClusteringMode()); + AliMUONVClusterServer* clusterServer = static_cast(fClusterServers.At(rp.GetEventSpecie())); - if ( !clusterFinder ) return; - - 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() ) { @@ -464,14 +498,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) + { + AliError("Could not get runinfo ?"); + } + else { - AliFatal("Could not access all required calibration data"); + 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(); @@ -480,9 +567,45 @@ AliMUONReconstructor::CreateCalibrator() const AliWarning("NOSTATUSMAP is obsolete"); } - TString calibMode = GetRecoParam()->GetCalibrationMode(); + Bool_t kTracker(kFALSE); + + const AliRunInfo* runInfo = GetRunInfo(); + if (!runInfo) + { + AliError("Could not get runinfo ?"); + } + else + { + TString detectors(runInfo->GetActiveDetectors()); + if (detectors.Contains("MUONTRK")) kTracker=kTRUE; + } + + if ( kTracker ) + { + fDigitCalibrator = new AliMUONDigitCalibrator(*fCalibrationData,GetRecoParam()); + } + else + { + AliWarning("Apparently running without MCH so will not instantiante the DigitCalibrator nor read MCH OCDB objects..."); + fShouldCalibrate=kFALSE; + } +} + +//_____________________________________________________________________________ +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,calibMode.Data()); + fTriggerProcessor->ResponseRemovingChambers(*triggerStore); } //_____________________________________________________________________________ @@ -523,21 +646,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 ( ! AliMUONReconstructor::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) { @@ -545,38 +669,40 @@ AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore, } } - AliMUONVClusterStore* clusterStore(0x0); - if ( !alone ) { - clusterStore = new AliMUONClusterStoreV2; + if (!fClusterStore) + { + 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 (AliMUONReconstructor::GetRecoParam()->UseChamber(i)) + if (rp->UseChamber(i)) { - if ( i >= 6 && AliMUONReconstructor::GetRecoParam()->BypassSt45() ) continue; + if ( ( i == 6 || i == 7 ) && rp->BypassSt4() ) continue; + if ( ( i == 8 || i == 9 ) && rp->BypassSt5() ) continue; - fClusterServer->Clusterize(i,*clusterStore,area); + clusterServer->Clusterize(i,*fClusterStore,area,rp); } } - Bool_t cok = clusterStore->Connect(clustersTree,alone); + Bool_t cok = fClusterStore->Connect(clustersTree,alone); if (!cok) AliError("Could not connect clusterStore to clusterTree"); - AliDebug(1,Form("Number of clusters found = %d",clusterStore->GetSize())); + AliDebug(1,Form("Number of clusters found = %d",fClusterStore->GetSize())); - StdoutToAliDebug(1,clusterStore->Print()); + StdoutToAliDebug(1,fClusterStore->Print()); } if (ok) // at least one type of branches created successfully @@ -584,9 +710,17 @@ AliMUONReconstructor::FillTreeR(AliMUONVTriggerStore* triggerStore, clustersTree.Fill(); } - delete clusterStore; + 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 @@ -631,7 +765,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,"");