fixes for rulechecker - mostly comments
authordsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 26 Jun 2012 21:12:14 +0000 (21:12 +0000)
committerdsilverm <dsilverm@f7af4fe6-9843-0410-8265-dc069ae4e863>
Tue, 26 Jun 2012 21:12:14 +0000 (21:12 +0000)
13 files changed:
PWGLF/totEt/AliAnalysisEmEtMonteCarlo.cxx
PWGLF/totEt/AliAnalysisEt.h
PWGLF/totEt/AliAnalysisEtMonteCarlo.cxx
PWGLF/totEt/AliAnalysisEtMonteCarlo.h
PWGLF/totEt/AliAnalysisEtMonteCarloPhos.cxx
PWGLF/totEt/AliAnalysisEtMonteCarloPhos.h
PWGLF/totEt/AliAnalysisEtReconstructed.h
PWGLF/totEt/AliAnalysisEtReconstructedPhos.cxx
PWGLF/totEt/AliAnalysisEtSelector.cxx
PWGLF/totEt/AliAnalysisEtSelector.h
PWGLF/totEt/AliAnalysisEtSelectorEmcal.cxx
PWGLF/totEt/AliAnalysisEtSelectorPhos.cxx
PWGLF/totEt/AliAnalysisEtSelectorPhos.h

index fac4010..774912b 100644 (file)
@@ -1458,10 +1458,15 @@ Int_t AliAnalysisEmEtMonteCarlo::AnalyseEvent(AliVEvent* ev)
     // Get us an mc event
     if(!ev)
     {
-        Printf("ERROR: Event does not exist");   
+        Printf("ERROR: ev does not exist");   
         return 0;
     }
     AliMCEvent *event = dynamic_cast<AliMCEvent*>(ev);
+    if(!event)
+    {
+        Printf("ERROR: event does not exist");   
+        return 0;
+    }
     
     // Hijing header
     AliGenEventHeader* genHeader = event->GenEventHeader();
index c7f77b6..e84d6d6 100644 (file)
@@ -1,3 +1,5 @@
+#ifndef ALIANALYSISET_H
+#define ALIANALYSISET_H
 //_________________________________________________________________________
 //  Utility Class for transverse energy studies
 //  Base class for ESD & MC analysis
@@ -6,9 +8,6 @@
 //*-- Authors: Oystein Djuvsland (Bergen), David Silvermyr (ORNL)
 //_________________________________________________________________________
 
-#ifndef ALIANALYSISET_H
-#define ALIANALYSISET_H
-
 #include "AliAnalysisEtCommon.h"
 #include "THnSparse.h"
 #include "AliESDCaloCluster.h"
@@ -255,9 +254,9 @@ protected:
     /** Minimum energy to cut on single cell cluster */
     Double_t fSingleCellEnergyCut;  // Minimum energy to cut on single cell cluster
 
-    Double_t fChargedEnergyRemoved;
-    Double_t fNeutralEnergyRemoved;
-    Double_t fGammaEnergyAdded;
+    Double_t fChargedEnergyRemoved; // Charged energy removed
+    Double_t fNeutralEnergyRemoved; // Neutral energy removed
+    Double_t fGammaEnergyAdded; // gamma energy added
 
     // Declare the histograms
 
index b796e7f..225219e 100644 (file)
@@ -1488,7 +1488,7 @@ void AliAnalysisEtMonteCarlo::FillHistograms()
 
 
 Int_t AliAnalysisEtMonteCarlo::PrintFamilyTree(Int_t partIdx, AliStack* stack)
