coverity fixes + changed to full loop over second particle

diff --git a/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.cxx b/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.cxx
index 7c296e2d5d0fda1366516e13be640e5918baf6f6..68dedf04f0ab02234819fcf48d7741c9dff2657e 100644 (file)
--- a/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.cxx
+++ b/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.cxx
@@ -24,6 +24,7 @@
 #include <TMath.h>
 #include <TAxis.h>
 #include <TH1D.h>
 #include <TMath.h>
 #include <TAxis.h>
 #include <TH1D.h>

+#include <TH2D.h>

 
 
 #include <AliTHn.h>
 
 
 #include <AliTHn.h>


@@ -273,11 +274,10 @@ void AliBalanceTriggered::FillBalance(Float_t fCentrality,vector<Double_t> **cha
     if(charge > 0)  fHistP->Fill(trackVarsSingle,0,1.); 
     else            fHistN->Fill(trackVarsSingle,0,1.); 
 
     if(charge > 0)  fHistP->Fill(trackVarsSingle,0,1.); 
     else            fHistN->Fill(trackVarsSingle,0,1.); 
 

-    // 2nd particle loop
-    for(Int_t j = 0; j < i; j++) {
+    // 2nd particle loop (now over all particles except the same!)
+    for(Int_t j = 0; j < gNtrack; j++) {

 
 

-      // need check for single particle region!!!???
-      //
+      if( j == i ) continue;  

       
       Short_t charge2 = (Short_t) chargeVector[0]->at(j);
       trackVarsPair[0]    =  chargeVector[2]->at(i) - chargeVector[2]->at(j) ;  //delta eta
       
       Short_t charge2 = (Short_t) chargeVector[0]->at(j);
       trackVarsPair[0]    =  chargeVector[2]->at(i) - chargeVector[2]->at(j) ;  //delta eta


@@ -285,19 +285,19 @@ void AliBalanceTriggered::FillBalance(Float_t fCentrality,vector<Double_t> **cha
       trackVarsPair[2]    =  chargeVector[7]->at(j);  //pt
       trackVarsPair[3]    =  chargeVector[7]->at(i);  //pt trigger
       trackVarsPair[4]    =  fCentrality;             //centrality (really as variable here????)
       trackVarsPair[2]    =  chargeVector[7]->at(j);  //pt
       trackVarsPair[3]    =  chargeVector[7]->at(i);  //pt trigger
       trackVarsPair[4]    =  fCentrality;             //centrality (really as variable here????)

-
-    if( charge > 0 && charge2 < 0)  fHistPN->Fill(trackVarsPair,0,1.); 
-    else if( charge < 0 && charge2 > 0)  fHistNP->Fill(trackVarsPair,0,1.); 
-    else if( charge > 0 && charge2 > 0)  fHistPP->Fill(trackVarsPair,0,1.); 
-    else if( charge < 0 && charge2 < 0)  fHistNN->Fill(trackVarsPair,0,1.); 
-    else AliWarning("Wrong charge combination!");
-
+      
+      if( charge > 0 && charge2 < 0)  fHistPN->Fill(trackVarsPair,0,1.); 
+      else if( charge < 0 && charge2 > 0)  fHistNP->Fill(trackVarsPair,0,1.); 
+      else if( charge > 0 && charge2 > 0)  fHistPP->Fill(trackVarsPair,0,1.); 
+      else if( charge < 0 && charge2 < 0)  fHistNN->Fill(trackVarsPair,0,1.); 
+      else AliWarning("Wrong charge combination!");
+      

     }//end of 2nd particle loop
   }//end of 1st particle loop
 }  
 
 
     }//end of 2nd particle loop
   }//end of 1st particle loop
 }  
 
 

