Refactoring of handling of constants.
authorphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 10 May 2010 05:51:42 +0000 (05:51 +0000)
committerphille <phille@f7af4fe6-9843-0410-8265-dc069ae4e863>
Mon, 10 May 2010 05:51:42 +0000 (05:51 +0000)
Constants used in array definitions etc.. is
defined in nested namespaces, instead of as class
memeber vraiables/constants.
This allows for porting of HLT/CALO constants
to offline without changing the  interface to the
HLT.

13 files changed:
HLT/CALO/AliHLTCaloConstants.cxx
HLT/CALO/AliHLTCaloConstants.h
HLT/CALO/AliHLTCaloDigitMaker.cxx
HLT/CALO/AliHLTCaloRcuFFTDataStruct.h
HLT/EMCAL/AliHLTEMCALConstants.cxx
HLT/EMCAL/AliHLTEMCALConstants.h
HLT/EMCAL/AliHLTEMCALDigitMakerComponent.cxx
HLT/EMCAL/AliHLTEMCALDigitMakerComponent.h
HLT/EMCAL/AliHLTEMCALMapper.cxx
HLT/EMCAL/AliHLTEMCALMapper.h
HLT/EMCAL/AliHLTEMCALRcuCellAccumulatedEnergyDataStruct.h
HLT/PHOS/AliHLTPHOSConstants.cxx
HLT/PHOS/AliHLTPHOSConstants.h

index 8159861..cd036a1 100644 (file)
 
 #include "AliHLTCaloConstants.h"
 
-ClassImp(AliHLTCaloConstants)
-
 
-// int AliHLTCaloConstants::TEST = 10;
+ClassImp(AliHLTCaloConstants)
 
 
-AliHLTCaloConstants::AliHLTCaloConstants() : //  fgInstance(0),
-                                              fkCELLSTEP(-1),
-                                              fkMAXCELLSTEPETA(-1),
-                                              fkMINCELLSTEPETA(-1),
-                                              fkCELLSTEPPHI(-1),
-                                              fkCELLHEIGHT(-1),
-                                              fkCELLANGLE(-1),
-                                              fkRADLENGTH(-1),
-                                              fkCRITICENERGY(-1),
-                                              fkCJ(-1),
-                                              fkDDLOFFSET(-1),
-                                              fkNZROWSRCU(-1),
-                                              fkNXCOLUMNSRCU(-1),
-                                              fkNZROWSMOD(-1),
-                                              fkNXCOLUMNSMOD(-1),
-                                              fkNMODULES(-1),  
-                                              fkNRCUS(-1),  
-                                              fkNRCUSPERMODULE(-1),
-                                              fkNRCUSPERTOTAL(-1), 
-                                              fkNFEECS(-1), 
-                                              fkNALTROS(4),
-                                              fkNALTROCHANNELS(16),
-                                              fkNBRANCHES(2),
-                                              fkCSPSPERFEE(32),
-                                              fkNDATATYPES(10),
-                                              fkMAXHOSTS(20),
-                                              fkDEFAULTEVENTPORT(42001)
-                                              
-                                       
+AliHLTCaloConstants::AliHLTCaloConstants() :  fkMAXCELLSTEPETA(-1),
+                                             fkMINCELLSTEPETA(-1),
+                                             fkCELLSTEPPHI(-1),
+                                             fkCELLHEIGHT(-1),
+                                             fkCELLANGLE(-1),
+                                             fkRADLENGTH(-1),
+                                             fkCRITICENERGY(-1),
+                                             fkCJ(-1),
+                                             fkCELLSTEP(-1),
+                                             fkDDLOFFSET(-1)
 {
   //Default constructor
 }
@@ -69,20 +48,3 @@ AliHLTCaloConstants::~AliHLTCaloConstants()
   //Default destructor
 }
  
-
-
-/*
-AliHLTCaloConstants*
-AliHLTCaloConstants::Instance()
-{
-  if (fgInstance != 0 )
-    {
-      return fgInstance; 
-    }
-  else
-    {
-      fgInstance = new  AliHLTCaloConstants();
-      return fgInstance; 
-    }
-}
-*/
index 9e4e86b..055f128 100644 (file)
 #ifndef ALIHLTCALOCONSTANTS_H
 #define ALIHLTCALOCONSTANTS_H
 