-{
+{ // print family tree
     TParticle *part = stack->Particle(partIdx);
     if(part->GetPdgCode() == fgK0SCode)
     {
@@ -1501,7 +1501,7 @@ Int_t AliAnalysisEtMonteCarlo::PrintFamilyTree(Int_t partIdx, AliStack* stack)
 }
 
 Int_t AliAnalysisEtMonteCarlo::PrintMothers(Int_t partIdx, AliStack* stack, Int_t gen)
-{
+{ // print mothers
     char *tabs = new char[gen+1];
     for(Int_t i = 0; i < gen; ++i)
     {
@@ -1537,7 +1537,7 @@ Int_t AliAnalysisEtMonteCarlo::PrintMothers(Int_t partIdx, AliStack* stack, Int_
 }
 
 Int_t AliAnalysisEtMonteCarlo::GetPrimMother(Int_t partIdx, AliStack *stack)
-{
+{ // get primary mother
     if(partIdx >= 0)
     {
         Int_t mothIdx = stack->Particle(partIdx)->GetMother(0);
@@ -1567,7 +1567,7 @@ Int_t AliAnalysisEtMonteCarlo::GetPrimMother(Int_t partIdx, AliStack *stack)
 }
 
 Int_t AliAnalysisEtMonteCarlo::GetK0InFamily(Int_t partIdx, AliStack* stack)
-{
+{ // get K0 in family
     if(partIdx >= 0)
     {
         if(stack->Particle(partIdx)->GetPdgCode() == fgK0SCode) return partIdx;
index a861fcb..8e25150 100644 (file)
@@ -1,19 +1,18 @@
 #ifndef ALIANALYSISETMONTECARLO_H
 #define ALIANALYSISETMONTECARLO_H
-
-class AliPHOSGeometry;//_________________________________________________________________________
+//_________________________________________________________________________
 //  Utility Class for transverse energy studies
 //  Base class for MC analysis
 //  - MC output
 //
 //*-- Authors: Oystein Djuvsland (Bergen), David Silvermyr (ORNL)
 //_________________________________________________________________________
-
 #include "AliAnalysisEt.h"
 class TParticle;
 class TH3F;
 class TH2I;
 class AliPHOSGeoUtils;
+class AliPHOSGeometry;
 class AliStack;
 //class AliMCEvent;
 //class AliESDEvent;
@@ -57,32 +56,32 @@ protected:
     
     TTree *fPrimaryTree; // Tree holding info on primaries
 
-    Double_t fTotEtWithSecondaryRemoved;
-    Double_t fTotEtSecondaryFromEmEtPrimary;
-    Double_t fTotEtSecondary;
+    Double_t fTotEtWithSecondaryRemoved; // enter comment here
+    Double_t fTotEtSecondaryFromEmEtPrimary; // enter comment here
+    Double_t fTotEtSecondary; // enter comment here
     
-    Int_t fPrimaryCode;
-    Int_t fPrimaryCharge;
+    Int_t fPrimaryCode; // enter comment here
+    Int_t fPrimaryCharge; // enter comment here
 
-    Double_t fPrimaryE;
-    Double_t fPrimaryEt;
+    Double_t fPrimaryE; // enter comment here
+    Double_t fPrimaryEt; // enter comment here
 
-    Double_t fPrimaryPx;
-    Double_t fPrimaryPy;
-    Double_t fPrimaryPz;
+    Double_t fPrimaryPx; // enter comment here
+    Double_t fPrimaryPy; // enter comment here
+    Double_t fPrimaryPz; // enter comment here
     
-    Double_t fPrimaryVx;
-    Double_t fPrimaryVy;
-    Double_t fPrimaryVz;
+    Double_t fPrimaryVx; // enter comment here
+    Double_t fPrimaryVy; // enter comment here
+    Double_t fPrimaryVz; // enter comment here
     
-    Bool_t fPrimaryAccepted;
-    Int_t fDepositedCode;
-    Double_t fDepositedEt;
-    Int_t fDepositedCharge;
-
-    Double_t fDepositedVx;
-    Double_t fDepositedVy;
-    Double_t fDepositedVz;
+    Bool_t fPrimaryAccepted; // enter comment here
+    Int_t fDepositedCode; // enter comment here
+    Double_t fDepositedEt; // enter comment here
+    Int_t fDepositedCharge; // enter comment here
+
+    Double_t fDepositedVx; // enter comment here
+    Double_t fDepositedVy; // enter comment here
+    Double_t fDepositedVz; // enter comment here
     
     
     TH3F *fHistDecayVertexNonRemovedCharged; // Decay vertex for non-removed charged particles
@@ -248,8 +247,10 @@ protected:
     TH1F *fHistPiMinusMultAcc; // enter comment here
     TH1F *fHistPiZeroMultAcc; // enter comment here
 
+    // DS: pi+- mult already defined in base class..
     Int_t fPiPlusMult; // enter comment here
     Int_t fPiMinusMult; // enter comment here
+    
     Int_t fPiZeroMult; // enter comment here
 
     Int_t fPiPlusMultAcc; // enter comment here
@@ -269,7 +270,7 @@ protected:
 
     Int_t fNClusters; // Number of clusters in event
 
-    Double_t fTotNeutralEtAfterMinEnergyCut;
+    Double_t fTotNeutralEtAfterMinEnergyCut; // enter comment here
     
 private:
 
index 126cf26..7aff80b 100644 (file)
@@ -31,7 +31,7 @@ AliAnalysisEtMonteCarloPhos::AliAnalysisEtMonteCarloPhos():AliAnalysisEtMonteCar
 }
 
 AliAnalysisEtMonteCarloPhos::~AliAnalysisEtMonteCarloPhos()
-{
+{ // dtor
   delete fBadMapM2;
   delete fBadMapM3;
   delete fBadMapM4;
@@ -60,7 +60,7 @@ void AliAnalysisEtMonteCarloPhos::Init()
 
 
 Bool_t AliAnalysisEtMonteCarloPhos::TooCloseToBadChannel(const AliESDCaloCluster &cluster) const
-{
+{ // too close to bad channel?
 
     Float_t gPos[3];
     cluster.GetPosition(gPos);
index 1ac18eb..374a338 100644 (file)
@@ -1,7 +1,5 @@
 #ifndef ALIANALYSISETMONTECARLOPHOS_H
 #define ALIANALYSISETMONTECARLOPHOS_H
-
-class AliPHOSGeoUtils;
 //_________________________________________________________________________
 //  Utility Class for transverse energy studies
 //  Base class for MC analysis, for PHOS
@@ -9,9 +7,8 @@ class AliPHOSGeoUtils;
 //
 //*-- Authors: Oystein Djuvsland (Bergen), David Silvermyr (ORNL)
 //_________________________________________________________________________
-
 #include "AliAnalysisEtMonteCarlo.h"
-
+class AliPHOSGeoUtils;
 class TH2I;
 
 class AliAnalysisEtMonteCarloPhos : public AliAnalysisEtMonteCarlo
index 3d4070e..f769d1b 100644 (file)
@@ -1,9 +1,5 @@
 #ifndef ALIANALYSISETRECONSTRUCTED_H
 #define ALIANALYSISETRECONSTRUCTED_H
-
-class TH2F;
-class TH2D;
-
 //_________________________________________________________________________
 //  Utility Class for transverse energy studies
 //  Base class for ESD analysis
@@ -13,7 +9,8 @@ class TH2D;
 //_________________________________________________________________________
 
 #include "AliAnalysisEt.h"
-
+class TH2F;
+class TH2D;
 class TH2I;
 class AliVParticle;
 class AliESDEvent;
@@ -67,9 +64,9 @@ protected:
     
     TH2D *fClusterPosition; // Position of clusters
     
-    TH2D *fHistChargedEnergyRemoved;
-    TH2D *fHistNeutralEnergyRemoved;
-    TH2D *fHistGammaEnergyAdded;
+    TH2D *fHistChargedEnergyRemoved; // Charged energy removed
+    TH2D *fHistNeutralEnergyRemoved; // Neutral energy removed
+    TH2D *fHistGammaEnergyAdded; // gamma energy added
     
 
 private:
index b694102..e9efc9a 100644 (file)
@@ -40,7 +40,7 @@ const Double_t kMEANGAMMA = 0.374/(0.51 + 0.02*0.374);
 
 AliAnalysisEtReconstructedPhos::AliAnalysisEtReconstructedPhos() :
         AliAnalysisEtReconstructed()
-{
+{ // ctor
     fHistogramNameSuffix = TString("PhosRec");
 
 //     fChargedContributionCorrectionParameters[0] = -0.017;
@@ -88,7 +88,7 @@ void AliAnalysisEtReconstructedPhos::Init()
 }
 
 bool AliAnalysisEtReconstructedPhos::TrackHitsCalorimeter(AliVParticle* track, Double_t magField)
-{
+{ 
     return  AliAnalysisEtReconstructed::TrackHitsCalorimeter(track, magField);
 }
 
@@ -133,7 +133,7 @@ Double_t AliAnalysisEtReconstructedPhos::GetGammaContribution(Int_t clusterMult)
 }
 
 Double_t AliAnalysisEtReconstructedPhos::GetSecondaryContribution(Int_t clusterMultiplicity)
-{
+{ // Secondary contrib
   if(clusterMultiplicity > 0)
   {
     return fSecondaryContributionCorrectionParameters[0] + fSecondaryContributionCorrectionParameters[1]*clusterMultiplicity + fSecondaryContributionCorrectionParameters[2]*clusterMultiplicity*clusterMultiplicity;
index 91e1545..de290a3 100644 (file)
@@ -1,3 +1,11 @@
+//_________________________________________________________________________
+//  Utility Class for transverse energy studies
+//  Selector Base class 
+//  -  
+// implementation file
+//
+//*-- Authors: Oystein Djuvsland (Bergen)
+//_________________________________________________________________________
 #include "AliAnalysisEtSelector.h"
 #include "AliAnalysisEtCuts.h"
 #include "AliESDCaloCluster.h"
@@ -18,7 +26,7 @@ AliAnalysisEtSelector::AliAnalysisEtSelector(AliAnalysisEtCuts *cuts) : AliAnaly
 }
 
 AliAnalysisEtSelector::~AliAnalysisEtSelector()
-{
+{ // dtor
   if(fClusterArray)
   {
     delete fClusterArray;
@@ -41,7 +49,7 @@ Bool_t AliAnalysisEtSelector::PrimaryIsEmEtParticle(const Int_t pIdx, AliStack&
   return IsEmEtParticle(stack.Particle(GetPrimary(pIdx, stack))->GetPdgCode());
 }
 Int_t AliAnalysisEtSelector::GetPrimary(const Int_t partIdx, AliStack& stack) const
-{
+{ // get primary
   if(partIdx >= 0) 
   {
     Int_t mothIdx = stack.Particle(partIdx)->GetMother(0);
index dc59d23..4684db6 100644 (file)
@@ -1,5 +1,12 @@
 #ifndef ALIANALYSISETSELECTOR_H
 #define ALIANALYSISETSELECTOR_H
+//_________________________________________________________________________
+//  Utility Class for transverse energy studies
+//  Selector Base class
+//  -  
+//
+//*-- Authors: Oystein Djuvsland (Bergen)
+//_________________________________________________________________________
 #include <Rtypes.h>
 #include "AliAnalysisEtCommon.h"
 #include "AliESDEvent.h"
@@ -70,11 +77,11 @@ protected:
   
     const AliVEvent *fEvent; // Pointer to current event
 
-    AliAnalysisEtCuts *fCuts; // Pointer to the cuts object
-    
     TRefArray *fClusterArray; // Array of clusters
-    
-    Int_t fRunNumber;
+
+    AliAnalysisEtCuts *fCuts; // Pointer to the cuts object; DS: also in base class?
+
+    Int_t fRunNumber; // run number
     
 
     
index 822acd8..8ce99fb 100644 (file)
@@ -32,7 +32,7 @@ void AliAnalysisEtSelectorEmcal::Init()
 }
 
 Int_t AliAnalysisEtSelectorEmcal::Init(const AliESDEvent* event)
-{
+{ // Init
     
     AliAnalysisEtSelector::Init(event);
     Printf("Initializing selector for run: %d", event->GetRunNumber());
@@ -41,7 +41,7 @@ Int_t AliAnalysisEtSelectorEmcal::Init(const AliESDEvent* event)
 }
 
 TRefArray* AliAnalysisEtSelectorEmcal::GetClusters()
-{
+{ // Get clusters
     
   if(!fClusterArray) fClusterArray = new TRefArray;
   
index 3c6fe99..05d70bc 100644 (file)
@@ -1,3 +1,11 @@
+//_________________________________________________________________________
+//  Utility Class for transverse energy studies
+//  Selector Base class for PHOS
+//  -  
+// implementation file
+//
+//*-- Authors: Oystein Djuvsland (Bergen)
+//_________________________________________________________________________
 #include "AliAnalysisEtSelectorPhos.h"
 #include "AliAnalysisEtCuts.h"
 #include "AliESDCaloCluster.h"
@@ -30,7 +38,7 @@ AliAnalysisEtSelectorPhos::~AliAnalysisEtSelectorPhos()
 }
 
 TRefArray* AliAnalysisEtSelectorPhos::GetClusters()
-{
+{ // Get clusters
   if(!fClusterArray) fClusterArray = new TRefArray;
   
   if(fClusterArray)
@@ -46,7 +54,7 @@ TRefArray* AliAnalysisEtSelectorPhos::GetClusters()
 }
 
 Int_t AliAnalysisEtSelectorPhos::Init(const AliESDEvent* event)
-{
+{ // Init
   
   AliAnalysisEtSelector::Init(event);
   Printf("Initializing selector for run: %d", event->GetRunNumber());
@@ -84,7 +92,7 @@ Bool_t AliAnalysisEtSelectorPhos::CutMinEnergy(const TParticle& part) const
 
 
 Bool_t AliAnalysisEtSelectorPhos::CutDistanceToBadChannel(const AliESDCaloCluster& cluster) const
-{
+{ // cut distance to bad channel
   if(!fMatrixInitialized)
   {
     Printf("Misalignment matrices are not initialized");
@@ -188,7 +196,7 @@ Bool_t AliAnalysisEtSelectorPhos::CutDistanceToBadChannel(const AliESDCaloCluste
 }
 
 Bool_t AliAnalysisEtSelectorPhos::CutTrackMatching(const AliESDCaloCluster& cluster) const
-{
+{ // cut track matching
 
   if(!fMatrixInitialized)
   {
@@ -243,7 +251,7 @@ Bool_t AliAnalysisEtSelectorPhos::CutTrackMatching(const AliESDCaloCluster& clus
 }
 
 int AliAnalysisEtSelectorPhos::LoadGeometry()
-{
+{ // load geometry
 
   fGeoUtils = AliPHOSGeometry::GetInstance("IHEP");
     // ifstream f("badchannels.txt", ios::in);
@@ -251,7 +259,7 @@ int AliAnalysisEtSelectorPhos::LoadGeometry()
 }
 
 int AliAnalysisEtSelectorPhos::LoadBadMaps()
-{
+{ // load bad maps
 TFile *f = TFile::Open("badchannels.root", "READ");
 
     if(!f)
@@ -283,7 +291,7 @@ TFile *f = TFile::Open("badchannels.root", "READ");
 }
 
 void AliAnalysisEtSelectorPhos::SetEvent(const AliESDEvent* event)
-{
+{ // set event
     //AliAnalysisEtSelector::SetEvent(event);
     fEvent = event;
     if(!fInitialized) Init(event);
index 86664b9..5f7f5eb 100644 (file)
@@ -1,6 +1,12 @@
 #ifndef ALIANALYSISETSELECTORPHOS_H
 #define ALIANALYSISETSELECTORPHOS_H
-
+//_________________________________________________________________________
+//  Utility Class for transverse energy studies
+//  Selector Base class for PHOS
+//  - 
+//
+//*-- Authors: Oystein Djuvsland (Bergen)
+//_________________________________________________________________________
 #include "AliAnalysisEtSelector.h"
 
 class TH2I;
@@ -31,10 +37,10 @@ public:
 private:
 
 
-    int LoadGeometry();
-    int LoadBadMaps();
+    int LoadGeometry(); // load geometry
+    int LoadBadMaps(); // load bad maps
     
-    AliPHOSGeometry *fGeoUtils;
+    AliPHOSGeometry *fGeoUtils; // geo utils
     
     TH2I *fBadMapM2; // Bad map
     TH2I *fBadMapM3; // Bad map