]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - TPC/AliTPCQADataMakerRec.cxx
Fixes for bug #49914: Compilation breaks in trunk, and bug #48629: Trunk cannot read...
[u/mrichter/AliRoot.git] / TPC / AliTPCQADataMakerRec.cxx
index 96423d3b2e059d89489f5012dde5531d130ab1a7..8298ea45eb51899b40ea22042dfc389020fbca55 100644 (file)
@@ -68,19 +68,15 @@ ClassImp(AliTPCQADataMakerRec)
 
 //____________________________________________________________________________ 
 AliTPCQADataMakerRec::AliTPCQADataMakerRec() : 
-  AliQADataMakerRec(AliQA::GetDetName(AliQA::kTPC), 
+  AliQADataMakerRec(AliQAv1::GetDetName(AliQAv1::kTPC), 
                    "TPC Rec Quality Assurance Data Maker"),
-  fTPCdataQA(0),
-  fHistESDclusters(0),fHistESDratio(0), fHistESDpt(0),
-  fHistRawsOccupancy(0), fHistRawsOccupancyVsSector(0),
-  fHistRawsNClustersPerEventVsSector(0), fHistRawsQVsSector(0),
-  fHistRawsQmaxVsSector(0),
-  fHistRecPointsQmaxShort(0), fHistRecPointsQmaxMedium(0), 
-  fHistRecPointsQmaxLong(0), fHistRecPointsQShort(0), 
-  fHistRecPointsQMedium(0), fHistRecPointsQLong(0),
-  fHistRecPointsRow(0)
+  fTPCdataQA(NULL)
 {
   // ctor
+  fTPCdataQA = new AliTPCdataQA*[AliRecoParam::kNSpecies] ;
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) 
+    fTPCdataQA[specie] = NULL ; 
+  
   for(Int_t i = 0; i < 6; i++)
     fMapping[i] = 0;
 }
@@ -88,15 +84,7 @@ AliTPCQADataMakerRec::AliTPCQADataMakerRec() :
 //____________________________________________________________________________ 
 AliTPCQADataMakerRec::AliTPCQADataMakerRec(const AliTPCQADataMakerRec& qadm) :
   AliQADataMakerRec(),
-  fTPCdataQA(0),
-  fHistESDclusters(0),fHistESDratio(0), fHistESDpt(0),
-  fHistRawsOccupancy(0), fHistRawsOccupancyVsSector(0),
-  fHistRawsNClustersPerEventVsSector(0), fHistRawsQVsSector(0),
-  fHistRawsQmaxVsSector(0),
-  fHistRecPointsQmaxShort(0), fHistRecPointsQmaxMedium(0), 
-  fHistRecPointsQmaxLong(0), fHistRecPointsQShort(0), 
-  fHistRecPointsQMedium(0), fHistRecPointsQLong(0),
-  fHistRecPointsRow(0)
+  fTPCdataQA(NULL)
 {
   //copy ctor 
   // Does not copy the calibration object, instead InitRaws have to be
@@ -104,6 +92,10 @@ AliTPCQADataMakerRec::AliTPCQADataMakerRec(const AliTPCQADataMakerRec& qadm) :
   SetName((const char*)qadm.GetName()) ; 
   SetTitle((const char*)qadm.GetTitle()); 
 
+  fTPCdataQA = new AliTPCdataQA*[AliRecoParam::kNSpecies] ;
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) 
+    fTPCdataQA[specie] = NULL ; 
+  
   for(Int_t i = 0; i < 6; i++)
     fMapping[i] = 0;
 
@@ -111,32 +103,7 @@ AliTPCQADataMakerRec::AliTPCQADataMakerRec(const AliTPCQADataMakerRec& qadm) :
   // Associate class histogram objects to the copies in the list
   // Could also be done with the indexes
   //