-// #define  ALTROMAXSAMPLES   1008  /**<The maximum number of samples of the ALTRO*/ 
-// #define  ALTROMAXPRESAMPLES  15  //Constant 
-// #define  NGAINS  2
-// #define  HIGHGAIN  1
-// #define  LOWGAIN  0
-// #define  MAXBINVALUE  1023 //Constant
-// #define  MAXHOSTS 20 // For the onlinedisplay
-// #define  DEFAULTEVENTPORT 42001
+
+namespace CaloHLTConst
+{
+  //Constants related to the ALTRO chip (Common to EMCAL / PHOS )
+  const int MAXBINVALUE = 1023;
+  const int NGAINS         =   2;    
+  const int HIGHGAIN    =   0;
+  const int LOWGAIN     =   1;
+  const int ALTROMAXSAMPLES = 1008;    /**<The maximum number of samples of the ALTRO*/
+  const int ALTROMAXPRESAMPLES = 15;        
+  const int NALTROS        =   4;      /**<Number of ALTROs per frontend card*/
+  const int NALTROCHANNELS =  16;
+
+  //FEE constants common to PHOS EMCAL
+  const int CSPSPERFEE    = 32;
+  const int NBRANCHES      =   2;   
+  
+  namespace EmcalHLTConst
+  {
+    const int NZROWSMOD      =  48;   /**<Number of rows per module*/       
+    const int NXCOLUMNSMOD   =  24;   /**<Number of columns per module*/ 
+    const int NRCUSPERSECTOR = 4;
+    const int NMODULES    =    10;    /**<Number of modules of the EMCAL detector*/
+    const int NRCUSPERMODULE =  2 ;   /**<Number of RCUs per Module*/
+    const int NFEECS         =  9; 
+  };
+
+  namespace PhosHLTConst
+  {
+    const int NZROWSMOD      =  56;   /**<Number of rows per module*/       
+    const int NXCOLUMNSMOD   =  64;   /**<Number of columns per module*/ 
+    const int NMODULES    =    5;     /**<Number of modules of the EMCAL detector*/
+    const int NRCUSPERMODULE =  4 ;   /**<Number of RCUs per Module*/
+    const int NFEECS         =  14;   /**<Number of Frontend cards per branch*/
+  };
+};
 
 
-// #define TEST  AliHLTCaloConstants::GetTEST()
+namespace CALO  =  CaloHLTConst; // just for easier notation
+namespace EMCAL =  CaloHLTConst::EmcalHLTConst;
+namespace PHOS  =  CaloHLTConst::PhosHLTConst;
 
 
 class AliHLTCaloConstants
 {
-
-private:
-  // extern static int TEST;
-  //  const static int TEST = 10;
-  // const int TEST = 10;
-  // int GetTEST() const {  return TEST; };
 public:
-  // const static int fgkTEST = 10;
-  const static int  fgkALTROMAXSAMPLES    =  1008;    /**<The maximum number of samples of the ALTRO*/   
-  const static int  fgkALTROMAXPRESAMPLES =  15;  //Constant 
-  const static int  fgkNGAINS = 2;
-  const static int  fgkHIGHGAIN = 1;
-  const static int  fgkLOWGAIN = 0;
-  const static int  fgkMAXBINVALUE = 1023; //Constant
-  const static int  fgkMAXHOSTS = 20; // For the onlinedisplay
-  const static int  fgkDEFAULTEVENTPORT = 42001;
-
-  //  const static int GetTEST()  {  return fgkTEST; }; 
   AliHLTCaloConstants();
   virtual ~AliHLTCaloConstants();
-   // virtual void InitConstants() = 0; 
+  virtual void InitConstants() = 0; 
+
+  // Common PHOS / EMCAL stuff
+  static  Int_t GetALTROMAXSAMPLES()    { return CALO::ALTROMAXSAMPLES; }; 
+  static  Int_t GetNGAINS()             { return CALO::NGAINS; };      
+  static  Int_t GetHIGHGAIN()           { return CALO::HIGHGAIN; };
+  static  Int_t GetLOWGAIN()          { return CALO::LOWGAIN; }; 
+  static  Int_t GetMAXBINVALUE()      { return CALO::MAXBINVALUE; };   
+  static  Int_t GetCSPSPERFEE()       { return CALO::CSPSPERFEE; };                                    
+  static  Int_t GetNALTROS()           { return CALO::NALTROS; };                                      
+  static  Int_t GetNALTROCHANNELS()    { return CALO::NALTROCHANNELS; };                                       
+  static  Int_t GetNBRANCHES()         { return CALO::NBRANCHES; };    
 
-  static  Int_t GetALTROMAXSAMPLES()    { return  fgkALTROMAXSAMPLES; }; 
-  static  Int_t GetNGAINS()       { return fgkNGAINS; };       
-  static  Int_t GetHIGHGAIN()     { return fgkHIGHGAIN; };
-  static  Int_t GetLOWGAIN()      { return fgkLOWGAIN; }; 
-  static  Int_t GetMAXBINVALUE()  { return fgkMAXBINVALUE; };  
+  // Detector specific stuff
 
+  //  virtual Int_t GetNZROWSRCU() const      = 0; 
+  // virtual Int_t GetNXCOLUMNSRCU() const   = 0;
   
-  Int_t GetNDATATYPES() const { return fkNDATATYPES; } ;                                       
-  Int_t GetCSPSPERFEE() const {  return fkCSPSPERFEE; };                                       
+  // PHOS Only, bad move somewher else, PTH
+  virtual Int_t GetNZROWSRCU() const  { return   56 ; } ; 
+  virtual Int_t GetNXCOLUMNSRCU() const { return 16; } ;
+  // END PHOS Only
+
+  virtual Int_t GetNZROWSMOD() const      = 0; 
+  virtual Int_t GetNXCOLUMNSMOD() const   = 0; 
+  virtual Int_t GetNMODULES() const       = 0; 
+  //  virtual Int_t GetNRCUS() const          = 0;
+  virtual Int_t GetNRCUSPERMODULE() const = 0;
+  //  virtual Int_t GetNRCUSPERTOTAL() const  = 0;
+  virtual Int_t GetNFEECS() const         = 0;
   
-  Int_t GetNZROWSRCU() const { return fkNZROWSRCU;}
-  Int_t GetNXCOLUMNSRCU() const { return fkNXCOLUMNSRCU;} 
-  Int_t GetNZROWSMOD() const { return fkNZROWSMOD;} 
-  Int_t GetNXCOLUMNSMOD() const { return fkNXCOLUMNSMOD;} 
-  Int_t GetNMODULES() const { return fkNMODULES;} 
-  Int_t GetNRCUS() const { return fkNRCUS;} 
-  Int_t GetNRCUSPERMODULE() const { return fkNRCUSPERMODULE;} 
-  Int_t GetNRCUSPERTOTAL() const { return fkNRCUSPERTOTAL;} 
-  Int_t GetNFEECS() const { return fkNFEECS;} 
   Int_t GetDDLOFFSET() const { return fkDDLOFFSET; }
-  Int_t GetNALTROS() {  return fkNALTROS; };                                   
-  Int_t GetNALTROCHANNELS() const { return fkNALTROCHANNELS; };                                        
-  Int_t GetNBRANCHES() const { return fkNBRANCHES; };                                  
-  //   static AliHLTCaloConstants* Instance();
-
-   // EMCAL specific
   Float_t GetCELLSTEP() const { return fkCELLSTEP; }
+  
+  //EMCAL specific, !! Move somewhere else, PTH
   Float_t GetMAXCELLSTEPETA() const { return fkMAXCELLSTEPETA; }  //FR
   Float_t GetMINCELLSTEPETA() const { return fkMINCELLSTEPETA; }  //FR
   Float_t GetCELLSTEPPHI() const { return fkCELLSTEPPHI; }        //FR
@@ -90,19 +109,16 @@ public:
   Float_t GetRADLENGTH() const { return fkRADLENGTH; }        //FR
   Float_t GetCRITICENERGY() const { return fkCRITICENERGY; }        //FR
   Float_t GetCJ() const { return fkCJ;} //FR
-  //  virtual Int_t GetDDLOFFSET() const = 0;
   TString GetDETNAME() { return fkDETNAME; };
-  
-  //  static AliHLTCaloConstants *fgInstance;
-
-  // AliHLTCaloConstants *fgInstance;
-
 
 protected:
-  
-
   TString fkDETNAME;
   Float_t fkCELLSTEP; //Constant
+  Int_t fkDDLOFFSET;   //Constant
+  
+  //EMCAL specific, !! Move somewhere else, PTH
+  // @todo: These variables should be declared constant, doesnt work right now
+  // because the default copy contructor is called somewhere.
   Float_t fkMAXCELLSTEPETA;
   Float_t fkMINCELLSTEPETA;
   Float_t fkCELLSTEPPHI;
@@ -111,42 +127,12 @@ protected:
   Float_t fkRADLENGTH;
   Float_t fkCRITICENERGY;
   Float_t fkCJ;
-  Int_t fkDDLOFFSET;   //Constant
-
-  Int_t fkNZROWSRCU; /**<Number of rows per module*/ 
-  Int_t fkNXCOLUMNSRCU;//Constant
-  Int_t fkNZROWSMOD;  /**<Number of rows per module*/ 
-  Int_t fkNXCOLUMNSMOD;  /**<Number of columns per module*/ 
-  Int_t fkNMODULES;   /**<Number of modules of the EMCAL detector*/
-  Int_t fkNRCUS;   /**<Number of RCUs per Module*/
-  Int_t fkNRCUSPERMODULE;   /**<Number of RCUs per Module*/
-  Int_t fkNRCUSPERTOTAL; /**<Total number of RCUs for EMCAL*/
-  Int_t fkNFEECS;  /**<Number of Frontend cards per branch*/
-
+  
 private:
-  const Int_t fkNALTROS;  /**<Number of ALTROs per frontend card*/
-  const Int_t fkNALTROCHANNELS;  //Constant
-  const Int_t fkNBRANCHES; //Constant
-  const Int_t fkCSPSPERFEE; //Constant
-  const Int_t fkNDATATYPES;          //Constant
-  const Int_t fkMAXHOSTS;                  //Constant
-  const Int_t fkDEFAULTEVENTPORT;          //Constant
-  ClassDef(AliHLTCaloConstants, 1);
+  
+  ClassDef(AliHLTCaloConstants, 1)
 };
 
 
