]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - ITS/AliITSQADataMakerRec.cxx
Fixes for Coverity defects
[u/mrichter/AliRoot.git] / ITS / AliITSQADataMakerRec.cxx
index ffe46214790fc71b951434536742fe4e5d340d53..9945df216927b2275fa4c1684334589aa4429336 100644 (file)
@@ -27,7 +27,6 @@
 
 // --- ROOT system ---
 #include <TH2.h>
-#include <TTree.h>
 // --- Standard library ---
 
 // --- AliRoot header files ---
 #include "AliITSQASPDDataMakerRec.h"
 #include "AliITSQASDDDataMakerRec.h"
 #include "AliITSQASSDDataMakerRec.h"
-#include "AliLog.h"
 #include "AliQAv1.h"
 #include "AliQAChecker.h"
 #include "AliITSQAChecker.h"
 #include "AliITSRecPoint.h"
+#include "AliITSRecPointContainer.h"
 #include "AliRawReader.h"
 #include "AliESDEvent.h"
 #include "AliESDtrack.h"
-#include "AliESDVertex.h"
 #include "AliMultiplicity.h"
 #include "AliITSgeomTGeo.h"
 
+//class TH2;
+//class TH2F;
+class AliESDVertex;
+class AliLog;
+class TTree;
+
 ClassImp(AliITSQADataMakerRec)
 
 //____________________________________________________________________________ 
@@ -56,6 +60,8 @@ fkOnline(kMode),
 fSubDetector(subDet),
 fLDC(ldc),
 fRunNumber(0),
+fEventNumber(0),
+fSelectedTaskIndex(AliQAv1::kNULLTASKINDEX),
 fSPDDataMaker(NULL),
 fSDDDataMaker(NULL),
 fSSDDataMaker(NULL)
@@ -95,7 +101,9 @@ AliQADataMakerRec(),
 fkOnline(qadm.fkOnline),
 fSubDetector(qadm.fSubDetector),
 fLDC(qadm.fLDC),
-fRunNumber(0),
+fRunNumber(qadm.fRunNumber),
+fEventNumber(qadm.fEventNumber),
+fSelectedTaskIndex(qadm.fSelectedTaskIndex),
 fSPDDataMaker(NULL),
 fSDDDataMaker(NULL),
 fSSDDataMaker(NULL)
@@ -125,6 +133,14 @@ void AliITSQADataMakerRec::StartOfDetectorCycle()
   if(fSubDetector == 0 || fSubDetector == 3) fSSDDataMaker->StartOfDetectorCycle();
 }
 
