]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - TRD/AliTRDQADataMaker.cxx
Radical performance improvement
[u/mrichter/AliRoot.git] / TRD / AliTRDQADataMaker.cxx
index 1cd3984a473bf96a5955e1977699c1e4ba30a8ab..e102bd9a70a42108e99f32a4d452e1442bd7aaf1 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)               //
 //                                                                        //
 ////////////////////////////////////////////////////////////////////////////
 
 #include <TFile.h> 
 #include <TH1D.h> 
 #include <TH2D.h>
+#include <TH3D.h>
 #include <TProfile.h>
+#include <TF1.h>
+#include <TCanvas.h>
+#include <TStopwatch.h>
 
 // --- AliRoot header files ---
 #include "AliESDEvent.h"
 #include "AliTRDdigitsManager.h"
 #include "AliTRDgeometry.h"
 #include "AliTRDdataArrayI.h"
+#include "AliTRDrawStreamTB.h"
 #include "AliTRDRawStreamV2.h"
 
+#include "AliQAChecker.h"
+
 ClassImp(AliTRDQADataMaker)
-           
+
 //____________________________________________________________________________ 
   AliTRDQADataMaker::AliTRDQADataMaker() : 
   AliQADataMaker(AliQA::GetDetName(AliQA::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(AliQA::TASKINDEX task, TObjArray * list)
 {
-  //Detector specific actions at end of cycle
+  //
+  // Detector specific actions at end of cycle
+  //
+  //TStopwatch watch;
+  //watch.Start();
+
+  //AliInfo(Form("EndOfCycle", "Fitting RecPoints %d", task))
+  TH1D *hist = new TH1D("fitHist", "", 200, -0.5, 199.5);
+  if (task == AliQA::kRECPOINTS) {
+
+    //list->Print();
+    
+    // Rec points full chambers
+    for (Int_t i=0; i<540; i++) {
+       
+      //TH1D *h = ((TH2D*)list->At(1))->ProjectionY(Form("qaTRD_recPoints_amp_%d",i), i+1, i+1);
+      hist->Reset();
+      for(Int_t b=1; b<hist->GetXaxis()->GetNbins()-1; b++) {
+       Double_t xvalue = hist->GetBinCenter(b);
+       Int_t bin = ((TH2D*)list->At(1))->FindBin(i,xvalue);
+       Double_t value =  ((TH2D*)list->At(1))->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");
+      ((TH1D*)list->At(12))->Fill(fit->GetParameter(1));
+      ((TH1D*)list->At(13))->Fill(fit->GetParameter(2));
+    }
+    // time-bin by time-bin
+    for (Int_t i=0; i<540; i++) {
+       
+      //TH1D *test = ((TH3D*)list->At(10))->ProjectionZ(Form("ampTime_%d",i), i+1, i+1, 0, 35);     
+      //if (test->GetSum() < 100) continue;
+      
+      //AliInfo(Form("fitting det = %d", i));
+      
+      for(Int_t j=0; j<35; j++) {
+       
+       //TH1D *h =  ((TH3D*)list->At(10))->ProjectionZ(Form("ampTime_%d",i), i+1, i+1, j+1, j+1);     
+       hist->Reset();
+       for(Int_t b=1; b<hist->GetXaxis()->GetNbins()-1; b++) {
+         Double_t xvalue = hist->GetBinCenter(b);
+         Int_t bin = ((TH3D*)list->At(10))->FindBin(i,j,xvalue);
+         Double_t value =  ((TH3D*)list->At(10))->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());
 
-  // Rec points
+       hist->Fit("landau", "q0", "goff", 10, 180);
+       TF1 *fit = hist->GetFunction("landau");
+       
+       Int_t sm = i/18;
+       Int_t det = i%18;
+       TH2D *h2 = (TH2D*)list->At(14+sm);
+       Int_t bin = h2->FindBin(det,j);
+       // printf("%d %d %d\n", det, j, bin);
+       h2->SetBinContent(bin, fit->GetParameter(1));
+      }
+    }
+  }
   
+  delete hist;
+  
+  // call the checker
+  AliQAChecker::Instance()->Run(AliQA::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 TH1D("qaTRD_esd_ntracks", ":Number of tracks", 300, -0.5, 299.5);
+  hist[++histoCounter] = new TH1D("qaTRD_esd_sector", ":Sector", 18, -0.5, 17.7);
+  hist[++histoCounter] = new TH1D("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[2*i+3] = new TH1D(Form("qaTRD_esd_pt%s",suf[i]), ";p_{T} (GeV/c);", 50, 0, 10);
-    hist[2*i+4] = new TH1D(Form("qaTRD_esd_trdz%s", suf[i]), ";z (cm)", 200, -400, 400); 
+    hist[++histoCounter] = new TH1D(Form("qaTRD_esd_pt%s",suf[i]), ";p_{T} (GeV/c);", 50, 0, 10);
+    hist[++histoCounter] = new TH1D(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 TH1D("qaTRD_esd_clsTRDo", "TRDo;number of clusters", 130, -0.5, 129.5);;
+  hist[++histoCounter] = new TH1D("qaTRD_esd_clsTRDr", "TRDr;number of clusters", 130, -0.5, 129.5);;
+  hist[++histoCounter] = new TH1D("qaTRD_esd_clsTRDz", "TRDz;number of clusters", 130, -0.5, 129.5);;
+  //hist[++histoCounter] = new TH1D("qaTRD_esd_clsRatio", ";cluster ratio", 100, 0., 1.3);;
+
+  hist[++histoCounter] = new TH2D("qaTRD_esd_sigMom", ";momentum (GeV/c);signal", 100, 0, 5, 200, 0, 1e3);
 
-  for(Int_t i=0; i<nhist; i++) {
+  for(Int_t i=0; i<=histoCounter; i++) {
     //hist[i]->Sumw2();
     Add2ESDsList(hist[i], i);
   }
+
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitHits()
 {
-  // create Hits histograms in Hits subdir
-  const Int_t nhist = 4;
-  TH1D *hist[nhist];
-  
+  //
+  // Create Hits histograms in Hits subdir
+  //
+
+  const Int_t kNhist = 4;
+  TH1D *hist[kNhist];
+
   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);
 
-  for(Int_t i=0; i<nhist; i++) {
+  for(Int_t i=0; i<kNhist; i++) {
     //hist[i]->Sumw2();
     Add2HitsList(hist[i], i);
   }
+
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitDigits()
 {
-  // create Digits histograms in Digits subdir
-  
-  const Int_t nhist = 3;
-  TH1D *hist[nhist];
-  
+  //
+  // Create Digits histograms in Digits subdir
+  //
+
+  const Int_t kNhist = 3;
+  TH1D *hist[kNhist];
+
   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.);
 
-  for(Int_t i=0; i<nhist; i++) {
+  for(Int_t i=0; i<kNhist; i++) {
     hist[i]->Sumw2();
     Add2DigitsList(hist[i], i);
   }
-
 }
 
 //____________________________________________________________________________ 
 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
+  //
+
+  const Int_t kNhist = 14 + 18;
+  TH1 *hist[kNhist];
 
   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);
@@ -170,72 +272,94 @@ void AliTRDQADataMaker::InitRecPoints()
   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[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[10] = new TH3D("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 TH1D("qaTRD_recPoints_ampDist", ";amplitude MPV", 100, 0, 100);
+  hist[12] = new TH1D("qaTRD_recPoints_ampMPV", ";amplitude MPV", 100, 0, 100);
+  hist[13] = new TH1D("qaTRD_recPoints_ampSigma", ";amplitude Sigma", 100, 0, 100); 
 
+  for(Int_t i=0; i<18; i++) {
+    hist[14+i] = new TH2D(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);
   }
+
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitRaws()
 {
+  //
   // create Raws histograms in Raws subdir
+  //
+
   const Int_t kSM = 18;
-  const Int_t nhist = 6+kSM;
-  TH1D *hist[nhist];
+  //const Int_t kNCh = 540;
+  const Int_t kNhist = 4+kSM;
+  TH1D *hist[kNhist];
+
+  // four histograms to be published
   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;
+  hist[2] = new TH1D("qaTRD_raws_timeBin", ";time bin", 40, -0.5, 39.5); 
+  hist[3] = new TH1D("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 TH1D(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);
   }
+
 }
 
 //____________________________________________________________________________ 
 void AliTRDQADataMaker::InitSDigits()
 {
-  // create SDigits histograms in SDigits subdir
-  
-  const Int_t nhist = 3;
-  TH1D *hist[nhist];
-  
+  //
+  // Create SDigits histograms in SDigits subdir
+  //
+
+  const Int_t kNhist = 3;
+  TH1D *hist[kNhist];
+
   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);
 
-  for(Int_t i=0; i<nhist; i++) {
+  for(Int_t i=0; i<kNhist; i++) {
     hist[i]->Sumw2();
     Add2SDigitsList(hist[i], i);
   }
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
 {
-  // make QA data from ESDs
+  //
+  // Make QA data from ESDs
+  //
+
   Int_t nTracks = esd->GetNumberOfTracks();
   GetESDsData(0)->Fill(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();
@@ -295,7 +419,7 @@ void AliTRDQADataMaker::MakeESDs(AliESDEvent * esd)
     /*
     // PID only
     if (status & AliESDtrack::kTRDpid) {
-      
+
       for(Int_t l=0; l<6; l++) fTime->Fill(track->GetTRDTimBin(l));
 
       // fill pid histograms
@@ -317,61 +441,66 @@ 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());
     }
     */
-    
-  }
-
 
+  }
 
 }
 
 //______________________________________________________________________________
-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)
 {
-  //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());
     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);
@@ -382,19 +511,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,42 +539,49 @@ void AliTRDQADataMaker::MakeHits(TTree * hitTree)
   tmp->Delete();
   delete tmp;
   MakeHits(hits);
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeDigits(TClonesArray * 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());
-  }  
+  }
+
 }
 
 //____________________________________________________________________________
 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);      
-    
+
     // This is to take care of switched off super modules
     if (digitsIn->GetNtime() == 0) continue;
-    
+
     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();
@@ -458,46 +595,54 @@ void AliTRDQADataMaker::MakeDigits(TTree * digits)
          GetDigitsData(1)->Fill(time);
          GetDigitsData(2)->Fill(signal);
        }
-    
+
     //delete digitsIn;
   }
+
   delete digitsManager;
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeSDigits(TClonesArray * 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());
-  }  
+  }
+
 }
 
 //____________________________________________________________________________
 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);      