-  fHistESDclusters   = (TH1F*)fESDsQAList->FindObject("hESDclusters");
-  fHistESDratio             = (TH1F*)fESDsQAList->FindObject("hESDratio");
-  fHistESDpt         = (TH1F*)fESDsQAList->FindObject("hESDpt");
-
-  fHistRawsOccupancy = (TH1F*)fRawsQAList->FindObject("hRawsOccupancy");
-  fHistRawsOccupancyVsSector = 
-    (TH1F*)fRawsQAList->FindObject("hRawsOccupancyVsSector");
-  fHistRawsNClustersPerEventVsSector = 
-    (TH1F*)fRawsQAList->FindObject("hRawsNClustersPerEventVsSector");
-  fHistRawsQVsSector = (TH1F*)fRawsQAList->FindObject("hRawsQVsSector");
-  fHistRawsQmaxVsSector = (TH1F*)fRawsQAList->FindObject("hRawsQmaxVsSector");
-  
-  fHistRecPointsQmaxShort  = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQmaxShort");
-  fHistRecPointsQmaxMedium = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQmaxMedium");
-  fHistRecPointsQmaxLong   = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQmaxLong"); 
-  fHistRecPointsQShort     = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQShort");
-  fHistRecPointsQMedium    = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQMedium");
-  fHistRecPointsQLong      = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsQLong");
-  fHistRecPointsRow      = 
-    (TH1F*)fRecPointsQAList->FindObject("hRecPointsRow");
+
 }
 
 //__________________________________________________________________
@@ -152,119 +119,139 @@ AliTPCQADataMakerRec& AliTPCQADataMakerRec::operator = (const AliTPCQADataMakerR
 AliTPCQADataMakerRec::~AliTPCQADataMakerRec()
 {
   // Destructor
-  delete fTPCdataQA; 
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) 
+    if ( fTPCdataQA[specie] != NULL )
+      delete fTPCdataQA[specie] ; 
+  delete[] fTPCdataQA; 
 
   for(Int_t i = 0; i < 6; i++) 
     delete fMapping[i];
 }
  
 //____________________________________________________________________________ 
