]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - TRD/AliTRDQADataMaker.cxx
Removing useless flag.
[u/mrichter/AliRoot.git] / TRD / AliTRDQADataMaker.cxx
index 0b87cffe3dc024f46c87e0ea53f121b36be829a6..e95096f9fa400af1e93d28cbd70a3c33d483ed46 100644 (file)
 
 ////////////////////////////////////////////////////////////////////////////
 //                                                                        //
-// Produces the data needed to calculate the quality assurance.           //
-// All data must be mergeable objects.                                    //
-// S.Radomski Uni-Heidelberg October 2007                                 //
+//  Produces the data needed to calculate the quality assurance.          //
+//  All data must be mergeable objects.                                   //
+//                                                                        //
+//  Author:                                                               //
+//    Sylwester Radomski (radomski@physi.uni-heidelberg.de)               //
 //                                                                        //
 ////////////////////////////////////////////////////////////////////////////
 
 // --- ROOT system ---
 #include <TClonesArray.h>
-#include <TFile.h> 
-#include <TH1D.h> 
-#include <TH2D.h>
+#include <TH1F.h> 
+#include <TH2F.h>
+#include <TH3F.h>
 #include <TProfile.h>
+#include <TF1.h>
 
 // --- AliRoot header files ---
 #include "AliESDEvent.h"
-#include "AliLog.h"
+#include "AliQAChecker.h"
+
 #include "AliTRDdigit.h"
 #include "AliTRDhit.h"
 #include "AliTRDcluster.h"
 #include "AliTRDQADataMaker.h"
 #include "AliTRDdigitsManager.h"
 #include "AliTRDgeometry.h"
-#include "AliTRDdataArrayI.h"
-#include "AliTRDRawStreamV2.h"
+#include "AliTRDarrayADC.h"
 
 ClassImp(AliTRDQADataMaker)
-           
+
 //____________________________________________________________________________ 
   AliTRDQADataMaker::AliTRDQADataMaker() : 
-  AliQADataMaker(AliQA::GetDetName(AliQA::kTRD), "TRD Quality Assurance Data Maker")
+  AliQADataMaker(AliQAv1::GetDetName(AliQAv1::kTRD), "TRD Quality Assurance Data Maker")
 {
-  // ctor
+  //
+  // Default constructor
 }
 
 //____________________________________________________________________________ 
 AliTRDQADataMaker::AliTRDQADataMaker(const AliTRDQADataMaker& qadm) :
   AliQADataMaker()
 {
-  //copy ctor 
+  //
+  // Copy constructor 
+  //
+
   SetName((const char*)qadm.GetName()) ; 
   SetTitle((const char*)qadm.GetTitle()); 
+
 }
 
 //__________________________________________________________________
-AliTRDQADataMaker& AliTRDQADataMaker::operator = (const AliTRDQADataMaker& qadm )
+AliTRDQADataMaker& AliTRDQADataMaker::operator=(const AliTRDQADataMaker& qadm)
 {
+  //
   // Equal operator.
+  //
+
   this->~AliTRDQADataMaker();
   new(this) AliTRDQADataMaker(qadm);
   return *this;
+
 }
+
 //____________________________________________________________________________ 