+//____________________________________________________________________________
+void AliITSQADataMakerRec::StartOfCycle(AliQAv1::TASKINDEX_t task, Int_t run, const Bool_t sameCycle) 
+{ 
+  // Start a cycle of QA data acquistion
+  fSelectedTaskIndex=task;
+  AliQADataMakerRec::StartOfCycle(task,run,sameCycle);
+}
+
 //____________________________________________________________________________ 
 void AliITSQADataMakerRec::EndOfDetectorCycle(AliQAv1::TASKINDEX_t task, TObjArray** list)
 {
@@ -136,11 +152,10 @@ void AliITSQADataMakerRec::EndOfDetectorCycle(AliQAv1::TASKINDEX_t task, TObjArr
        if(AliQAv1::Instance()->IsEventSpecieSet(specie)){
          Int_t idnumber=list[specie]->GetUniqueID();
          //printf("specie %s \t id number == %d\n",AliRecoParam::GetEventSpecieName(specie),idnumber);
-         if(idnumber==40||idnumber==0)
-           {
+         if(idnumber==40||idnumber==0){
              //AliInfo(Form("No check for %s\n",AliQAv1::GetTaskName(task).Data() ))
-               continue;
-           } //skip kDigitsR and not filled TobjArray specie
+           continue;
+         } //skip kDigitsR and not filled TobjArray specie
          else{
            AliDebug(AliQAv1::GetQADebugLevel(),"AliITSDM instantiates checker with Run(AliQAv1::kITS, task, list[specie])\n"); 
            if(fSubDetector == 0 || fSubDetector == 1) fSPDDataMaker->EndOfDetectorCycle(task, list[/*GetEventSpecie()*/specie]);
@@ -173,14 +188,14 @@ void AliITSQADataMakerRec::EndOfDetectorCycle(AliQAv1::TASKINDEX_t task, TObjArr
 }
 
 //____________________________________________________________________________ 
-void AliITSQADataMakerRec::EndOfDetectorCycle(const char * /*fgDataName*/)
-{
+//void AliITSQADataMakerRec::EndOfDetectorCycle(const char * /*fgDataName*/)
+//{
   //eventually used for different  AliQAChecker::Instance()->Run
-}
+//}
 
 //____________________________________________________________________________ 
-void AliITSQADataMakerRec::InitRaws()
-{  
+void AliITSQADataMakerRec::InitRaws() {
+  // Initialization of RAW data histograms  
 
   //if(fRawsQAList[AliRecoParam::AConvert(fEventSpecie)]->GetEntries()) return;
        
@@ -297,13 +312,9 @@ void AliITSQADataMakerRec::InitRecPoints()
          Int_t offset = fRecPointsQAList [AliRecoParam::AConvert(fEventSpecie)]->GetEntries();
          const Bool_t expert   = kTRUE ; 
          const Bool_t image    = kTRUE ; 
-         Char_t name[50];
-         Char_t title[50];
-         TH2F**hPhiEta = new TH2F*[6];
+         TH2F* hPhiEta[6];
          for (Int_t iLay=0;iLay<6;iLay++) {
-           sprintf(name,"Phi_vs_Eta_ITS_Layer%d",iLay+1);
-           sprintf(title,"Phi vs Eta - ITS Layer %d",iLay+1);
-           hPhiEta[iLay]=new TH2F(name,title,30,-1.5,1.5,200,0.,2*TMath::Pi());
+           hPhiEta[iLay]=new TH2F(Form("Phi_vs_Eta_ITS_Layer%d",iLay+1),Form("Phi_vs_Eta_ITS_Layer%d",iLay+1),30,-1.5,1.5,200,0.,2*TMath::Pi());
            hPhiEta[iLay]->GetXaxis()->SetTitle("Pseudorapidity");
            hPhiEta[iLay]->GetYaxis()->SetTitle("#varphi [rad]");
            Add2RecPointsList(hPhiEta[iLay], iLay + offset, !expert, image);
@@ -317,9 +328,9 @@ void AliITSQADataMakerRec::InitRecPoints()
 
 //____________________________________________________________________________ 
 void AliITSQADataMakerRec::MakeRecPoints(TTree * clustersTree)
-{
+{ 
   // Fill QA for recpoints
+
   if(fSubDetector == 0 || fSubDetector == 1) {
     fSPDDataMaker->MakeRecPoints(clustersTree) ; 
   }
@@ -331,25 +342,30 @@ void AliITSQADataMakerRec::MakeRecPoints(TTree * clustersTree)
   if(fSubDetector == 0 || fSubDetector == 3) fSSDDataMaker->MakeRecPoints(clustersTree);
 
 
-
+  
   if(fSubDetector == 0){
+
     // Check id histograms already created for this Event Specie
-    TBranch *branchRecP = clustersTree->GetBranch("ITSRecPoints");
-    if (!branchRecP) {
-      AliError("can't get the branch with the ITS clusters !");
+    AliITSRecPointContainer* rpcont=AliITSRecPointContainer::Instance();
+    TClonesArray *recpoints =NULL;
+    if(fkOnline){
+      recpoints= rpcont->FetchClusters(0,clustersTree,GetEventNumber());
+    } 
+    else{
+      recpoints= rpcont->FetchClusters(0,clustersTree);
+    }
+    if(!rpcont->GetStatusOK()){
+      AliError("cannot access to ITS recpoints");
       return;
     }
-    
+  
     Int_t offset = fRecPointsQAList [AliRecoParam::AConvert(fEventSpecie)]->GetEntries();
     Float_t cluGlo[3] = {0.,0.,0.};
     Int_t lay, lad, det; 
-    static TClonesArray statRecpoints("AliITSRecPoint") ;
-    TClonesArray *recpoints = &statRecpoints;
-    branchRecP->SetAddress(&recpoints);
     // Fill QA for recpoints
-    for(Int_t module=0; module<clustersTree->GetEntries();module++){
+    for(Int_t module=0; module<rpcont->GetNumberOfModules();module++){
       //  AliInfo(Form("Module %d\n",module));
-      branchRecP->GetEvent(module);
+      recpoints = rpcont->UncheckedGetClusters(module);
       AliITSgeomTGeo::GetModuleId(module, lay, lad, det);
       for(Int_t j=0;j<recpoints->GetEntries();j++){
        AliITSRecPoint *rcp = (AliITSRecPoint*)recpoints->At(j);    
@@ -573,7 +589,7 @@ void AliITSQADataMakerRec::MakeESDs(AliESDEvent *esd)
   // Check id histograms already created for this Event Specie
 //  if ( ! GetESDsData(0) )
 //    InitESDs() ;
-  
   const Int_t nESDTracks = esd->GetNumberOfTracks();
   Int_t nITSrefit5 = 0; 
 
@@ -581,6 +597,7 @@ void AliITSQADataMakerRec::MakeESDs(AliESDEvent *esd)
   Float_t xloc,zloc;
 
   // loop on tracks
+  AliInfo(Form("Filling histograms for ESD. Number of tracks %d",nESDTracks)); 
   for(Int_t i = 0; i < nESDTracks; i++) {
     
     AliESDtrack *track = esd->GetTrack(i);
@@ -620,6 +637,7 @@ void AliITSQADataMakerRec::MakeESDs(AliESDEvent *esd)
   const AliESDVertex *vtxTrk = esd->GetPrimaryVertexTracks();
 
   Int_t mult = ((AliMultiplicity*)(esd->GetMultiplicity()))->GetNumberOfTracklets();
+  AliInfo(Form("Multiplicity %d ; Number of SPD vert contributors %d",mult,vtxSPD->GetNContributors()));
   if(mult>0)
     GetESDsData(7)->Fill((Float_t)(vtxSPD->GetNContributors())/(Float_t)mult);
 
@@ -672,7 +690,7 @@ void AliITSQADataMakerRec::MakeESDs(AliESDEvent *esd)
 //_________________________________________________________________
 Int_t AliITSQADataMakerRec::GetDetTaskOffset(Int_t subdet,AliQAv1::TASKINDEX_t task, Int_t specie)
 {
-
+  //number of booked histos for the QAchecking Raws offset
   Int_t offset=0;
   switch(subdet)
     {
@@ -708,7 +726,7 @@ Bool_t AliITSQADataMakerRec::AreEqual(Double_t a1,Double_t a2)
 //_________________________________________________________________
 Int_t AliITSQADataMakerRec::GetDetTaskHisto(Int_t subdet,AliQAv1::TASKINDEX_t task)
 {
-
+  //return the number of histo booked for each the Raws Task 
 
   Int_t histo=0;
   switch(subdet)
@@ -734,3 +752,43 @@ Int_t AliITSQADataMakerRec::GetDetTaskHisto(Int_t subdet,AliQAv1::TASKINDEX_t ta
   //return offset;
   return histo;
 }
+
+
+//____________________________________________________________________
+
+void AliITSQADataMakerRec::ResetDetector(AliQAv1::TASKINDEX_t task)
+{
+  //reset the detector histograms for a given task
+  AliQADataMakerRec::ResetDetector(task);
+
+  if(fSubDetector==0||fSubDetector==1)fSPDDataMaker->ResetDetector(task);
+  
+  if(fSubDetector==0||fSubDetector==2)fSDDDataMaker->ResetDetector(task);
+
+  if(fSubDetector==0||fSubDetector==3)fSSDDataMaker->ResetDetector(task);
+  
+}
+
+
+//____________________________________________________________________
+
+AliITSDDLModuleMapSDD *AliITSQADataMakerRec::GetDDLSDDModuleMap()
+{
+  //return the SDD module map
+  if(fSubDetector==2){return fSDDDataMaker->GetDDLSDDModuleMap();}
+  else {return NULL;}
+}
+
+//____________________________________________________________________
+
+Bool_t AliITSQADataMakerRec::ListExists(AliQAv1::TASKINDEX_t task) const
+{
+  //Check the existence of a list for a given task
+  Bool_t havethelist=kFALSE;
+  if( ( task == AliQAv1::kRAWS && fRawsQAList ) ||
+      ( task == AliQAv1::kRECPOINTS && fRecPointsQAList ) ||
+      ( task == AliQAv1::kDIGITSR && fDigitsQAList ) ||
+      ( task == AliQAv1::kESDS && fESDsQAList ) ) havethelist=kTRUE;
+  return havethelist;
+
+}