-void AliTPCQADataMakerRec::EndOfDetectorCycle(AliQA::TASKINDEX_t task, TObjArray * list)
+void AliTPCQADataMakerRec::EndOfDetectorCycle(AliQAv1::TASKINDEX_t task, TObjArray ** list)
 {
   //Detector specific actions at end of cycle
+  
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) {
+    if ( !AliQAv1::Instance()->IsEventSpecieSet(specie) ) 
+      continue ; 
+    if(fTPCdataQA[specie] != NULL) { // do the final step of the QA for Raw data
 
-  if(fTPCdataQA) { // do the final step of the QA for Raw data
-
-    fTPCdataQA->Analyse(); // 31/1-08 Analyse is now protected against
+      fTPCdataQA[specie]->Analyse(); // 31/1-08 Analyse is now protected against
                            //         RAW data files with no TPC data
-    
-    //Add2RawsList(fTPCdataQA, 0);
-    // get the histograms and add them to the output
-    // 31/8-08 Histogram is only added if the Calibration class 
-    //         receives TPC data 
-    const Int_t eventCounter = fTPCdataQA->GetEventCounter();
-    if(eventCounter>0) { // some TPC data has been processed
-
-      // Reset histograms and refill them 
-      fHistRawsOccupancy->Reset();
-      fHistRawsOccupancyVsSector->Reset();
-      fHistRawsNClustersPerEventVsSector->Reset();
-      fHistRawsQVsSector->Reset();
-      fHistRawsQmaxVsSector->Reset();
+
+      SetEventSpecie(specie) ; 
+      TH1F * histRawsOccupancy                 = (TH1F*)GetRawsData(kOccupancy) ;
+      TH1F * histRawsOccupancyVsSector         = (TH1F*)GetRawsData(kOccupancyVsSector) ;
+      TH1F * histRawsNClustersPerEventVsSector = (TH1F*)GetRawsData(kNClustersPerEventVsSector) ;
+      TH1F * histRawsQVsSector                 = (TH1F*)GetRawsData(kQVsSector) ;
+      TH1F * histRawsQmaxVsSector              = (TH1F*)GetRawsData(kQmaxVsSector) ;
+      if ( !histRawsOccupancy ||
+          !histRawsOccupancyVsSector ||
+          !histRawsNClustersPerEventVsSector ||
+          !histRawsQVsSector ||
+          !histRawsQmaxVsSector) {
+        AliError("Something very wrong here, corrupted memory ?????. Please check\n") ; 
+        continue ; 
+      }
+        
+      //Add2RawsList(fTPCdataQA, 0);
+      // get the histograms and add them to the output
+      // 31/8-08 Histogram is only added if the Calibration class 
+      //         receives TPC data 
+      const Int_t eventCounter = fTPCdataQA[specie]->GetEventCounter();
+      if(eventCounter>0) { // some TPC data has been processed
+
+        // Reset histograms and refill them 
+        histRawsOccupancy->Reset();
+        histRawsOccupancyVsSector->Reset();
+        histRawsNClustersPerEventVsSector->Reset();
+        histRawsQVsSector->Reset();
+        histRawsQmaxVsSector->Reset();
       
-      TH1F* hNorm72 = new TH1F("hNorm72", "histogram to normalize 72 sectors",
-                              72, 0, 72);
-      hNorm72->Sumw2();
-      TH1F* hNorm108 = new TH1F("hNorm108", "histogram to normalize 108 sectors (medium and long pads are split up)",
-                              108, 0, 108);
-      hNorm108->Sumw2();
-
-      for (Int_t iSec = 0; iSec < 72; iSec++) {
+        TH1F* hNorm72 = new TH1F("hNorm72", "histogram to normalize 72 sectors",
+                                 72, 0, 72);
+        hNorm72->Sumw2();
+        TH1F* hNorm108 = new TH1F("hNorm108", "histogram to normalize 108 sectors (medium and long pads are split up)",
+                                  108, 0, 108);
+        hNorm108->Sumw2();
+
+        for (Int_t iSec = 0; iSec < 72; iSec++) {
        
-       AliTPCCalROC* occupancyROC = 
-         fTPCdataQA->GetNoThreshold()->GetCalROC(iSec); 
-       AliTPCCalROC* nclusterROC = 
-         fTPCdataQA->GetNLocalMaxima()->GetCalROC(iSec); 
-       AliTPCCalROC* qROC = 
-         fTPCdataQA->GetMeanCharge()->GetCalROC(iSec); 
-       AliTPCCalROC* qmaxROC = 
-         fTPCdataQA->GetMaxCharge()->GetCalROC(iSec); 
-
-       const Int_t nRows = occupancyROC->GetNrows(); 
-       for (Int_t iRow = 0; iRow < nRows; iRow++) {
-
-         Int_t helpSector = iSec;
-         if(iRow>=64)
-           helpSector += 36; // OROC (long pads)
-
-         const Int_t nPads = occupancyROC->GetNPads(iRow); 
-         for (Int_t iPad = 0; iPad < nPads; iPad++) {
+          AliTPCCalROC* occupancyROC = 
+          fTPCdataQA[specie]->GetNoThreshold()->GetCalROC(iSec); 
+          AliTPCCalROC* nclusterROC = 
+          fTPCdataQA[specie]->GetNLocalMaxima()->GetCalROC(iSec); 
+          AliTPCCalROC* qROC = 
+          fTPCdataQA[specie]->GetMeanCharge()->GetCalROC(iSec); 
+          AliTPCCalROC* qmaxROC = 
+          fTPCdataQA[specie]->GetMaxCharge()->GetCalROC(iSec); 
+
+          const Int_t nRows = occupancyROC->GetNrows(); 
+          for (Int_t iRow = 0; iRow < nRows; iRow++) {
+
+            Int_t helpSector = iSec;
+            if(iRow>=64)
+              helpSector += 36; // OROC (long pads)
+
+            const Int_t nPads = occupancyROC->GetNPads(iRow); 
+            for (Int_t iPad = 0; iPad < nPads; iPad++) {
        
-           fHistRawsOccupancy->Fill(occupancyROC->GetValue(iRow, iPad));
-           hNorm72->Fill(iSec);
-           fHistRawsOccupancyVsSector
-             ->Fill(iSec, occupancyROC->GetValue(iRow, iPad));
+              histRawsOccupancy->Fill(occupancyROC->GetValue(iRow, iPad));
+              hNorm72->Fill(iSec);
+              histRawsOccupancyVsSector
+              ->Fill(iSec, occupancyROC->GetValue(iRow, iPad));
 
-           const Int_t nClusters = TMath::Nint(nclusterROC->GetValue(iRow, iPad));
+              const Int_t nClusters = TMath::Nint(nclusterROC->GetValue(iRow, iPad));
            
-           if(nClusters>0) {
+              if(nClusters>0) {
              
-             fHistRawsNClustersPerEventVsSector->Fill(iSec, nClusters);
-             hNorm108->Fill(helpSector, nClusters);
-             fHistRawsQVsSector->Fill(helpSector, 
-                                      nClusters*qROC->GetValue(iRow, iPad));
-             fHistRawsQmaxVsSector->Fill(helpSector, 
-                                         nClusters*qmaxROC->GetValue(iRow, iPad));
-           }
-         }
-       }
-      } // end loop over sectors
+                histRawsNClustersPerEventVsSector->Fill(iSec, nClusters);
+                hNorm108->Fill(helpSector, nClusters);
+                histRawsQVsSector->Fill(helpSector, 
+                                       nClusters*qROC->GetValue(iRow, iPad));
+                histRawsQmaxVsSector->Fill(helpSector, 
+                                          nClusters*qmaxROC->GetValue(iRow, iPad));
+              }
+            }
+          }
+        } // end loop over sectors
       
-      // Normalize histograms
-      fHistRawsOccupancyVsSector->Divide(hNorm72);
-      fHistRawsNClustersPerEventVsSector->Scale(1.0/Float_t(eventCounter));
-      fHistRawsQVsSector->Divide(hNorm108);
-      fHistRawsQmaxVsSector->Divide(hNorm108);
-      delete hNorm72;
-      delete hNorm108;
-
+        // Normalize histograms
+        histRawsOccupancyVsSector->Divide(hNorm72);
+        histRawsNClustersPerEventVsSector->Scale(1.0/Float_t(eventCounter));
+        histRawsQVsSector->Divide(hNorm108);
+        histRawsQmaxVsSector->Divide(hNorm108);
+        delete hNorm72;
+        delete hNorm108;
+      }
     }
   }
-
-  AliQAChecker::Instance()->Run(AliQA::kTPC, task, list) ;  
+  AliQAChecker::Instance()->Run(AliQAv1::kTPC, task, list) ;  
 }
 
 //____________________________________________________________________________ 
 void AliTPCQADataMakerRec::InitESDs()
 {
   //create ESDs histograms in ESDs subdir  
-  fHistESDclusters = 
+  TH1F * histESDclusters = 
     new TH1F("hESDclusters", "N TPC clusters per track; N clusters; Counts",
             160, 0, 160);
-  fHistESDclusters->Sumw2();
-  Add2ESDsList(fHistESDclusters, 0);
+  histESDclusters->Sumw2();
+  Add2ESDsList(histESDclusters, KClusters);
 
-  fHistESDratio = 
+  TH1F * histESDratio = 
     new TH1F("hESDratio", "Ratio: TPC clusters / findable; Ratio: cluster/findable; Counts",
             100, 0, 1);
-  fHistESDratio->Sumw2();
-  Add2ESDsList(fHistESDratio, 1);
+  histESDratio->Sumw2();
+  Add2ESDsList(histESDratio, kRatio);
   
-  fHistESDpt = 
+  TH1F * histESDpt = 
     new TH1F("hESDpt", "P_{T} distribution; p_{T} [GeV/c]; Counts",
             50, 0, 5);
-  fHistESDpt->Sumw2();
-  Add2ESDsList(fHistESDpt, 2);
+  histESDpt->Sumw2();
+  Add2ESDsList(histESDpt, kPt);
 }
 
 //____________________________________________________________________________ 
@@ -278,87 +265,89 @@ void AliTPCQADataMakerRec::InitRaws()
   // Laurent Aphecetche pointed out that the mapping was read from file
   // for each event, so now we read in the map here and set if for 
   // the raw data qa
-  fTPCdataQA = new AliTPCdataQA();
-  LoadMaps(); // Load Altro maps
-  fTPCdataQA->SetAltroMapping(fMapping); // set Altro mapping
-  fTPCdataQA->SetRangeTime(100, 920); // set time bin interval 
-  Add2RawsList(fTPCdataQA, 0); // This is used by the AMORE monitoring
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) {
+    fTPCdataQA[specie] = new AliTPCdataQA(AliRecoParam::Convert(specie));
+    LoadMaps(); // Load Altro maps
+    fTPCdataQA[specie]->SetAltroMapping(fMapping); // set Altro mapping
+    fTPCdataQA[specie]->SetRangeTime(100, 920); // set time bin interval 
+//    Add2RawsList(fTPCdataQA, kTPCdataQA); // This is used by the AMORE monitoring <------- THIS WILL FAIL (YS)
+  }
 