-void AliTRDQADataMaker::EndOfDetectorCycle()
+void AliTRDQADataMaker::EndOfDetectorCycle(AliQAv1::TASKINDEX task, TObjArray * list)
 {
-  //Detector specific actions at end of cycle
-
-  // Rec points
+  //
+  // Detector specific actions at end of cycle
+  //
+  //TStopwatch watch;
+  //watch.Start();
+  ResetEventTrigClasses();
+  //
+  //AliDebug(AliQAv1::GetQADebugLevel(), Form("EndOfCycle", "Fitting RecPoints %d", task))
+  TH1F *hist = new TH1F("fitHist", "", 200, -0.5, 199.5);
+  //
+  // RS Add a loop over species
+  for (Int_t specie = 0 ; specie < AliRecoParam::kNSpecies ; specie++) {
+    if ( !AliQAv1::Instance()->IsEventSpecieSet(specie) ) continue ; 
+    SetEventSpecie(specie);
+    //
+    for (int itc=-1;itc<GetNTrigClasses();itc++) { // RS: loop over eventual clones per trigger class
+      //
+      if (task == AliQAv1::kRECPOINTS) {
+
+       //list->Print();
+       TObjArray& arrRP = *GetRecPointsDataOfTrigClass(itc); // RS Histos matching to trigger class
+       // Rec points full chambers
+       TH2* h2tmp = (TH2*) arrRP[1];
+       if (h2tmp) {
+         for (Int_t i=0; i<540; i++) {
+           hist->Reset();
+           for(Int_t b=1; b<hist->GetXaxis()->GetNbins()-1; b++) {
+             Double_t xvalue = hist->GetBinCenter(b);
+             Int_t bin = h2tmp->FindBin(i,xvalue);
+             Double_t value =  h2tmp->GetBinContent(bin);
+             hist->SetBinContent(b, value);
+           }
+           //printf("Sum = %d %f\n", i, hist->GetSum());
+           if (hist->GetSum() < 100) continue; // chamber not present
+           //
+           hist->Fit("landau", "q0", "goff", 10, 180);
+           TF1 *fit = hist->GetFunction("landau");
+           if (arrRP[12]) ((TH1*)arrRP[12])->Fill(fit->GetParameter(1));
+           if (arrRP[13]) ((TH1*)arrRP[13])->Fill(fit->GetParameter(2));
+         }
+       }
+       //
+       // time-bin by time-bin
+       TH3* h3tmp = (TH3*) arrRP[10]; 
+       if (h3tmp) {
+         for (Int_t i=0; i<540; i++) {
+           for(Int_t j=0; j<35; j++) {
+             hist->Reset();
+             for(Int_t b=1; b<hist->GetXaxis()->GetNbins()-1; b++) {
+               Double_t xvalue = hist->GetBinCenter(b);
+               Int_t bin = h3tmp->FindBin(i,j,xvalue);
+               Double_t value =  h2tmp->GetBinContent(bin);
+               //printf("v = %f\n", value);
+               hist->SetBinContent(b, value);
+             }
+             if (hist->GetSum() < 100) continue;
+             //printf("fitting %d %d %f\n", i, j, hist->GetSum());           
+             hist->Fit("landau", "q0", "goff", 10, 180);
+             TF1 *fit = hist->GetFunction("landau");
+             //
+             Int_t sm = i/18;
+             Int_t det = i%18;
+             TH2* h2 = (TH2*)arrRP[14+sm];
+             if (!h2) continie;
+             Int_t bin = h2->FindBin(det,j);
+             // printf("%d %d %d\n", det, j, bin);
+             h2->SetBinContent(bin, fit->GetParameter(1));
+           }
+         }
+       } // h3tmp
+      } // RESPOINTS
+    } // RS: loop over eventual clones per trigger class
+  } // loop over species
+  
+  
+  delete hist;
   
+  // call the checker
+  AliQAChecker::Instance()->Run(AliQAv1::kTRD, task, list) ;    
+
+  //watch.Stop();
+  //watch.Print();
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitESDs()
 {
-  //create ESDs histograms in ESDs subdir
-  const Int_t nhist = 19;
-  TH1 *hist[nhist];
-  hist[0] = new TH1D("qaTRD_esd_ntracks", ":Number of tracks", 300, -0.5, 299.5);
-  hist[1] = new TH1D("qaTRD_esd_sector", ":Sector", 18, -0.5, 17.7);
-  hist[2] = new TH1D("qaTRD_esd_bits", ";Bits", 64, -0.5, 63.5);
+  //
+  // Create ESDs histograms in ESDs subdir
+  //
+
+  const Int_t kNhist = 19;
+  TH1 *hist[kNhist];
+  Int_t histoCounter = -1 ;
+
+  hist[++histoCounter] = new TH1F("qaTRD_esd_ntracks", ":Number of tracks", 300, -0.5, 299.5);
+  hist[++histoCounter] = new TH1F("qaTRD_esd_sector", ":Sector", 18, -0.5, 17.7);
+  hist[++histoCounter] = new TH1F("qaTRD_esd_bits", ";Bits", 64, -0.5, 63.5);
 
   const Int_t knbits = 6;
   const char *suf[knbits] = {"TPCi", "TPCo", "TPCz", "TRDo", "TRDr", "TRDz"};
-  
-  // 3
+
   for(Int_t i=0; i<knbits; i++) {
-    hist[i*knbits+3] = new TH1D(Form("qaTRD_esd_pt%s",suf[i]), ";p_{T} (GeV/c);", 50, 0, 10);
-    hist[i+knbits+4] = new TH1D(Form("qaTRD_esd_trdz%s", suf[i]), ";z (cm)", 200, -400, 400); 
+    hist[++histoCounter] = new TH1F(Form("qaTRD_esd_pt%s",suf[i]), ";p_{T} (GeV/c);", 50, 0, 10);
+    hist[++histoCounter] = new TH1F(Form("qaTRD_esd_trdz%s", suf[i]), ";z (cm)", 200, -400, 400); 
   }
-  
-  // 3 + 12 = 15
-  hist[15] = new TH1D("qaTRD_esd_clsTRDo", "TRDo;number of clusters", 130, -0.5, 129.5);;
-  hist[16] = new TH1D("qaTRD_esd_clsTRDr", "TRDr;number of clusters", 130, -0.5, 129.5);;
-  hist[17] = new TH1D("qaTRD_esd_clsTRDz", "TRDz;number of clusters", 130, -0.5, 129.5);;
-  //  hist[18] = new TH1D("qaTRD_esd_clsRatio", ";cluster ratio", 100, 0., 1.3);;
 
-  hist[18] = new TH2D("qaTRD_esd_sigMom", ";momentum (GeV/c);signal", 100, 0, 5, 200, 0, 1e3);
+  hist[++histoCounter] = new TH1F("qaTRD_esd_clsTRDo", "TRDo;number of clusters", 130, -0.5, 129.5);;
+  hist[++histoCounter] = new TH1F("qaTRD_esd_clsTRDr", "TRDr;number of clusters", 130, -0.5, 129.5);;
+  hist[++histoCounter] = new TH1F("qaTRD_esd_clsTRDz", "TRDz;number of clusters", 130, -0.5, 129.5);;
+  //hist[++histoCounter] = new TH1F("qaTRD_esd_clsRatio", ";cluster ratio", 100, 0., 1.3);;
 
-  for(Int_t i=0; i<nhist; i++) {
+  hist[++histoCounter] = new TH2F("qaTRD_esd_sigMom", ";momentum (GeV/c);signal", 100, 0, 5, 200, 0, 1e3);
+
+  for(Int_t i=0; i<=histoCounter; i++) {
     //hist[i]->Sumw2();
     Add2ESDsList(hist[i], i);
   }
+  //
+  ClonePerTrigClass(AliQAv1::kESDS); // this should be the last line
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitHits()
 {
-  // create Hits histograms in Hits subdir
-  const Int_t nhist = 4;
-  TH1D *hist[nhist];
-  
-  hist[0] = new TH1D("qaTRD_hits_det", ";Detector Id of the hit", 540, -0.5, 539.5) ; 
-  
-  hist[1] = new TH1D("qaTRD_hist_Qdrift", ";Charge from tracks", 100, 0, 100);
-  hist[2] = new TH1D("qaTRD_hist_Qamp", ";Charge from TRD photon", 100, 0, 100);
-  hist[3] = new TH1D("qaTRD_hist_Qphoton", ";Charge from TRD photon", 100, 0, 100);
+  //
+  // Create Hits histograms in Hits subdir
+  //
+
+  const Int_t kNhist = 4;
+  TH1F *hist[kNhist];
+
+  hist[0] = new TH1F("qaTRD_hits_det", ";Detector Id of the hit", 540, -0.5, 539.5) ; 
 
-  for(Int_t i=0; i<nhist; i++) {
+  hist[1] = new TH1F("qaTRD_hist_Qdrift", ";Charge from tracks", 100, 0, 100);
+  hist[2] = new TH1F("qaTRD_hist_Qamp", ";Charge from TRD photon", 100, 0, 100);
+  hist[3] = new TH1F("qaTRD_hist_Qphoton", ";Charge from TRD photon", 100, 0, 100);
+
+  for(Int_t i=0; i<kNhist; i++) {
     //hist[i]->Sumw2();
     Add2HitsList(hist[i], i);
   }
+  //
+  ClonePerTrigClass(AliQAv1::kHITS); // this should be the last line
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitDigits()
 {
-  // create Digits histograms in Digits subdir
-  
-  const Int_t nhist = 3;
-  TH1D *hist[nhist];
-  
-  hist[0] = new TH1D("qaTRD_digits_det", ";Detector Id of the digit", 540, -0.5, 539.5);
-  hist[1] = new TH1D("qaTRD_digits_time", ";Time bin", 40, -0.5, 39.5);
-  hist[2] = new TH1D("qaTRD_digits_amp", ";Amplitude", 100, 0, 100.);
+  //
+  // Create Digits histograms in Digits subdir
+  //
 
-  for(Int_t i=0; i<nhist; i++) {
+  const Int_t kNhist = 3;
+  TH1F *hist[kNhist];
+
+  hist[0] = new TH1F("qaTRD_digits_det", ";Detector Id of the digit", 540, -0.5, 539.5);
+  hist[1] = new TH1F("qaTRD_digits_time", ";Time bin", 40, -0.5, 39.5);
+  hist[2] = new TH1F("qaTRD_digits_amp", ";Amplitude", 100, 0, 100.);
+
+  for(Int_t i=0; i<kNhist; i++) {
     hist[i]->Sumw2();
     Add2DigitsList(hist[i], i);
   }
-
+  //
+  ClonePerTrigClass(AliQAv1::kDIGITS); // this should be the last line
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitRecPoints()
 {
-  // create Reconstructed Points histograms in RecPoints subdir
-  const Int_t nhist = 12;
-  TH1 *hist[nhist];
+  //
+  // Create Reconstructed Points histograms in RecPoints subdir
+  //
 
-  hist[0] = new TH1D("qaTRD_recPoints_det", ";Detector ID of the cluster", 540, -0.5, 539.5);
-  hist[1] = new TH2D("qaTRD_recPoints_amp", ";Amplitude", 540, -0.5, 539, 200, -0.5, 199.5);
-  hist[2] = new TH1D("qaTRD_recPoints_npad", ";Number of Pads", 12, -0.5, 11.5);
-  hist[3] = new TH1D("qaTRD_recPoints_dist2", ";residuals [2pad]", 100, -1, 1);
-  hist[4] = new TH1D("qaTRD_recPoints_dist3", ";residuals [3pad]", 100, -1, 1);
-  hist[5] = new TH1D("qaTRD_recPoints_dist4", ";residuals [4pad]", 100, -1, 1);
-  hist[6] = new TH1D("qaTRD_recPoints_dist5", ";residuals [5pad]", 100, -1, 1);
+  const Int_t kNhist = 14 + 18;
+  TH1 *hist[kNhist];
 
-  hist[7] = new TH2D("qaTRD_recPoints_rowCol", ";row;col", 16, -0.5, 15.5, 145, -0.5, 144.5);
-  hist[8] = new TH1D("qaTRD_recPoints_time", ";time bin", 35, -0.5, 34.5);
-  hist[9] = new TH1D("qaTRD_recPoints_nCls", ";number of clusters", 500, -0.5, 499.5);
+  hist[0] = new TH1F("qaTRD_recPoints_det", ";Detector ID of the cluster", 540, -0.5, 539.5);
+  hist[1] = new TH2F("qaTRD_recPoints_amp", ";Amplitude", 540, -0.5, 539, 200, -0.5, 199.5);
+  hist[2] = new TH1F("qaTRD_recPoints_npad", ";Number of Pads", 12, -0.5, 11.5);
 
-  hist[10] = new TProfile("qaTRD_recPoints_sigTime", ";time bin;signal", 35, -0.5, 34.5, 0, 200, "");
-  hist[11] = new TProfile("qaTRD_recPoints_prf", ";distance;center of gravity", 120, -0.6, 0.6, -1.2, 1.2, "");
+  hist[3] = new TH1F("qaTRD_recPoints_dist2", ";residuals [2pad]", 100, -1, 1);
+  hist[4] = new TH1F("qaTRD_recPoints_dist3", ";residuals [3pad]", 100, -1, 1);
+  hist[5] = new TH1F("qaTRD_recPoints_dist4", ";residuals [4pad]", 100, -1, 1);
+  hist[6] = new TH1F("qaTRD_recPoints_dist5", ";residuals [5pad]", 100, -1, 1);
 
-  hist[12] = new TH1D("qaTRD_recPoints_ampDist", ";amplitude MPV", 100, 0, 100);
+  hist[7] = new TH2F("qaTRD_recPoints_rowCol", ";row;col", 16, -0.5, 15.5, 145, -0.5, 144.5);
+  hist[8] = new TH1F("qaTRD_recPoints_time", ";time bin", 35, -0.5, 34.5);
+  hist[9] = new TH1F("qaTRD_recPoints_nCls", ";number of clusters", 500, -0.5, 499.5);
 
+  hist[10] = new TH3F("qaTRD_recPoints_sigTime", ";chamber;time bin;signal", 
+                     540, -0.5, 539.5, 35, -0.5, 34.5, 200, 0.5, 199.5);
+  hist[11] = new TProfile("qaTRD_recPoints_prf", ";distance;center of gravity"
+                         , 120, -0.6, 0.6, -1.2, 1.2, "");
+
+  hist[12] = new TH1F("qaTRD_recPoints_ampMPV", ";amplitude MPV", 100, 0, 100);
+  hist[13] = new TH1F("qaTRD_recPoints_ampSigma", ";amplitude Sigma", 100, 0, 100); 
+
+  for(Int_t i=0; i<18; i++) {
+    hist[14+i] = new TH2F(Form("qaTRD_recPoints_sigTime_sm%d",i), Form("sm%d;det;time bin"), 
+                       30, -0.5, 29.5, 35, -0.5, 34.5);
+    hist[14+i]->SetMinimum(20);
+    hist[14+i]->SetMaximum(40);
+  }
 
-  for(Int_t i=0; i<nhist; i++) {
+  for(Int_t i=0; i<kNhist; i++) {
     //hist[i]->Sumw2();
     Add2RecPointsList(hist[i], i);
   }
+  //
+  ClonePerTrigClass(AliQAv1::kRECPOINTS); // this should be the last line
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitRaws()
 {
+  //
   // create Raws histograms in Raws subdir
+  //
+
   const Int_t kSM = 18;
-  const Int_t nhist = 6+kSM;
-  TH1D *hist[nhist];
-  hist[0] = new TH1D("qaTRD_raws_det", ";detector", 540, -0.5, 539.5);
-  hist[1] = new TH1D("qaTRD_raws_sig", ";signal", 100, -0.5, 99.5);
-  hist[2] = new TH1D("qaTRD_raws_sigCentral", "; signal central bin", 100, -0.5, 99.5);
-  hist[3] = new TH1D("qaTRD_raws_sigTail", ";signal cluster", 100, -0.5, 99.5);
-  hist[4] = new TH1D("qaTRD_raws_tiemBin", ";time bin", 40, -0.5, 39.5); 
-  hist[5] = new TH1D("qaTRD_rows_smId", ";supermodule", 18, -0.5, 17.5);
-
-  // one char per ADC chanell
-  const Int_t nADC = 30 * 8 * 16 * 22;
+  //const Int_t kNCh = 540;
+  const Int_t kNhist = 4+kSM;
+  TH1F *hist[kNhist];
+
+  // four histograms to be published
+  hist[0] = new TH1F("qaTRD_raws_det", ";detector", 540, -0.5, 539.5);
+  hist[1] = new TH1F("qaTRD_raws_sig", ";signal", 100, -0.5, 99.5);
+  hist[2] = new TH1F("qaTRD_raws_timeBin", ";time bin", 40, -0.5, 39.5); 
+  hist[3] = new TH1F("qaTRD_raws_smId", ";supermodule", 18, -0.5, 17.5);
+  //
+  
+  // one double per MCM (not published)
+  const Int_t kNMCM = 30 * 8 * 16;
   for(Int_t i=0; i<kSM; i++)
-    hist[6+i] = new TH1D(Form("qaTRD_raws_sm%d",i),"",nADC, -0.5, nADC-0.5); 
-
-  for(Int_t i=0; i<nhist; i++) {
+    hist[4+i] = new TH1F(Form("qaTRD_raws_sm%d",i),"",kNMCM, -0.5, kNMCM-0.5); 
+  
+  // register
+  for(Int_t i=0; i<kNhist; i++) {
     //hist[i]->Sumw2();
     Add2RawsList(hist[i], i);
   }
+  //
+  ClonePerTrigClass(AliQAv1::kRAWS); // this should be the last line
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitSDigits()
 {
-  // create SDigits histograms in SDigits subdir
-  
-  const Int_t nhist = 3;
-  TH1D *hist[nhist];
-  
-  hist[0] = new TH1D("qaTRD_sdigits_det", ";Detector Id of the digit", 540, -0.5, 539.5);
-  hist[1] = new TH1D("qaTRD_sdigits_time", ";Time bin", 40, -0.5, 39.5);
-  hist[2] = new TH1D("qaTRD_sdigits_amp", ";Amplitude", 100, 0, 1e7);
+  //
+  // Create SDigits histograms in SDigits subdir
+  //
 
-  for(Int_t i=0; i<nhist; i++) {
+  const Int_t kNhist = 3;
+  TH1F *hist[kNhist];
+
+  hist[0] = new TH1F("qaTRD_sdigits_det", ";Detector Id of the digit", 540, -0.5, 539.5);
+  hist[1] = new TH1F("qaTRD_sdigits_time", ";Time bin", 40, -0.5, 39.5);
+  hist[2] = new TH1F("qaTRD_sdigits_amp", ";Amplitude", 100, 0, 1e7);
+
+  for(Int_t i=0; i<kNhist; i++) {
     hist[i]->Sumw2();
     Add2SDigitsList(hist[i], i);
   }
+  //
+  ClonePerTrigClass(AliQAv1::kSDIGITS); // this should be the last line
 }
 
 //____________________________________________________________________________
-void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
+void AliTRDQADataMaker::MakeESDs(AliESDEvent * const esd)
 {
-  // make QA data from ESDs
+  //
+  // Make QA data from ESDs
+  //
+
   Int_t nTracks = esd->GetNumberOfTracks();
-  GetESDsData(0)->Fill(nTracks);
-  
+  FillESDsData(0,nTracks);
+
   // track loop
-  for(Int_t i=0; i<nTracks; i++) {
-    
+  for (Int_t i=0; i<nTracks; i++) {
+
     AliESDtrack *track = esd->GetTrack(i);
     const AliExternalTrackParam *paramOut = track->GetOuterParam();
     const AliExternalTrackParam *paramIn = track->GetInnerParam();
@@ -252,12 +382,11 @@ void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
 
     // .. in the acceptance
     Int_t sector = GetSector(paramOut->GetAlpha());
-    GetESDsData(1)->Fill(sector);
 
     UInt_t u = 1;
     UInt_t status = track->GetStatus();
     for(Int_t bit=0; bit<32; bit++) 
-      if (u<<bit & status) GetESDsData(2)->Fill(bit);
+      if (u<<bit & status) FillESDsData(2,bit);
 
     const Int_t knbits = 6; 
     Int_t bit[6] = {0,0,0,0,0,0};    
@@ -274,14 +403,14 @@ void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
 
     for(Int_t b=0; b<knbits; b++) {
       if (bit[b]) {
-       GetESDsData(b*knbits+3)->Fill(pt); 
-       GetESDsData(b*knbits+4)->Fill(extZ);
+       FillESDsData(2*b+3,pt); 
+       FillESDsData(2*b+4,extZ);
       }
     }
 
     // clusters
     for(Int_t b=0; b<3; b++) 
-      if (bit[3+b]) GetESDsData(b+15)->Fill(track->GetTRDncls());
+      if (bit[3+b]) FillESDsData(b+15,track->GetTRDncls());
 
     // refitted only
     if (!bit[4]) continue;
@@ -290,12 +419,13 @@ void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
     //fBudget->Fill(track->GetTRDBudget());
     //fSignal->Fill(track->GetTRDsignal());
        
-    GetESDsData(18)->Fill(track->GetP(), track->GetTRDsignal());
+    FillESDsData(18,track->GetP(), track->GetTRDsignal());
+    FillESDsData(1,sector);
 
     /*
     // PID only
     if (status & AliESDtrack::kTRDpid) {
-      
+
       for(Int_t l=0; l<6; l++) fTime->Fill(track->GetTRDTimBin(l));
 
       // fill pid histograms
@@ -317,64 +447,72 @@ void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
        if (track->GetTRDpid(pid) > kminPidValue) trdBestPid = pid;
        //if (pp[pid] > kminPidValue) tpcBestPid = pid;
       }
-      
+
       fTrdPID[5]->Fill(trdr0); // check unitarity
       fTrdSigMomPID[trdBestPid]->Fill(track->GetP(), track->GetTRDsignal());
-      
+
       //fTpcPID[5]->Fill(tpcr0); // check unitarity
       //fTpcSigMomPID[tpcBestPid]->Fill(track->GetP(), track->GetTPCsignal());
     }
     */
-    
-  }
-
-
 
+  }
+  //
+  IncEvCountCycleESDs();
+  IncEvCountTotalESDs();
+  //
 }
 
 //______________________________________________________________________________
-Int_t AliTRDQADataMaker::GetSector(const Double_t alpha) const {
+Int_t AliTRDQADataMaker::GetSector(const Double_t alpha) const 
+{
+  //
   // Gets the sector number 
+  //
 
   Double_t size = TMath::DegToRad() * 20.; // shall use TRDgeo
   Int_t sector = (Int_t)((alpha + TMath::Pi())/size);
   return sector;
+
 }
 
 //______________________________________________________________________________
-Double_t AliTRDQADataMaker::GetExtZ(const AliExternalTrackParam *in) const {
+Double_t AliTRDQADataMaker::GetExtZ(const AliExternalTrackParam *in) const 
+{
   //
-  // returns the Z position at the entry to TRD
+  // Returns the Z position at the entry to TRD
   // using parameters from the TPC in
   //
 
-  const Double_t x0 = 300;
+  const Double_t kX0 = 300;
 
   Double_t x = in->GetX();
   const Double_t *par = in->GetParameter();
   Double_t theta = par[3];
   Double_t z = in->GetZ();
-  
-  Double_t zz = z + (x0-x) * TMath::Tan(theta);
+
+  Double_t zz = z + (kX0-x) * TMath::Tan(theta);
   return zz;
+
 }
 
 //____________________________________________________________________________
-void AliTRDQADataMaker::MakeHits(TClonesArray * hits)
+void AliTRDQADataMaker::MakeHits(TClonesArray * const hits)
 {
-  //make QA data from Hits
-  //printf("making QA for TRD hits from an array %d\n", hits->GetEntriesFast());
+  //
+  // Make QA data from Hits
+  //
 
   TIter next(hits); 
   AliTRDhit * hit; 
-  
+
   while ( (hit = dynamic_cast<AliTRDhit *>(next())) ) {
-    GetHitsData(0)->Fill(hit->GetDetector());
+    FillHitsData(0,hit->GetDetector());
     Double_t q = TMath::Abs(hit->GetCharge());
-    
-    if (hit->FromDrift()) GetHitsData(1)->Fill(q);
-    if (hit->FromAmplification()) GetHitsData(2)->Fill(q);
-    if (hit->FromTRphoton()) GetHitsData(3)->Fill(q);
+
+    if (hit->FromDrift()) FillHitsData(1,q);
+    if (hit->FromAmplification()) FillHitsData(2,q);
+    if (hit->FromTRphoton()) FillHitsData(3,q);
   }
 
 }
@@ -382,19 +520,20 @@ void AliTRDQADataMaker::MakeHits(TClonesArray * hits)
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeHits(TTree * hitTree)
 {
-  //make QA data from Hits
-  //printf("making QA for TRD hits from a tree\n");
-  
+  //
+  // Make QA data from Hits
+  //
+
   if (!CheckPointer(hitTree, "TRD hits tree")) return;
+
   TBranch *branch = hitTree->GetBranch("TRD");
   if (!CheckPointer(branch, "TRD hits branch")) return;
+
   Int_t nhits = (Int_t)(hitTree->GetTotBytes()/sizeof(AliTRDhit));
   TClonesArray *hits = new TClonesArray("AliTRDhit", nhits+1000);
   TClonesArray *tmp = new TClonesArray("AliTRDhit", 1000);
   branch->SetAddress(&tmp);
-  
+
   Int_t index = 0;
   Int_t nEntries = (Int_t)branch->GetEntries();
   for(Int_t i = 0; i < nEntries; i++) {
@@ -409,248 +548,284 @@ void AliTRDQADataMaker::MakeHits(TTree * hitTree)
   tmp->Delete();
   delete tmp;
   MakeHits(hits);
+  //
+  IncEvCountCycleHits();
+  IncEvCountTotalHits();
+  //
 }
 
 //____________________________________________________________________________
-void AliTRDQADataMaker::MakeDigits(TClonesArray * digits)
+void AliTRDQADataMaker::MakeDigits(TClonesArray * const digits)
 {
-  // makes data from Digits
-  
+  //
+  // Makes data from Digits
+  //
+
   TIter next(digits) ; 
   AliTRDdigit * digit ; 
   while ( (digit = dynamic_cast<AliTRDdigit *>(next())) ) {
-    GetDigitsData(0)->Fill(digit->GetDetector());
-    GetDigitsData(1)->Fill(digit->GetTime());
-    GetDigitsData(2)->Fill(digit->GetAmp());
-  }  
+    FillDigitsData(0,digit->GetDetector());
+    FillDigitsData(1,digit->GetTime());
+    FillDigitsData(2,digit->GetAmp());
+  }
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeDigits(TTree * digits)
 {
+  //
+  // Makes data from digits tree
+  //
 
   AliTRDdigitsManager *digitsManager = new AliTRDdigitsManager();
   digitsManager->CreateArrays();
   digitsManager->ReadDigits(digits);
 
   for (Int_t i = 0; i < AliTRDgeometry::kNdet; i++) {
-    
-    AliTRDdataArrayI *digitsIn = digitsManager->GetDigits(i);      
-    
+
+    AliTRDarrayADC *digitsIn = (AliTRDarrayADC *) digitsManager->GetDigits(i);      
+
     // This is to take care of switched off super modules
     if (digitsIn->GetNtime() == 0) continue;
-    
-    digitsIn->Expand();
-    
+
+    digitsIn->Expand(); 
+
     //AliTRDSignalIndex* indexes = digitsManager->GetIndexes(i);
     //if (indexes->IsAllocated() == kFALSE) digitsManager->BuildIndexes(i);
-    
+
     Int_t nRows = digitsIn->GetNrow();
     Int_t nCols = digitsIn->GetNcol();
     Int_t nTbins = digitsIn->GetNtime();
 
     for(Int_t row = 0; row < nRows; row++) 
       for(Int_t col = 0; col < nCols; col++) 
-       for(Int_t time = 0; time < nTbins; time++) {
+       for(Int_t time = 0; time < nTbins; time++) 
+         {
+           Float_t signal = digitsIn->GetData(row,col,time);
+           FillDigitsData(0,i);
+           FillDigitsData(1,time);
+           FillDigitsData(2,signal);
+         }
 
-         Float_t signal = digitsIn->GetDataUnchecked(row,col,time);
-         GetDigitsData(0)->Fill(i);
-         GetDigitsData(1)->Fill(time);
-         GetDigitsData(2)->Fill(signal);
-       }
-    
     //delete digitsIn;
   }
+
   delete digitsManager;
+  //
+  IncEvCountCycleDigits();
+  IncEvCountTotalDigits();
+  //
 }
 
 //____________________________________________________________________________
-void AliTRDQADataMaker::MakeSDigits(TClonesArray * sdigits)
+void AliTRDQADataMaker::MakeSDigits(TClonesArray * const sdigits)
 {
-  // makes data from Digits
-  
+  //
+  // Makes data from Digits
+  //
+
   TIter next(sdigits) ; 
   AliTRDdigit * digit ; 
   while ( (digit = dynamic_cast<AliTRDdigit *>(next())) ) {
-    GetDigitsData(0)->Fill(digit->GetDetector());
-    GetDigitsData(1)->Fill(digit->GetTime());
-    GetDigitsData(2)->Fill(digit->GetAmp());
-  }  
+    FillDigitsData(0,digit->GetDetector());
+    FillDigitsData(1,digit->GetTime());
+    FillDigitsData(2,digit->GetAmp());
+  }
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeSDigits(TTree * digits)
 {
+  //
+  // Makes data from SDigits
+  //
 
   AliTRDdigitsManager *digitsManager = new AliTRDdigitsManager();
   digitsManager->CreateArrays();
   digitsManager->ReadDigits(digits);
 
-  for (Int_t i = 0; i < AliTRDgeometry::kNdet; i++) {
-    
-    AliTRDdataArrayI *digitsIn = digitsManager->GetDigits(i);      
-    
+  for (Int_t i = 0; i < AliTRDgeometry::kNdet; i++) 
+    {
+    AliTRDarrayADC *digitsIn = (AliTRDarrayADC *) digitsManager->GetDigits(i);      
+
     // This is to take care of switched off super modules
     if (digitsIn->GetNtime() == 0) continue;
-    
-    digitsIn->Expand();
-    
+
+    digitsIn->Expand();  
+
     //AliTRDSignalIndex* indexes = digitsManager->GetIndexes(i);
     //if (indexes->IsAllocated() == kFALSE) digitsManager->BuildIndexes(i);
-    
+
     Int_t nRows = digitsIn->GetNrow();
     Int_t nCols = digitsIn->GetNcol();
     Int_t nTbins = digitsIn->GetNtime();
 
     for(Int_t row = 0; row < nRows; row++) 
       for(Int_t col = 0; col < nCols; col++) 
-       for(Int_t time = 0; time < nTbins; time++) {
-
-         Float_t signal = digitsIn->GetDataUnchecked(row,col,time);
-         if (signal <= 0) continue;
-         GetSDigitsData(0)->Fill(i);
-         GetSDigitsData(1)->Fill(time);
-         GetSDigitsData(2)->Fill(signal);
-       }
+       for(Int_t time = 0; time < nTbins; time++) 
+         {
+           Float_t signal = digitsIn->GetData(row,col,time);
+           if (signal <= 0) continue;
+           FillSDigitsData(0,i);
+           FillSDigitsData(1,time);
+           FillSDigitsData(2,signal);
+         }
     
     // delete digitsIn;
   }
 
   delete digitsManager;
+  //
+  IncEvCountCycleSDigits();
+  IncEvCountTotalSDigits();
+  //
 }
 
 //____________________________________________________________________________
-void AliTRDQADataMaker::MakeRaws(AliRawReader* rawReader)
+void AliTRDQADataMaker::MakeRaws(AliRawReader * const rawReader)
 {
+  //
+  // Makes QA data from raw data
+  //
+
   // 157
   // T9 -- T10
 
-  //const Int_t kSM  = 18;
+  //const Int_t kSM = 18;
   //const Int_t kROC = 30;
+  const Int_t kROB = 8;
   //const Int_t kLayer = 6;
   //const Int_t kStack = 5;
-  const Int_t kROB = 8;
   const Int_t kMCM = 16;
-  const Int_t kADC = 22;
+  //  const Int_t kADC = 22;
+
+  AliTRDrawStream raw(rawReader);
+  raw.SetRawVersion(3);
+  raw.Init();
+
+  while (raw.Next()) {
+
+    FillRawsData(0,raw.GetDet());
+
+    // possibly needs changes with the new reader !!
+    Int_t *sig = raw.GetSignals();
+    for(Int_t i=0; i<3; i++) FillRawsData(1,sig[i]);
+    // ---
+
+    FillRawsData(2,raw.GetTimeBin());
 
-  AliTRDRawStreamV2 *raw = new AliTRDRawStreamV2(rawReader);
-  
-  raw->SetRawVersion(3);
-  raw->Init();
-  
-  while (raw->Next()) {
-    
-    GetRawsData(0)->Fill(raw->GetDet());
-    
-    Int_t *sig = raw->GetSignals();
-    for(Int_t i=0; i<3; i++) GetRawsData(1)->Fill(sig[i]);
-    
-    GetRawsData(2)->Fill(sig[1]);
-    GetRawsData(3)->Fill(sig[0]);
-    GetRawsData(3)->Fill(sig[2]);
-    
-    GetRawsData(4)->Fill(raw->GetTimeBin());
-    
     // calculate the index;
-    Int_t sm = raw->GetSM();
-    Int_t roc = raw->GetROC();
-    Int_t rob = raw->GetROB();
-    Int_t mcm = raw->GetMCM();
-    Int_t adc = raw->GetADC();
-    
-    Int_t index = roc * (kROB*kMCM*kADC) + rob * (kMCM*kADC) + mcm * kADC + adc;
-    GetRawsData(5)->Fill(sm);
-    //printf("SM = %d\n", sm);
-    GetRawsData(6+sm)->Fill(index);
+    Int_t sm = raw.GetSM();
+    Int_t roc = raw.GetROC();
+    Int_t rob = raw.GetROB();
+    Int_t mcm = raw.GetMCM();
+    //Int_t adc = raw.GetADC();
+
+    //Int_t index = roc * (kROB*kMCM*kADC) + rob * (kMCM*kADC) + mcm * kADC + adc;
+    Int_t  index = roc * (kROB*kMCM) + rob * kMCM + mcm;
+    FillRawsData(3,sm);
+    FillRawsData(4+sm,index);
   }
+  //
+  IncEvCountCycleRaws();
+  IncEvCountTotalRaws();
+  //
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeRecPoints(TTree * clustersTree)
 {
-  
-  // makes data from RecPoints
+  //  
+  // Makes data from RecPoints
   // 
-  
+
   Int_t nsize = Int_t(clustersTree->GetTotBytes() / (sizeof(AliTRDcluster))); 
   TObjArray *clusterArray = new TObjArray(nsize+1000); 
-  
+
   TBranch *branch = clustersTree->GetBranch("TRDcluster");
   if (!branch) {
     AliError("Can't get the branch !");
     return;
   }
   branch->SetAddress(&clusterArray); 
-  
+
   // Loop through all entries in the tree
   Int_t nEntries   = (Int_t) clustersTree->GetEntries();
   Int_t nbytes     = 0;
   AliTRDcluster *c = 0;
   Int_t nDet[540];
   for (Int_t i=0; i<540; i++) nDet[i] = 0;
-  
+
   for (Int_t iEntry = 0; iEntry < nEntries; iEntry++) {    
-    
+
     // Import the tree
     nbytes += clustersTree->GetEvent(iEntry);  
-    
+
     // Get the number of points in the detector
     Int_t nCluster = clusterArray->GetEntriesFast();  
-    
+
     // Loop through all TRD digits
     for (Int_t iCluster = 0; iCluster < nCluster; iCluster++) { 
       c = (AliTRDcluster *) clusterArray->UncheckedAt(iCluster);
-      
+
       Int_t iDet = c->GetDetector();
       nDet[iDet]++;
-      GetRecPointsData(0)->Fill(iDet);
-      GetRecPointsData(1)->Fill(iDet, c->GetQ());
-      GetRecPointsData(2)->Fill(c->GetNPads());
+      FillRecPointsData(0,iDet);
+      FillRecPointsData(1,iDet, c->GetQ());
+      FillRecPointsData(2,c->GetNPads());
       if (c->GetNPads() < 6)
-       GetRecPointsData(1+c->GetNPads())->Fill(c->GetCenter());
+       FillRecPointsData(1+c->GetNPads(),c->GetCenter());
 
       //if (c->GetPadTime() < 5)
-      ((TH2D*)GetRecPointsData(7))->Fill(c->GetPadRow(), c->GetPadCol());
-      GetRecPointsData(8)->Fill(c->GetPadTime());
-      
-      ((TProfile*)GetRecPointsData(10))->Fill(c->GetPadTime(), c->GetQ());
-      
+      FillRecPointsData(7,c->GetPadRow(), c->GetPadCol());
+      FillRecPointsData(8,c->GetPadTime());
+
+      TObjArray *hists = GetMatchingRecPointsData(10); //RS no alias for 3d histo filling, to directly
+      for (int ih=hists->GetEntriesFast();ih--;) ((TH3F*)hists->UncheckedAt(ih))->Fill(iDet, c->GetPadTime(), c->GetQ());
+
       // PRF for 2pad
       //if (c->GetNPads() == 2) {
       Short_t *sig = c->GetSignals();
       Double_t frac = -10;
-      
+
       if (sig[0] == 0 && sig[1] == 0 && sig[2] == 0 && sig[5] == 0 && sig[6] == 0) 
        frac = 1. * sig[4] / (sig[3] + sig[4]);
-      
-      
+
       if (sig[0] == 0 && sig[1] == 0 && sig[4] == 0 && sig[5] == 0 && sig[6] == 0)
        frac = -1. * sig[2] / (sig[2] + sig[3]);
-      
-      if (frac > -10)  ((TProfile*)GetRecPointsData(11))->Fill(c->GetCenter(), frac);
+
+      if (frac > -10)  FillRecPointsData(11,c->GetCenter(), frac);
        
       //}
     }
   }
-  
-  for(Int_t i=0; i<540; i++) 
-    if (nDet[i] > 0) GetRecPointsData(9)->Fill(nDet[i]);
 
+  for(Int_t i=0; i<540; i++) 
+    if (nDet[i] > 0) FillRecPointsData(9,nDet[i]);
 
   delete clusterArray;
+
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::StartOfDetectorCycle()
 {
-  //Detector specific actions at start of cycle
+  //
+  // Detector specific actions at start of cycle
+  //
 
 }
 
 //__________________________________________________________________________
-Int_t AliTRDQADataMaker::CheckPointer(TObject *obj, const char *name) {
+Int_t AliTRDQADataMaker::CheckPointer(TObject * const obj, const char *name) 
+{
+  //
+  // Checks initialization of pointers
+  //
 
   if (!obj) AliWarning(Form("null pointer: %s", name));
   return !!obj;
+
 }
-//__________________________________________________________________________