- fix coding rule violations
authorjklein <jklein@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 2 Feb 2012 16:15:15 +0000 (16:15 +0000)
committerjklein <jklein@f7af4fe6-9843-0410-8265-dc069ae4e863>
Thu, 2 Feb 2012 16:15:15 +0000 (16:15 +0000)
16 files changed:
PWGPP/TRD/AliTRDonlineTrackletFilter.cxx
PWGPP/TRD/AliTRDonlineTrackletFilter.h
PWGPP/TRD/AliTRDonlineTrackletQA.cxx
PWGPP/TRD/AliTRDonlineTrackletQA.h
TRD/AliTRDTriggerL0.cxx
TRD/AliTRDgtuTMU.cxx
TRD/AliTRDgtuTMU.h
TRD/AliTRDltuParam.cxx
TRD/AliTRDmcmSim.cxx
TRD/AliTRDmcmSim.h
TRD/AliTRDrawStream.cxx
TRD/AliTRDrawStream.h
TRD/AliTRDtrackletMCM.cxx
TRD/AliTRDtrapConfig.cxx
TRD/AliTRDtrapConfig.h
TRD/AliTRDtrapConfigHandler.h

index 9a02680..5da9656 100644 (file)
@@ -1,3 +1,10 @@
+// AliTRDonlineTrackletFilter implements an analysis task which reads
+// the tracklets from the TRD.Tracklets.root file and makes them
+// available to other analysis tasks. This allows to access the full
+// tracklet information without repeating the techncicalities in each
+// analysis task. The tracklets are made available in an exchange
+// container on the output slot 1.
+
 #include "TFile.h"
 #include "TTree.h"
 #include "TChain.h"