-  fHistRawsOccupancy = 
+  TH1F * histRawsOccupancy = 
     new TH1F("hRawsOccupancy", "Occupancy (all pads); Occupancy; Counts",
             100, 0, 1);
-  fHistRawsOccupancy->Sumw2();
-  Add2RawsList(fHistRawsOccupancy, 1);
-
-  fHistRawsOccupancyVsSector = 
+  histRawsOccupancy->Sumw2();
+  Add2RawsList(histRawsOccupancy, kOccupancy);
+  
+  TH1F * histRawsOccupancyVsSector = 
     new TH1F("hRawsOccupancyVsSector", "Occupancy vs sector; Sector; Occupancy",
             72, 0, 72);
-  fHistRawsOccupancyVsSector->Sumw2();
-  Add2RawsList(fHistRawsOccupancyVsSector, 2);
+  histRawsOccupancyVsSector->Sumw2();
+  Add2RawsList(histRawsOccupancyVsSector, kOccupancyVsSector);
 
-  fHistRawsNClustersPerEventVsSector = 
+  TH1F * histRawsNClustersPerEventVsSector = 
     new TH1F("hRawsNClustersPerEventVsSector", "Nclusters per event vs sector; Sector; Nclusters per event",
             72, 0, 72);
-  fHistRawsNClustersPerEventVsSector->Sumw2();
-  Add2RawsList(fHistRawsNClustersPerEventVsSector, 3);
+  histRawsNClustersPerEventVsSector->Sumw2();
+  Add2RawsList(histRawsNClustersPerEventVsSector, kNClustersPerEventVsSector);
   