-#define  ALTROMAXSAMPLES    AliHLTCaloConstants::fgkALTROMAXSAMPLES    /**<The maximum number of samples of the ALTRO*/   
-#define  ALTROMAXPRESAMPLES AliHLTCaloConstants::fgkALTROMAXPRESAMPLES  //Constant 
-#define  NGAINS             AliHLTCaloConstants::fgkNGAINS
-#define  HIGHGAIN           AliHLTCaloConstants::fgkHIGHGAIN
-#define  LOWGAIN            AliHLTCaloConstants::fgkLOWGAIN
-#define  MAXBINVALUE        AliHLTCaloConstants::fgkMAXBINVALUE //Constant
-#define  MAXHOSTS           AliHLTCaloConstants::fgkMAXHOSTS // For the onlinedisplay
-#define  DEFAULTEVENTPORT   AliHLTCaloConstants::fgkDEFAULTEVENTPORT
-
-//#define TEST  AliHLTCaloConstants::GetTEST()
-//#define TEST  AliHLTCaloConstants::TEST
-
-
 
 #endif
index ef84e32..2a4eb43 100644 (file)
@@ -167,6 +167,7 @@ AliHLTCaloDigitMaker::SetGlobalHighGainFactor(Float_t factor)
     }\r
 }\r
 \r