-    
+
     // This is to take care of switched off super modules
     if (digitsIn->GetNtime() == 0) continue;
-    
+
     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();
@@ -512,95 +657,106 @@ void AliTRDQADataMaker::MakeSDigits(TTree * digits)
          GetSDigitsData(1)->Fill(time);
          GetSDigitsData(2)->Fill(signal);
        }
-    
+
     // delete digitsIn;
   }
 
   delete digitsManager;
+
 }
 
 //____________________________________________________________________________
 void AliTRDQADataMaker::MakeRaws(AliRawReader* 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;
 
+  //<<<<<<< AliTRDQADataMaker.cxx
+  //AliTRDrawStreamTB *raw = new AliTRDrawStreamTB(rawReader);
+  //raw->SetRawVersion(3);
+  //raw->Init();
+  //=======
   AliTRDRawStreamV2 *raw = new AliTRDRawStreamV2(rawReader);
-  
   raw->SetRawVersion(3);
   raw->Init();
-  
+  //>>>>>>> 1.12
+
   while (raw->Next()) {
-    
+
     GetRawsData(0)->Fill(raw->GetDet());
-    
+
+    // possibly needs changes with the new reader !!
     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());
-    
+    // ---
+
+    GetRawsData(2)->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 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;
+    GetRawsData(3)->Fill(sm);
+    GetRawsData(4+sm)->Fill(index);
   }
+
+  delete raw;
+
 }
 
 //____________________________________________________________________________
 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);
@@ -612,45 +768,50 @@ void AliTRDQADataMaker::MakeRecPoints(TTree * clustersTree)
       //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());
-      
+
+      ((TH3D*)GetRecPointsData(10))->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);
        
       //}
     }
   }
-  
+
   for(Int_t i=0; i<540; i++) 
     if (nDet[i] > 0) GetRecPointsData(9)->Fill(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 *obj, const char *name) 
+{
+  //
+  // Checks initialization of pointers
+  //
 
   if (!obj) AliWarning(Form("null pointer: %s", name));
   return !!obj;
+
 }
-//__________________________________________________________________________