-  fHistRawsQVsSector = 
+  TH1F * histRawsQVsSector = 
     new TH1F("hRawsQVsSector", "<Q> vs sector (OROC med: 36-71, long: 72-107); Sector; <Q>",
             108, 0, 108);
-  fHistRawsQVsSector->Sumw2();
-  Add2RawsList(fHistRawsQVsSector, 4);
+  histRawsQVsSector->Sumw2();
+  Add2RawsList(histRawsQVsSector, kQVsSector);
 
-  fHistRawsQmaxVsSector = 
+  TH1F * histRawsQmaxVsSector = 
     new TH1F("hRawsQmaxVsSector", "<Qmax> vs sector (OROC med: 36-71, long: 72-107); Sector; <Qmax>",
             108, 0, 108);
-  fHistRawsQmaxVsSector->Sumw2();
-  Add2RawsList(fHistRawsQmaxVsSector, 5);
+  histRawsQmaxVsSector->Sumw2();
+  Add2RawsList(histRawsQmaxVsSector, kQmaxVsSector);
 }
 
 //____________________________________________________________________________ 
 void AliTPCQADataMakerRec::InitRecPoints()
 {
-  fHistRecPointsQmaxShort = 
+  TH1F * histRecPointsQmaxShort = 
     new TH1F("hRecPointsQmaxShort", "Qmax distrbution (short pads); Qmax; Counts",
             100, 0, 300);
-  fHistRecPointsQmaxShort->Sumw2();
-  Add2RecPointsList(fHistRecPointsQmaxShort, 0);
+  histRecPointsQmaxShort->Sumw2();
+  Add2RecPointsList(histRecPointsQmaxShort, kQmaxShort);
 
-  fHistRecPointsQmaxMedium = 
+  TH1F * histRecPointsQmaxMedium = 
     new TH1F("hRecPointsQmaxMedium", "Qmax distrbution (medium pads); Qmax; Counts",
             100, 0, 300);
-  fHistRecPointsQmaxMedium->Sumw2();
-  Add2RecPointsList(fHistRecPointsQmaxMedium, 1);
+  histRecPointsQmaxMedium->Sumw2();
+  Add2RecPointsList(histRecPointsQmaxMedium, kQmaxMedium);
 
-  fHistRecPointsQmaxLong = 
+  TH1F * histRecPointsQmaxLong = 
     new TH1F("hRecPointsQmaxLong", "Qmax distrbution (long pads); Qmax; Counts",
             100, 0, 300);
-  fHistRecPointsQmaxLong->Sumw2();
-  Add2RecPointsList(fHistRecPointsQmaxLong, 2);
+  histRecPointsQmaxLong->Sumw2();
+  Add2RecPointsList(histRecPointsQmaxLong, kQmaxLong);
 
-  fHistRecPointsQShort = 
+  TH1F * histRecPointsQShort = 
     new TH1F("hRecPointsQShort", "Q distrbution (short pads); Q; Counts",
             100, 0, 2000);
-  fHistRecPointsQShort->Sumw2();
-  Add2RecPointsList(fHistRecPointsQShort, 3);
+  histRecPointsQShort->Sumw2();
+  Add2RecPointsList(histRecPointsQShort, kQShort);
 
-  fHistRecPointsQMedium = 
+  TH1F * histRecPointsQMedium = 
     new TH1F("hRecPointsQMedium", "Q distrbution (medium pads); Q; Counts",
             100, 0, 2000);
-  fHistRecPointsQMedium->Sumw2();
-  Add2RecPointsList(fHistRecPointsQMedium, 4);
+  histRecPointsQMedium->Sumw2();
+  Add2RecPointsList(histRecPointsQMedium, kQMedium);
 
-  fHistRecPointsQLong = 
+  TH1F * histRecPointsQLong = 
     new TH1F("hRecPointsQLong", "Q distrbution (long pads); Q; Counts",
             100, 0, 2000);
-  fHistRecPointsQLong->Sumw2();
-  Add2RecPointsList(fHistRecPointsQLong, 5);
+  histRecPointsQLong->Sumw2();
+  Add2RecPointsList(histRecPointsQLong, kQLong);
 
-  fHistRecPointsRow = 
+  TH1F * histRecPointsRow = 
     new TH1F("hRecPointsRow", "Clusters per row; Row; Counts",
             159, 0, 159);
-  fHistRecPointsRow->Sumw2();
-  Add2RecPointsList(fHistRecPointsRow, 6);
+  histRecPointsRow->Sumw2();
+  Add2RecPointsList(histRecPointsRow, kRow);
 }
 
 //____________________________________________________________________________