+\r
 void\r
 AliHLTCaloDigitMaker::SetGlobalLowGainFactor(Float_t factor)\r
 {\r
@@ -180,6 +181,7 @@ AliHLTCaloDigitMaker::SetGlobalLowGainFactor(Float_t factor)
     }\r
 }\r
 \r
+\r
 void\r
 AliHLTCaloDigitMaker::SetBadChannelMask(TH2F* badChannelHGHist, TH2F* badChannelLGHist, Float_t qCut)\r
 {\r
index a820326..82a63ac 100644 (file)
@@ -4,13 +4,24 @@
 #ifndef ALIHLTCALORCUFFTDATASTRUCT_H
 #define ALIHLTCALORCUFFTDATASTRUCT_H
 #include "Rtypes.h"
+
+//#include "AliHLTCaloConstants.h"
+
 #include "AliHLTCaloConstants.h"
 
+// using AliHLTCaloConstants::TEST1;
+// using AliHLTCaloConstants::TEST2;
+
+using CALO::NGAINS;
+using CALO::ALTROMAXSAMPLES;
 
 struct AliHLTCaloRcuFFTDataStruct
 {
   int fDataLength;
-  //   Double_t fTest [TEST];
+  
+  //  Double_t fTest1[AliHLTCaloConstants::TEST1];
+  //  Double_t fTest2[AliHLTCaloConstants::TEST2];
+
   Double_t fGlobalAccumulatedPSD[NGAINS][ALTROMAXSAMPLES];
   Double_t fGlobalLastPSD[NGAINS][ALTROMAXSAMPLES];
 };
index 7507c09..229dc7d 100644 (file)
 /// @brief  Class containing constants for EMCAL and EMCAL
 ///         loaded libraries
 
-#include "AliHLTCaloConstants.h"
+
+//#include "AliHLTCaloConstants.h"
 #include "AliHLTEMCALConstants.h"
 
-ClassImp(AliHLTEMCALConstants);
+ClassImp(AliHLTEMCALConstants)
 
 AliHLTEMCALConstants::AliHLTEMCALConstants() : AliHLTCaloConstants()
 {
@@ -42,27 +43,10 @@ AliHLTEMCALConstants::~AliHLTEMCALConstants()
 
 
 void 
-AliHLTEMCALConstants::InitConstants()
+AliHLTEMCALConstants::InitConstants() 
 {
-  fkNZROWSRCU =  56;
-  fkNXCOLUMNSRCU = 16;
-  fkNZROWSMOD = 48;
-  fkNXCOLUMNSMOD = 24;
-  fkNMODULES = 10;
-  fkNRCUS = 4;
-  fkNRCUSPERMODULE = 2;
-  fkNRCUSPERTOTAL = fkNMODULES*fkNRCUSPERMODULE;
-  fkNFEECS = 9;
   fkDETNAME = "EMCAL";
   fkCELLSTEP = 6.0;
-  fkMAXCELLSTEPETA =6.32;      // FR: max tower dimension along eta
-  fkMINCELLSTEPETA = 5.99;     // FR: min tower dimension along eta
-  fkCELLSTEPPHI = 6.04667;     // FR: tower dimension along phi
-  fkCELLHEIGHT =  27.74;       // FR: tower height
-  fkCELLANGLE  =  1.50;                // FR: tower tapeiring angle (DEG)
-  fkRADLENGTH  =  1.23;
-  fkCRITICENERGY =  8;
-  fkCJ  =   0.5;
   fkDDLOFFSET =  4608;
 }
                                   
index f072020..9ecd977 100644 (file)
@@ -22,7 +22,8 @@
 #ifndef ALIHLTEMCALCONSTANTS_H
 #define ALIHLTEMCALCONSTANTS_H
 
-class AliHLTCaloConstants;
+
+#include "AliHLTCaloConstants.h"
 
 
 class AliHLTEMCALConstants : public AliHLTCaloConstants
@@ -31,7 +32,15 @@ class AliHLTEMCALConstants : public AliHLTCaloConstants
 public:
   AliHLTEMCALConstants();
   ~AliHLTEMCALConstants();
+  virtual Int_t GetNZROWSMOD() const      { return EMCAL::NZROWSMOD;} 
+  virtual Int_t GetNXCOLUMNSMOD() const   { return EMCAL::NXCOLUMNSMOD;}; 
+  virtual Int_t GetNMODULES() const       { return EMCAL::NMODULES; }; 
+  virtual Int_t GetNRCUSPERMODULE() const { return EMCAL::NRCUSPERMODULE;};
+  virtual Int_t GetNFEECS() const         { return EMCAL::NFEECS; } ;
   virtual void InitConstants(); 
+private:
+  
   ClassDef(AliHLTEMCALConstants, 1)
 };
 
index 2a54164..2f56915 100644 (file)
 #include <sys/stat.h>
 #include <sys/types.h>
 
+//#include "AliHLTEMCALConstant.h"
+#include "AliHLTCaloConstants.h"
+
+using EMCAL::NZROWSMOD;
+using EMCAL::NXCOLUMNSMOD;  
+
 
 /** 
  * @file   AliHLTEMCALDigitMakerComponent.cxx
@@ -52,7 +58,7 @@ AliHLTEMCALDigitMakerComponent gAliHLTEMCALDigitMakerComponent;
 
 AliHLTEMCALDigitMakerComponent::AliHLTEMCALDigitMakerComponent() :
   AliHLTCaloProcessor(),
-  AliHLTCaloConstantsHandler("EMCAL"),
+  //  AliHLTCaloConstantsHandler("EMCAL"),
   fDigitMakerPtr(0),
   fDigitContainerPtr(0),
   fPedestalData(0),
@@ -60,6 +66,7 @@ AliHLTEMCALDigitMakerComponent::AliHLTEMCALDigitMakerComponent() :
   fBCMInitialised(true),
   fGainsInitialised(true)
 {
+  
   //see header file for documentation
 }
 
@@ -145,14 +152,17 @@ AliHLTEMCALDigitMakerComponent::DoEvent(const AliHLTComponentEventData& evtData,
        {
          continue;
        }
+      
       if(!fBCMInitialised)
       {
         AliHLTEMCALMapper mapper(iter->fSpecification);
         Int_t module = mapper.GetModuleFromSpec(iter->fSpecification);
-        for(Int_t x = 0; x < fCaloConstants->GetNXCOLUMNSMOD(); x++)
+        //      for(Int_t x = 0; x < fCaloConstants->GetNXCOLUMNSMOD(); x++)
+        for(Int_t x = 0; x < NXCOLUMNSMOD ; x++) // PTH  
         {
-           for(Int_t z = 0; z < fCaloConstants->GetNZROWSMOD(); z++)
-           {
+          //   for(Int_t z = 0; z < fCaloConstants->GetNZROWSMOD(); z++)
+          for(Int_t z = 0; z <  NZROWSMOD ; z++) // PTH
+             {
               fDigitMakerPtr->SetBadChannel(x, z, fPedestalData->IsBadChannel(module, z+1, x+1));
            }
         }
@@ -163,10 +173,12 @@ AliHLTEMCALDigitMakerComponent::DoEvent(const AliHLTComponentEventData& evtData,
       {
         AliHLTEMCALMapper mapper(iter->fSpecification);;
         Int_t module = mapper.GetModuleFromSpec(iter->fSpecification);
-        for(Int_t x = 0; x < fCaloConstants->GetNXCOLUMNSMOD(); x++)
-        {
-           for(Int_t z = 0; z < fCaloConstants->GetNZROWSMOD(); z++)
-           {
+        //      for(Int_t x = 0; x < fCaloConstants->GetNXCOLUMNSMOD(); x++)
+        for(Int_t x = 0; x < NXCOLUMNSMOD; x++)   //PTH
+          {
+            //     for(Int_t z = 0; z < fCaloConstants->GetNZROWSMOD(); z++)
+            for(Int_t z = 0; z < NZROWSMOD; z++)   //PTH
+              {
               module = 0; //removing warning
                //fDigitMakerPtr->SetGain(x, z, fCalibData->GE(module, z+1, x+1), fCalibData->GetADCchannelEmc(module, z+1, x+1));
            }
@@ -235,14 +247,12 @@ AliHLTEMCALDigitMakerComponent::DoInit(int argc, const char** argv )
   return 0;
 }
 
+
 int AliHLTEMCALDigitMakerComponent::GetBCMFromCDB()
 {
    // See header file for class documentation
-   
-     fBCMInitialised = false;
-   
-//   HLTInfo("Getting bad channel map...");
-
+  fBCMInitialised = false;
+  //   HLTInfo("Getting bad channel map...");
   AliCDBPath path("EMCAL","Calib","Pedestals");
   if(path.GetPath())
     {
@@ -266,13 +276,12 @@ int AliHLTEMCALDigitMakerComponent::GetBCMFromCDB()
    return 0;
 }
 
+
 int AliHLTEMCALDigitMakerComponent::GetGainsFromCDB()
 {
    // See header file for class documentation
-   
-     fGainsInitialised = false;
-   
-//   HLTInfo("Getting bad channel map...");
+  fGainsInitialised = false;
+  //  HLTInfo("Getting bad channel map...");
 
   AliCDBPath path("EMCAL","Calib","Data");
   if(path.GetPath())
@@ -290,14 +299,10 @@ int AliHLTEMCALDigitMakerComponent::GetGainsFromCDB()
        }
     }
    if(!fCalibData) return -1;
-
-
-
    return 0;
 }
 
 
-
 AliHLTComponent*
 AliHLTEMCALDigitMakerComponent::Spawn()
 {
index c9ef4ef..7581366 100644 (file)
@@ -31,7 +31,8 @@
 // visit http://web.ift.uib.no/~kjeks/doc/alice-hlt
 
 #include "AliHLTCaloProcessor.h"
-#include "AliHLTCaloConstantsHandler.h"
+
+//#include "AliHLTCaloConstantsHandler.h"
 
 class AliEMCALCalibData;
 class AliCaloCalibPedestal;
@@ -86,7 +87,8 @@ class AliHLTCaloDigitContainerDataStruct;
  * @ingroup alihlt_emcal
  */ 
 
-class AliHLTEMCALDigitMakerComponent : public AliHLTCaloProcessor, public AliHLTCaloConstantsHandler
+//class AliHLTEMCALDigitMakerComponent : public AliHLTCaloProcessor, public AliHLTCaloConstantsHandler
+class AliHLTEMCALDigitMakerComponent : public AliHLTCaloProcessor
 {
 public:
 
index fd2ae4d..efda341 100644 (file)
  * about the suitability of this software for any purpose. It is          *
  * provided "as is" without express or implied warranty.                  *
  **************************************************************************/
+
 #include "AliHLTEMCALMapper.h"
 
-#include "AliHLTEMCALConstant.h"
+//#include "AliHLTEMCALConstant.h"
+// #include "AliHLTCaloConstant.h"
 
 #include "assert.h"
 
-using namespace EmcalHLTConst;
-
+//using namespace EmcalHLTConst;
 
+#include "AliHLTCaloConstants.h"
 
+using EMCAL::NXCOLUMNSMOD;
+using EMCAL::NZROWSMOD;
+using EMCAL::NMODULES;
+using EMCAL::NRCUSPERMODULE;
+using EMCAL::NRCUSPERSECTOR;
 
 AliHLTEMCALMapper::AliHLTEMCALMapper(const unsigned long specification ) : AliHLTCaloMapper(specification, "EMCAL")
 {
@@ -110,11 +117,11 @@ AliHLTEMCALMapper::InitAltroMapping(const unsigned long specification )
 void 
 AliHLTEMCALMapper::InitDDLSpecificationMapping()
 {
-  fSpecificationMapPtr = new fDDLSpecificationMap[EmcalHLTConst::NMODULES*EmcalHLTConst::NRCUSPERMODULE];
+  fSpecificationMapPtr = new fDDLSpecificationMap[NMODULES*NRCUSPERMODULE];
   
-  for(Int_t ddl = 0; ddl < EmcalHLTConst::NMODULES*EmcalHLTConst::NRCUSPERMODULE; ddl++)
+  for(Int_t ddl = 0; ddl < NMODULES*NRCUSPERMODULE; ddl++)
     {
-      fSpecificationMapPtr[ddl].fModId = ddl/(EmcalHLTConst::NRCUSPERMODULE);
+      fSpecificationMapPtr[ddl].fModId = ddl/( NRCUSPERMODULE );
     }
 }
 
index 4c955d8..d4fe773 100644 (file)
@@ -20,7 +20,6 @@
  **************************************************************************/
 
 
-//#include  "AliHLTCaloMapper.h"
 #include  "AliHLTCaloMapper.h"
 
 
@@ -32,14 +31,7 @@ class  AliHLTEMCALMapper : public AliHLTCaloMapper
   virtual Bool_t InitAltroMapping( const unsigned long specification ); 
   virtual void InitDDLSpecificationMapping();
   virtual void GetLocalCoord(const int channelId, Float_t* localCoord) const; 
-  //  static  const int GlobalX2ModuleId(const int globalX);
-  //  static  const int GlobalZ2ModuleId(const int globalZ);
-
-  // static  const int Global2ModuleId( const int globalZ,  const int globalX );
-
-  //  const virtual int GetDDLFromSpec( const AliHLTUInt32_t spec );
-  //  const virtual int GetChannelID(const AliHLTUInt32_t spec, const Int_t hadd);
-
+  
  private:
   AliHLTEMCALMapper();
   AliHLTEMCALMapper(const AliHLTEMCALMapper & );
index ece0366..f3289dc 100644 (file)
  * provided "as is" without express or implied warranty.                   *
  **************************************************************************/
 
+//#include "AliHLTEMCALConstant.h"
+//using namespace EmcalHLTConst;
 
-#include "AliHLTEMCALConstant.h"
 
-using namespace EmcalHLTConst;
+#include "AliHLTCaloConstant.h"
+
+using EMCAL::NXCOLUMNSMOD;
+using EMCAL::NZROWSMOD;
+using EMCAL::NGAINS;
 
 struct AliHLTCaloRcuCellAccumulatedEnergyDataStruct
 {
@@ -32,16 +37,6 @@ struct AliHLTCaloRcuCellAccumulatedEnergyDataStruct
   float fAccumulatedEnergies[NXCOLUMNSMOD][NZROWSMOD][NGAINS];
   AliHLTUInt32_t fHits[NXCOLUMNSMOD][NZROWSMOD][NGAINS];
   AliHLTUInt32_t fDeadChannelMap[NXCOLUMNSMOD][NZROWSMOD][NGAINS]; 
-  
-  /*
-  AliHLTUInt8_t fModuleID;
-  AliHLTUInt8_t fRcuX;
-  AliHLTUInt8_t fRcuZ; 
-  float fAccumulatedEnergies[NXCOLUMNSMOD][NZROWSMOD][NGAINS];
-  AliHLTUInt32_t fHits[NXCOLUMNSMOD][NZROWSMOD][NGAINS];
-  AliHLTUInt32_t fDeadChannelMap[NXCOLUMNSMOD][NZROWSMOD][NGAINS];
-  */  
-
 };
 
 
index 1336087..0614ec9 100644 (file)
@@ -25,7 +25,9 @@
 
 #include "AliHLTPHOSConstants.h"
 
-ClassImp(AliHLTPHOSConstants);
+
+ClassImp(AliHLTPHOSConstants)
+
 
 AliHLTPHOSConstants::AliHLTPHOSConstants() : AliHLTCaloConstants()
 {
@@ -42,15 +44,6 @@ AliHLTPHOSConstants::~AliHLTPHOSConstants()
 void 
 AliHLTPHOSConstants::InitConstants()
 {
-  fkNZROWSRCU = 56;
-  fkNXCOLUMNSRCU = 16;
-  fkNZROWSMOD = 56;
-  fkNXCOLUMNSMOD = 64;
-  fkNMODULES= 5;
-  fkNRCUS = 4; 
-  fkNRCUSPERMODULE = 4;
-  fkNRCUSPERTOTAL = fkNMODULES*fkNRCUSPERMODULE;
-  fkNFEECS = 14;
   fkDETNAME = "PHOS";
   fkCELLSTEP = 2.255;
   fkDDLOFFSET = 1792;
index 1794fb7..4eb20f5 100644 (file)
 #include <TString.h>
 #include "Rtypes.h"
 
-#include "AliHLTCaloConstants.h"
 
-//class AliHLTCaloConstants;
+#include "AliHLTCaloConstants.h"
 
 
 class AliHLTPHOSConstants : public AliHLTCaloConstants
 {
 public:
   AliHLTPHOSConstants();
-
-  ~AliHLTPHOSConstants();
-  
+  virtual ~AliHLTPHOSConstants();
   virtual void InitConstants(); 
-
-  // private:
+  virtual Int_t GetNZROWSMOD() const      { return PHOS::NZROWSMOD;} 
+  virtual Int_t GetNXCOLUMNSMOD() const   { return PHOS::NXCOLUMNSMOD;}; 
+  virtual Int_t GetNMODULES() const       { return PHOS::NMODULES; }; 
+  virtual Int_t GetNRCUSPERMODULE() const { return PHOS::NRCUSPERMODULE;};
+  virtual Int_t GetNFEECS() const         { return PHOS::NFEECS; } ;
+  
   ClassDef(AliHLTPHOSConstants, 1)
 };