@@ -64,6 +71,8 @@ AliTRDonlineTrackletFilter::~AliTRDonlineTrackletFilter()
 
 void AliTRDonlineTrackletFilter::ConnectInputData(const Option_t */* option */)
 {
+  // connect all the input handlers to access the data
+
   fInputHandler = (AliInputEventHandler*) AliAnalysisManager::GetAnalysisManager()->GetInputEventHandler();
   if (fInputHandler)
     fInputEvent = fInputHandler->GetEvent();
@@ -75,6 +84,9 @@ void AliTRDonlineTrackletFilter::ConnectInputData(const Option_t */* option */)
 
 void AliTRDonlineTrackletFilter::CreateOutputObjects()
 {
+  // create the output objects used to make the tracklets available to
+  // other tasks
+
   OpenFile(1); 
   
   fTrackletTree = new TTree("tracklets", "on-line tracklets");
@@ -84,6 +96,8 @@ void AliTRDonlineTrackletFilter::CreateOutputObjects()
 
 Bool_t AliTRDonlineTrackletFilter::Notify()
 {
+  // we use notify to figure out which TRD.Tracklets.root file we have
+  // to use for accessing the tracklets
 
   TString filename(AliAnalysisManager::GetAnalysisManager()->GetTree()->GetCurrentFile()->GetName());
 
index d543a2d..6f6dffe 100644 (file)
@@ -1,3 +1,6 @@
+// AliTRDonlineTrackletFilter implements an analysis task which makes
+// the TRD on-line tracklets accessible for other analysis tasks.
+
 #ifndef ALITRDONLINETRACKLETFILTER
 #define ALITRDONLINETRACKLETFILTER
 
@@ -48,7 +51,7 @@ class AliTRDonlineTrackletFilter : public AliAnalysisTask
   // ----- internal use -----
   AliTRDgeometry       *fGeo; //! TRD geometry
 
-  Int_t fNevent;
+  Int_t fNevent; // current event number
 
   TString fPath; //!
   TFile *fTrackletFile; //!
index e23a23d..77b42dd 100644 (file)
@@ -1,3 +1,10 @@
+// AliTRDonlineTrackletQA implements the standard QA for the TRD
+// on-line tracklets. It reads the tracklets provided by
+// AliTRDonlineTrackletFilter and analyses them. The QA can be run
+// on tracklets from simulation or from real data. In addition it is
+// possible to compare the tracklets from real data to re-simulation
+// based on the ADC data.
+
 #include "TFile.h"
 #include "TTree.h"
 #include "TChain.h"
@@ -677,7 +684,7 @@ void AliTRDonlineTrackletQA::PlotESD(AliTRDtrackletWord *trkl)
 }
 
 
-Int_t AliTRDonlineTrackletQA::GetTrackletsForMC(Int_t /* label */, Int_t /*idx*/ [])
+Int_t AliTRDonlineTrackletQA::GetTrackletsForMC(Int_t /* label */, Int_t /*idx*/ []) const
 {
   // get tracklets for MC label
   // not implemented
index 76e400d..6716cc5 100644 (file)
@@ -1,3 +1,6 @@
+// AliTRDonlineTrackletQA implements the standard QA for the TRD
+// on-line tracklets.
+
 #ifndef ALITRDONLINETRACKLETQA_H
 #define ALITRDONLINETRACKLETQA_H
 
@@ -32,7 +35,7 @@ class AliTRDonlineTrackletQA : public AliAnalysisTask
 
   void PlotESD(AliTRDtrackletWord *trkl);
 
-  Int_t GetTrackletsForMC(Int_t label, Int_t idx[]);
+  Int_t GetTrackletsForMC(Int_t label, Int_t idx[]) const;
 
  protected:
   AliESDEvent *fESD;                    //! current ESD event
index cc69271..bba3389 100644 (file)
 
 AliTRDTriggerL0::AliTRDTriggerL0()
 {
+  // constructor
+
   SetName("TRD");
 }
 
 AliTRDTriggerL0::~AliTRDTriggerL0()
 {
-
+  // destructor
 }
 
 void AliTRDTriggerL0::CreateInputs()
 {
+  // create the L0 inputs which are provided by the TRD
+
   if (fInputs.GetEntriesFast() > 0)
     return;
 
@@ -56,6 +60,7 @@ void AliTRDTriggerL0::CreateInputs()
 
 void AliTRDTriggerL0::Trigger()
 {
+  // do the actual trigger calculation
 
   AliRunLoader *runLoader = AliRunLoader::Instance();
   if (!runLoader)
index 9a3be50..6ca3909 100644 (file)
@@ -1015,7 +1015,7 @@ Bool_t AliTRDgtuTMU::CalculateTrackParams(AliTRDtrackGTU *track)
 }
 
 
-Bool_t AliTRDgtuTMU::Uniquifier(TList *inlist, TList *outlist)
+Bool_t AliTRDgtuTMU::Uniquifier(const TList *inlist, TList *outlist)
 {
   // remove multiple occurences of the same track
 
index 91deb77..8310b09 100644 (file)
@@ -43,7 +43,7 @@ class AliTRDgtuTMU : public TObject {
   Bool_t RunTrackReconstruction(TList* ListOfTracks);
 
   Bool_t CalculateTrackParams(AliTRDtrackGTU *track);
-  Bool_t Uniquifier(TList* inlist, TList *outlist);
+  Bool_t Uniquifier(const TList* inlist, TList *outlist);
   Bool_t CalculatePID(AliTRDtrackGTU *track);
 
 protected:
index 8945b8f..14194b5 100644 (file)
@@ -42,12 +42,12 @@ AliTRDltuParam::AliTRDltuParam() :
   fTiltCorr(kFALSE),
   fPidGainCorr(kFALSE)
 {
-
+  // default constructor
 }
 
 AliTRDltuParam::~AliTRDltuParam()
 {
-
+  // destructor
 }
 
 Int_t AliTRDltuParam::GetDyCorrection(Int_t det, Int_t rob, Int_t mcm) const
@@ -111,6 +111,9 @@ void AliTRDltuParam::GetDyRange(Int_t det, Int_t rob, Int_t mcm, Int_t ch,
 
 Float_t AliTRDltuParam::GetElongation(Int_t det, Int_t rob, Int_t mcm, Int_t ch) const
 {
+  // calculate the ratio of the distance to the primary vertex and the
+  // distance in x-direction for the given ADC channel
+
   Int_t layer = det % 6;
 
   Float_t elongation = TMath::Abs(GetDist(det, rob, mcm, ch) / fgX[layer]);
@@ -125,6 +128,8 @@ Float_t AliTRDltuParam::GetElongation(Int_t det, Int_t rob, Int_t mcm, Int_t ch)
 void AliTRDltuParam::GetCorrectionFactors(Int_t det, Int_t rob, Int_t mcm, Int_t ch,
                                          UInt_t &cor0, UInt_t &cor1, Float_t gain) const
 {
+  // calculate the gain correction factors for the given ADC channel
+
   if (fPidGainCorr==kFALSE)
     gain=1;
 
@@ -140,17 +145,23 @@ void AliTRDltuParam::GetCorrectionFactors(Int_t det, Int_t rob, Int_t mcm, Int_t
 
 Int_t AliTRDltuParam::GetNtimebins() const
 {
+  // return the number of timebins used
+
   return fNtimebins;
 }
 
 Float_t AliTRDltuParam::GetX(Int_t det, Int_t /* rob */, Int_t /* mcm */) const
 {
+  // return the distance to the beam axis in x-direction
+
   Int_t layer = det%6;
   return fgX[layer];
 }
 
 Float_t AliTRDltuParam::GetLocalY(Int_t det, Int_t rob, Int_t mcm, Int_t ch) const
 {
+  // get local y-position (r-phi) w.r.t. the chamber centre
+
   Int_t layer = det%6;
   // calculate the pad position as in the TRAP
   Float_t ypos = (-4 + 1 + (rob&0x1) * 4 + (mcm&0x3)) * 18 - ch - 0.5; // y position in bins of pad widths
@@ -159,6 +170,8 @@ Float_t AliTRDltuParam::GetLocalY(Int_t det, Int_t rob, Int_t mcm, Int_t ch) con
 
 Float_t AliTRDltuParam::GetLocalZ(Int_t det, Int_t rob, Int_t mcm) const
 {
+  // get local z-position w.r.t. to the chamber boundary
+
   Int_t stack = (det%30) / 6;
   Int_t layer = det % 6;
   Int_t row   = (rob/2) * 4 + mcm/4;
@@ -183,17 +196,23 @@ Float_t AliTRDltuParam::GetLocalZ(Int_t det, Int_t rob, Int_t mcm) const
 
 Float_t AliTRDltuParam::GetPerp(Int_t det, Int_t rob, Int_t mcm, Int_t ch) const
 {
+  // get transverse distance to the beam axis
+
   return TMath::Sqrt(GetLocalY(det, rob, mcm, ch)*GetLocalY(det, rob, mcm, ch) +
                     GetX(det, rob, mcm)*GetX(det, rob, mcm) );
 }
 
 Float_t AliTRDltuParam::GetPhi(Int_t det, Int_t rob, Int_t mcm, Int_t ch) const
 {
+  // calculate the azimuthal angle for the given ADC channel
+
   return TMath::ATan2(GetLocalY(det, rob, mcm, ch), GetX(det, rob, mcm));
 }
 
 Float_t AliTRDltuParam::GetDist(Int_t det, Int_t rob, Int_t mcm, Int_t ch) const
 {
+  // calculate the distance from the origin for the given ADC channel
+
   return TMath::Sqrt(GetLocalY(det, rob, mcm, ch)*GetLocalY(det, rob, mcm, ch) +
                     GetX(det, rob, mcm)*GetX(det, rob, mcm) +
                     GetLocalZ(det, rob, mcm)*GetLocalZ(det, rob, mcm) );
index 7450e98..e0a8791 100644 (file)
@@ -498,7 +498,7 @@ void AliTRDmcmSim::Draw(Option_t* const option)
   }
 }
 
-void AliTRDmcmSim::SetData( Int_t adc, Int_t* const data )
+void AliTRDmcmSim::SetData( Int_t adc, const Int_t* const data )
 {
   //
   // Store ADC data into array of raw data
@@ -591,7 +591,7 @@ void AliTRDmcmSim::SetData(AliTRDarrayADC* const adcArray, AliTRDdigitsManager *
   }
 }
 
-void AliTRDmcmSim::SetDataByPad(AliTRDarrayADC* const adcArray, AliTRDdigitsManager * const digitsManager)
+void AliTRDmcmSim::SetDataByPad(const AliTRDarrayADC* const adcArray, AliTRDdigitsManager * const digitsManager)
 {
   // Set the ADC data from an AliTRDarrayADC
   // (by pad, to be used during initial reading in simulation)
index 3b59d3d..de4da5c 100644 (file)
@@ -44,11 +44,11 @@ class AliTRDmcmSim : public TObject {
          Int_t     GetDataFiltered(Int_t iadc, Int_t timebin) const { return (fADCF[iadc][timebin] >> 2); }
          // Get filtered ADC data
 
-          void      SetData(Int_t iadc, Int_t *adc);           // Set ADC data with array
+          void      SetData(Int_t iadc, const Int_t* const adc);           // Set ADC data with array
           void      SetData(Int_t iadc, Int_t it, Int_t adc); // Set ADC data
          void      SetData(AliTRDarrayADC * const adcArray,
                            AliTRDdigitsManager * const digitsManager = 0x0);         // Set ADC data from adcArray
-         void      SetDataByPad(AliTRDarrayADC *const adcArray,
+         void      SetDataByPad(const AliTRDarrayADC *const adcArray,
                                 AliTRDdigitsManager * const digitsManager = 0x0);    // Set ADC data from adcArray
           void      SetDataPedestal(Int_t iadc);              // Fill ADC data with pedestal values
 
@@ -72,7 +72,7 @@ class AliTRDmcmSim : public TObject {
 
          void      WriteData(AliTRDarrayADC *digits);
          Bool_t    StoreTracklets();                          // Stores tracklets via runloader
-         TString   GetTrklBranchName() { return fTrklBranchName; }
+         TString   GetTrklBranchName() const { return fTrklBranchName; }
          void      SetTrklBranchName(TString name) { fTrklBranchName = name; }
 
          Int_t     ProduceRawStream( UInt_t *buf, Int_t bufsize, UInt_t iEv = 0 ) const; // Produce raw data stream - Real data format
index b33bec2..021e6f3 100644 (file)
@@ -1090,7 +1090,7 @@ Int_t AliTRDrawStream::ReadStackHeader(Int_t stack)
   AliDebug(1, DumpRaw(Form("stack %i header", stack), fPayloadCurr, fCurrStackHeaderSize[stack]));
 
   if (fPayloadCurr - fPayloadStart >= fPayloadSize - (Int_t) fCurrStackHeaderSize[stack]) {
-    EquipmentError(kStackHeaderInvalid, "Stack index header %i aborted", stack);
+    EquipmentError(kStackHeaderInvalid, "Stack index header %i incomplete", stack);
     // dumping stack header
     AliError(DumpRaw(Form("stack %i header", stack), fPayloadCurr, fCurrStackHeaderSize[stack]));
 
@@ -2152,7 +2152,7 @@ Bool_t AliTRDrawStream::DumpingMCM(Int_t det, Int_t rob, Int_t mcm)  const
   return kFALSE;
 }
 
-TString AliTRDrawStream::DumpRaw(TString title, UInt_t *start, Int_t length, UInt_t endmarker)
+TString AliTRDrawStream::DumpRaw(TString title, const UInt_t *start, Int_t length, UInt_t endmarker)
 {
   // dump raw data
 
index c436ec0..c078f91 100644 (file)
 #ifndef ALITRDRAWSTREAM_H
 #define ALITRDRAWSTREAM_H
 
-#include "TObject.h"
-#include "TClonesArray.h"
 #include "TTree.h"
-
 #include "AliLog.h"
+#include "TClonesArray.h"
 
+class TObject;
 class TObjArray;
 class TString;
 class TBranch;
@@ -172,7 +171,7 @@ class AliTRDrawStream : public TObject
   Bool_t IsDumping() const { return (fNDumpMCMs > 0); }
   Bool_t DumpingMCM(Int_t det, Int_t rob, Int_t mcm) const;
 
-  TString DumpRaw(TString title, UInt_t *start, Int_t length, UInt_t endmarker = 0xffffffff);
+  TString DumpRaw(TString title, const UInt_t *start, Int_t length, UInt_t endmarker = 0xffffffff);
   TString DumpMcmHeader(TString title, UInt_t word);
   TString DumpAdcMask(TString title, UInt_t word);
 
index ff4cbfd..f46a7a2 100644 (file)
@@ -47,6 +47,8 @@ AliTRDtrackletMCM::AliTRDtrackletMCM(UInt_t trackletWord) :
   fResiduals(0x0),
   fClsCharges(0x0)
 {
+  // constructor
+
     fGeo = new AliTRDgeometry();
     fLabel[0] = -1;
     fLabel[1] = -1;
@@ -72,6 +74,8 @@ AliTRDtrackletMCM::AliTRDtrackletMCM(UInt_t trackletWord, Int_t hcid) :
   fResiduals(0x0),
   fClsCharges(0x0)
 {
+  // constructor
+
     fGeo = new AliTRDgeometry();
     fLabel[0] = -1;
     fLabel[1] = -1;
@@ -97,6 +101,8 @@ AliTRDtrackletMCM::AliTRDtrackletMCM(UInt_t trackletWord, Int_t hcid, Int_t rob,
   fResiduals(0x0),
   fClsCharges(0x0)
 {
+  // constructor
+
     fGeo = new AliTRDgeometry();
     fLabel[0] = -1;
     fLabel[1] = -1;
@@ -122,6 +128,8 @@ AliTRDtrackletMCM::AliTRDtrackletMCM(const AliTRDtrackletMCM &rhs) :
   fResiduals(0x0),
   fClsCharges(0x0)
 {
+  // copy constructor
+
     fGeo = new AliTRDgeometry();
     fResiduals = new Float_t[fNClusters];
     fClsCharges = new Float_t[fNClusters];
@@ -136,6 +144,8 @@ AliTRDtrackletMCM::AliTRDtrackletMCM(const AliTRDtrackletMCM &rhs) :
 
 AliTRDtrackletMCM::~AliTRDtrackletMCM()
 {
+  // destructor
+
   delete [] fResiduals;
   delete [] fClsCharges;
     delete fGeo;
@@ -164,6 +174,8 @@ Int_t AliTRDtrackletMCM::GetdY() const
 
 void AliTRDtrackletMCM::SetLabel(Int_t label[])
 {
+  // set the labels (up to 3)
+
   fLabel[0] = label[0];
   fLabel[1] = label[1];
   fLabel[2] = label[2];
index bd53e94..cde01f7 100644 (file)
@@ -555,6 +555,8 @@ AliTRDtrapConfig* AliTRDtrapConfig::Instance()
 
 AliTRDtrapConfig::~AliTRDtrapConfig()
 {
+  // destructor
+
   for(Int_t iAddr = 0; iAddr < fgkDmemWords; iAddr++) {
      if(iAddr == fgkDmemAddrDeflCorr - fgkDmemStartAddress)
        delete [] fDmem[iAddr];
@@ -621,6 +623,8 @@ void AliTRDtrapConfig::ResetRegs()
 
 void AliTRDtrapConfig::ResetDmem()
 {
+  // reset the data memory
+
      for(Int_t iAddr = 0; iAddr < fgkDmemWords; iAddr++) {
        if(fDmemDepth[iAddr] == 0)
           continue;
@@ -935,6 +939,9 @@ Bool_t AliTRDtrapConfig::SetDmem(Int_t addr, UInt_t value, Int_t det, Int_t rob,
 
 UInt_t AliTRDtrapConfig::GetDmemUnsigned(Int_t addr) const
 {
+  // get the content of the data memory at the given address
+  // (only if the value is the same for all MCMs)
+
    addr = addr - fgkDmemStartAddress;
    if(addr < 0 || addr >=  fgkDmemWords) {
       AliError(Form("No DMEM address: 0x%08x", addr+fgkDmemStartAddress));
@@ -952,6 +959,9 @@ UInt_t AliTRDtrapConfig::GetDmemUnsigned(Int_t addr) const
 
 UInt_t AliTRDtrapConfig::GetDmemUnsigned(Int_t addr, Int_t det, Int_t rob, Int_t mcm) const
 {
+  // get the content of the data memory at the given address
+  // for a given MCM
+
    addr = addr - fgkDmemStartAddress;
    Int_t roc = det%30;
    Int_t loc;
@@ -1290,11 +1300,15 @@ AliTRDtrapConfig::TrapReg_t AliTRDtrapConfig::GetRegByAddress(Int_t address) con
 
 void AliTRDtrapConfig::PrintMemDatx(ostream &os, Int_t addr) const
 {
+  // print the content of the data memory as datx
+
    PrintMemDatx(os, addr, 0, 0, 127);
 }
 
 void AliTRDtrapConfig::PrintMemDatx(ostream &os, Int_t addr, Int_t det, Int_t rob, Int_t mcm) const
 {
+  // print the content of the data memory as datx
+
    if(addr < fgkDmemStartAddress || addr >= fgkDmemStartAddress+fgkDmemWords) {
       AliError(Form("Invalid DMEM address 0x%08x!", addr));
       return;
@@ -1305,12 +1319,16 @@ void AliTRDtrapConfig::PrintMemDatx(ostream &os, Int_t addr, Int_t det, Int_t ro
 
 void AliTRDtrapConfig::PrintMemDatx(ostream &os, TrapReg_t reg) const
 {
+  // print the content of the data memory as datx
+
    PrintMemDatx(os, reg, 0, 0, 127);
 }
 
 
 void AliTRDtrapConfig::PrintMemDatx(ostream &os, TrapReg_t reg, Int_t det, Int_t rob, Int_t mcm) const
 {
+  // print the content of the data memory as datx
+
    if(reg>= kLastReg) {
       AliError(Form("Invalid register %i!", reg));
       return;
@@ -1321,6 +1339,8 @@ void AliTRDtrapConfig::PrintMemDatx(ostream &os, TrapReg_t reg, Int_t det, Int_t
 
 void AliTRDtrapConfig::PrintDatx(ostream &os, UInt_t addr, UInt_t data, Int_t rob, Int_t mcm) const
 {
+  // print the value at the given address as datx
+
    os << std::setw(5) << 10
       << std::setw(8) << addr
       << std::setw(12) << data;
index ddf7786..1e0126f 100644 (file)
@@ -500,9 +500,9 @@ class AliTRDtrapConfig : public TObject
   Bool_t AddValues(UInt_t det, UInt_t cmd, UInt_t extali, Int_t addr, UInt_t data);
 
   // DMEM addresses
-  static const Int_t fgkDmemAddrLUTcor0       = 0xC02A;
-  static const Int_t fgkDmemAddrLUTcor1       = 0xC028;
-  static const Int_t fgkDmemAddrLUTnbins      = 0xC029;
+  static const Int_t fgkDmemAddrLUTcor0       = 0xC02A; // address for correction factor 0
+  static const Int_t fgkDmemAddrLUTcor1       = 0xC028; // address for correction factor 1
+  static const Int_t fgkDmemAddrLUTnbins      = 0xC029; // address for number of timebins
 
   static const Int_t fgkDmemAddrLUTStart      = 0xC100; // LUT start address
   static const Int_t fgkDmemAddrLUTEnd        = 0xC3FF; // maximum possible end address for the LUT table
@@ -557,8 +557,8 @@ class AliTRDtrapConfig : public TObject
   UInt_t* fDmem[fgkDmemWords]; // DMEM storage
   Int_t fDmemDepth[fgkDmemWords]; // memory space indicator for fDmem
 
-  static const Int_t fgkDmemSizeEmpty=0;
-  static const   Int_t fgkDmemSizeUniform = 1;
+  static const Int_t fgkDmemSizeEmpty=0; // size if no value stored
+  static const   Int_t fgkDmemSizeUniform = 1; // size for global values
   static const   Int_t fgkDmemSizeSmIndividual = 30*8*16;   // storage for each MCM within one supermodule
   static const   Int_t fgkDmemSizeTotalIndividual = 540*6*8*16;  // one individual value for each and every MCM in the TRD
   static const   Int_t fgkDmemSizeSmRocIndividual = 540; // one individual value for each chamber in TRD
index 7d2ea6c..2e9d251 100644 (file)
@@ -38,7 +38,7 @@ class AliTRDtrapConfigHandler : public TObject {
   void  ConfigureNTimebins(Int_t det);                          // timebins in the drift region
   void  ConfigurePIDcorr(Int_t det);                            // Calculate the mcm individual correction factors for the PID
 
-  Double_t Square(Double_t val) { return val*val; };          // returns the square of a given number
+  Double_t Square(Double_t val) const { return val*val; };          // returns the square of a given number
 
   AliTRDtrapConfigHandler(const AliTRDtrapConfigHandler &h);             // not implemented
   AliTRDtrapConfigHandler &operator=(const AliTRDtrapConfigHandler &h);  // not implemented