@@ -380,9 +369,9 @@ void AliTPCQADataMakerRec::MakeESDs(AliESDEvent * esd)
     Int_t nTPCclusters         = track->GetTPCNcls();
     Int_t nTPCclustersFindable = track->GetTPCNclsF();
     if ( nTPCclustersFindable<=0) continue;
-    fHistESDclusters->Fill(nTPCclusters);
-    fHistESDratio->Fill(Float_t(nTPCclusters)/Float_t(nTPCclustersFindable));
-    fHistESDpt->Fill(track->Pt()); 
+    GetESDsData(KClusters)->Fill(nTPCclusters);
+    GetESDsData(kRatio)->Fill(Float_t(nTPCclusters)/Float_t(nTPCclustersFindable));
+    GetESDsData(kPt)->Fill(track->Pt()); 
   }
 }
 
@@ -394,8 +383,8 @@ void AliTPCQADataMakerRec::MakeRaws(AliRawReader* rawReader)
   // to handle the data and then in the end extract the data
   //
        rawReader->Reset() ; 
-  fTPCdataQA->ProcessEvent(rawReader);  
-}
+  fTPCdataQA[AliRecoParam::AConvert(fEventSpecie)]->ProcessEvent(rawReader);  
+ }
 
 //____________________________________________________________________________
 void AliTPCQADataMakerRec::MakeRecPoints(TTree* recTree)
@@ -425,22 +414,22 @@ void AliTPCQADataMakerRec::MakeRecPoints(TTree* recTree)
 
       if(cluster->GetDetector()<36) { // IROC (short pads)
 
-       fHistRecPointsQmaxShort->Fill(Qmax);
-       fHistRecPointsQShort->Fill(Q);
+       GetRecPointsData(kQmaxShort)->Fill(Qmax);
+       GetRecPointsData(kQShort)->Fill(Q);
       } else { // OROC (medium and long pads)
        row += 63;
        if(cluster->GetRow()<64) { // medium pads
 
-         fHistRecPointsQmaxMedium->Fill(Qmax);
-         fHistRecPointsQMedium->Fill(Q);
+         GetRecPointsData(kQmaxMedium)->Fill(Qmax);
+         GetRecPointsData(kQMedium)->Fill(Q);
        } else { // long pads
 
-         fHistRecPointsQmaxLong->Fill(Qmax);
-         fHistRecPointsQLong->Fill(Q);
+         GetRecPointsData(kQmaxLong)->Fill(Qmax);
+         GetRecPointsData(kQLong)->Fill(Q);
        }
       }
       
-      fHistRecPointsRow->Fill(row);
+      GetRecPointsData(kRow)->Fill(row);
     } // end loop over clusters
   } // end loop over tree