-TH1D* AliBalanceTriggered::GetBalanceFunctionHistogram1D(Int_t var, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){
+TH1D* AliBalanceTriggered::GetBalanceFunctionHistogram1D(Int_t var, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){

 
   // check which variable should be analyzed
   // 0 = Delta eta
 
   // check which variable should be analyzed
   // 0 = Delta eta


@@ -314,12 +314,12 @@ TH1D* AliBalanceTriggered::GetBalanceFunctionHistogram1D(Int_t var, Double_t pTM
   // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality
 
   // pT trigger
   // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality
 
   // pT trigger

-  fHistP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
-  fHistN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
-  fHistPN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
-  fHistNP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
-  fHistPP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
-  fHistNN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
+  fHistP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistPN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistNP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistPP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistNN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 

 
   // pT
   fHistPN->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
 
   // pT
   fHistPN->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 


@@ -344,27 +344,93 @@ TH1D* AliBalanceTriggered::GetBalanceFunctionHistogram1D(Int_t var, Double_t pTM
   TH1D* hTemp5 = (TH1D*)fHistP->Project(0,var);
   TH1D* hTemp6 = (TH1D*)fHistN->Project(0,var);
 
   TH1D* hTemp5 = (TH1D*)fHistP->Project(0,var);
   TH1D* hTemp6 = (TH1D*)fHistN->Project(0,var);
 

-  TH1D* gHistBalanceFunctionHistogram = (TH1D*)hTemp1->Clone();
-  gHistBalanceFunctionHistogram->Reset();
+  TH1D* gHistBalanceFunctionHistogram = NULL;
+
+  // Calculate BF
+  if((hTemp1)&&(hTemp2)&&(hTemp3)&&(hTemp4)&&(hTemp5)&&(hTemp6)) {
+    
+    gHistBalanceFunctionHistogram = (TH1D*)hTemp1->Clone();
+    gHistBalanceFunctionHistogram->Reset();
+
+    // Calculate BF
+    hTemp1->Add(hTemp3,-1.);
+    hTemp1->Scale(1./hTemp5->GetEntries());
+    hTemp2->Add(hTemp4,-1.);
+    hTemp2->Scale(1./hTemp6->GetEntries());
+    gHistBalanceFunctionHistogram->Add(hTemp1,hTemp2,1.,1.);
+  }
+
+  return gHistBalanceFunctionHistogram;
+}
+
+TH2D* AliBalanceTriggered::GetBalanceFunctionHistogram2D(Int_t var1, Int_t var2, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){
+
+  // check which variable should be analyzed
+  // 0 = Delta eta
+  // 1 = Delta phi
+
+  if( var1 < 0 || var1 > 1 || var2 < 0 || var2 > 1){
+    AliError("Only Variable 0 (= Delta eta) or 1 (= Delta phi) allowed");
+    return NULL;
+  }
+
+
+  // Choose region to analyze 
+  // for Single Histograms (P,N):       2 = pT,trigger; 3 = centrality
+  // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality
+
+  // pT trigger
+  fHistP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistPN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistNP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistPP->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+  fHistNN->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+
+  // pT
+  fHistPN->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
+  fHistNP->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
+  fHistPP->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
+  fHistNN->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
+
+  // centrality
+  fHistP->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(centrMin,centrMax); 
+  fHistN->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(centrMin,centrMax); 
+  fHistPN->GetGrid(0)->GetGrid()->GetAxis(4)->SetRange(centrMin,centrMax); 
+  fHistNP->GetGrid(0)->GetGrid()->GetAxis(4)->SetRange(centrMin,centrMax); 
+  fHistPP->GetGrid(0)->GetGrid()->GetAxis(4)->SetRange(centrMin,centrMax); 
+  fHistNN->GetGrid(0)->GetGrid()->GetAxis(4)->SetRange(centrMin,centrMax); 

   
   

+
+  // Project into the wanted space (1st: analysis step, 2nd: axis1, 3rd: axis2)
+  TH2D* hTemp1 = (TH2D*)fHistPN->Project(0,var1,var2);
+  TH2D* hTemp2 = (TH2D*)fHistNP->Project(0,var1,var2);
+  TH2D* hTemp3 = (TH2D*)fHistPP->Project(0,var1,var2);
+  TH2D* hTemp4 = (TH2D*)fHistNN->Project(0,var1,var2);
+  TH2D* hTemp5 = (TH2D*)fHistP->Project(0,var1,var2);
+  TH2D* hTemp6 = (TH2D*)fHistN->Project(0,var1,var2);
+
+  TH2D* gHistBalanceFunctionHistogram = NULL;
+

   // Calculate BF
   // Calculate BF

-  if((hTemp1)&&(hTemp2)&&(hTemp3)&&(hTemp4)) {
-    hTemp1->Sumw2();
-    hTemp2->Sumw2();
-    hTemp3->Sumw2();
-    hTemp4->Sumw2();
+  if((hTemp1)&&(hTemp2)&&(hTemp3)&&(hTemp4)&&(hTemp5)&&(hTemp6)) {
+    
+    gHistBalanceFunctionHistogram = (TH2D*)hTemp1->Clone();
+    gHistBalanceFunctionHistogram->Reset();
+    
+    // Calculate BF

     hTemp1->Add(hTemp3,-1.);
     hTemp1->Scale(1./hTemp5->GetEntries());
     hTemp2->Add(hTemp4,-1.);
     hTemp2->Scale(1./hTemp6->GetEntries());
     gHistBalanceFunctionHistogram->Add(hTemp1,hTemp2,1.,1.);
     hTemp1->Add(hTemp3,-1.);
     hTemp1->Scale(1./hTemp5->GetEntries());
     hTemp2->Add(hTemp4,-1.);
     hTemp2->Scale(1./hTemp6->GetEntries());
     gHistBalanceFunctionHistogram->Add(hTemp1,hTemp2,1.,1.);

-    gHistBalanceFunctionHistogram->Scale(0.5/1.);

   }
 
   return gHistBalanceFunctionHistogram;
 }
 
   }
 
   return gHistBalanceFunctionHistogram;
 }
 

-TH1D* AliBalanceTriggered::GetHistogram1D(Int_t histo, Int_t var, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){
+
+TH1D* AliBalanceTriggered::GetHistogram1D(Int_t histo, Int_t var, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){

 
   // check which variable should be analyzed
   //
 
   // check which variable should be analyzed
   //


@@ -422,10 +488,95 @@ TH1D* AliBalanceTriggered::GetHistogram1D(Int_t histo, Int_t var, Double_t pTMin
   case 5:
     gTHn = fHistNN;
     break;
   case 5:
     gTHn = fHistNN;
     break;

+    
+  }
+
+  if(!gTHn){
+    AliError(Form("AliTHn number %d = NULL",histo));
+    return NULL;
+  }
+
+  // Choose region to analyze 
+  // for Single Histograms (P,N):       2 = pT,trigger; 3 = centrality
+  // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality

 
 

-  default:
+  // pT trigger
+  gTHn->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 
+ 
+  // pT
+  if(histo > 1) gTHn->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
+ 
+  // centrality
+  if(histo < 2) gTHn->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(centrMin,centrMax); 
+  else          gTHn->GetGrid(0)->GetGrid()->GetAxis(4)->SetRange(centrMin,centrMax); 
+ 
+
+  // Project into the wanted space (1st: analysis step, 2nd: axis)
+  TH1D* gHisto = (TH1D*)gTHn->Project(0,var);
+
+  return gHisto;
+}
+
+
+TH2D* AliBalanceTriggered::GetHistogram2D(Int_t histo, Int_t var1, Int_t var2, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax){
+
+  // check which variable should be analyzed
+  //
+  // pair histograms:
+  // 0 = Delta eta 
+  // 1 = Delta phi
+  // 2 = pT, trigger
+  // 3 = centrality
+  //
+  // pair histograms:
+  // 0 = Delta eta 
+  // 1 = Delta phi
+  // 2 = pT
+  // 3 = pT, trigger
+  // 4 = centrality
+
+  if(histo < 0 || histo > 5){
+    AliError("Only 6 histograms available: 0(P), 1(N), 2(PN), 3(NP), 4(PP), 5(NN)");
+    return NULL;
+  }
+
+  if( histo > 1 && (var1 < 0 || var1 > 5 || var2 < 0 || var2 > 5)){
+    AliError("Only Variable 0 to 4 allowed for pair histograms (histo > 1)");
+    return NULL;
+  }
+  if( histo < 2 && (var1 < 0 || var1 > 4 || var2 < 0 || var2 > 4)){
+    AliError("Only Variable 0 to 3 allowed for single histograms (histo < 2)");
+    return NULL;
+  }
+
+  // get the histogram
+  AliTHn *gTHn = NULL;
+  switch(histo){
+ 
+  case 0:
+    gTHn = fHistP;
+    break;
+
+  case 1:
+    gTHn = fHistN;

     break;
 
     break;
 

+  case 2:
+    gTHn = fHistPN;
+    break;
+
+  case 3:
+    gTHn = fHistNP;
+    break;
+
+  case 4:
+    gTHn = fHistPP;
+    break;
+
+  case 5:
+    gTHn = fHistNN;
+    break;
+    

   }
 
   if(!gTHn){
   }
 
   if(!gTHn){


@@ -438,7 +589,7 @@ TH1D* AliBalanceTriggered::GetHistogram1D(Int_t histo, Int_t var, Double_t pTMin
   // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality
 
   // pT trigger
   // for Pair Histograms (PN,NP,NN,PP): 2 = pT; 3 = pT,trigger; 4 = centrality
 
   // pT trigger

-  gTHn->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMin,pTMax); 
+  gTHn->GetGrid(0)->GetGrid()->GetAxis(2)->SetRange(pTMinTrigger,pTMaxTrigger); 

  
   // pT
   if(histo > 1) gTHn->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 
  
   // pT
   if(histo > 1) gTHn->GetGrid(0)->GetGrid()->GetAxis(3)->SetRange(pTMin,pTMax); 


@@ -449,7 +600,7 @@ TH1D* AliBalanceTriggered::GetHistogram1D(Int_t histo, Int_t var, Double_t pTMin
  
 
   // Project into the wanted space (1st: analysis step, 2nd: axis)
  
 
   // Project into the wanted space (1st: analysis step, 2nd: axis)

-  TH1D* gHisto = (TH1D*)gTHn->Project(0,var);
+  TH2D* gHisto = (TH2D*)gTHn->Project(0,var1,var2);

 
   return gHisto;
 }
 
   return gHisto;
 }

diff --git a/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.h b/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.h
index 7b58eab18cab90e12c705227029f87498ef08f44..5d90fe61f37ab76f85120f64c3ce43255b13e97f 100644 (file)
--- a/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.h
+++ b/PWGCF/EBYE/BalanceFunctions/AliBalanceTriggered.h
@@ -35,7 +35,7 @@ class AliBalanceTriggered : public TObject {
 
   // analysis getters
   const char* GetAnalysisLevel() {return fAnalysisLevel.Data();}
 
   // analysis getters
   const char* GetAnalysisLevel() {return fAnalysisLevel.Data();}

-  const Bool_t GetShuffle() {return bShuffle;}
+  Bool_t GetShuffle() {return bShuffle;}

 
   // initialize histograms
   void InitHistograms(void);
 
   // initialize histograms
   void InitHistograms(void);


@@ -60,10 +60,16 @@ class AliBalanceTriggered : public TObject {
   void FillBalance(Float_t fCentrality,vector<Double_t> **chargeVector);
  
   // Get the balance function histogram 
   void FillBalance(Float_t fCentrality,vector<Double_t> **chargeVector);
  
   // Get the balance function histogram 

-  TH1D *GetBalanceFunctionHistogram1D(Int_t var, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);
+  TH1D *GetBalanceFunctionHistogram1D(Int_t var, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);
+
+  // Get the balance function histogram (2D) 
+  TH2D *GetBalanceFunctionHistogram2D(Int_t var1, Int_t var2, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);

 
   // Get 1D histogram
 
   // Get 1D histogram

-  TH1D* GetHistogram1D(Int_t histo, Int_t var, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);
+  TH1D* GetHistogram1D(Int_t histo, Int_t var, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);
+
+  // Get 2D histogram
+  TH2D* GetHistogram2D(Int_t histo, Int_t var1, Int_t var2, Double_t pTMinTrigger, Double_t pTMaxTrigger, Double_t pTMin, Double_t pTMax, Double_t centrMin, Double_t centrMax);

 
  private:
   Bool_t bShuffle; //shuffled balance function object
 
  private:
   Bool_t bShuffle; //shuffled balance function object

diff --git a/PWGCF/EBYE/macros/AddTaskBalanceTriggered.C b/PWGCF/EBYE/macros/AddTaskBalanceTriggered.C
index 69d9f5e2a8c944df58e490eb3e1f17a73e123bba..46fbfe894ce226946af0c343fe7902b7359b1ff1 100644 (file)
--- a/PWGCF/EBYE/macros/AddTaskBalanceTriggered.C
+++ b/PWGCF/EBYE/macros/AddTaskBalanceTriggered.C
@@ -139,7 +139,7 @@ AliAnalysisTaskTriggeredBF *AddTaskBalanceTriggered(Double_t centrMin=0.,
   // Get and connect other common input/output containers via the manager as below\r
   //==============================================================================\r
   TString outputFileName = AliAnalysisManager::GetCommonFileName();\r
   // Get and connect other common input/output containers via the manager as below\r
   //==============================================================================\r
   TString outputFileName = AliAnalysisManager::GetCommonFileName();\r

-  outputFileName += ":PWGCFEbyE.outputBalanceFunctionAnalysis";\r
+  outputFileName += ":PWGCFEbyE.outputBalanceFunctionTriggeredAnalysis";\r

   AliAnalysisDataContainer *coutQA = mgr->CreateContainer(Form("listQA_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r
   AliAnalysisDataContainer *coutTriggeredBF = mgr->CreateContainer(Form("listTriggeredBF_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r
   if(gRunShuffling) AliAnalysisDataContainer *coutTriggeredBFS = mgr->CreateContainer(Form("listTriggeredBFShuffled_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r
   AliAnalysisDataContainer *coutQA = mgr->CreateContainer(Form("listQA_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r
   AliAnalysisDataContainer *coutTriggeredBF = mgr->CreateContainer(Form("listTriggeredBF_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r
   if(gRunShuffling) AliAnalysisDataContainer *coutTriggeredBFS = mgr->CreateContainer(Form("listTriggeredBFShuffled_%s",centralityName.Data()), TList::Class(),AliAnalysisManager::kOutputContainer,outputFileName.Data());\r