]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - EMCAL/AliEMCALRecPoint.cxx
avoid warnings with modern compilers due to unused variables or wrong comparison...
[u/mrichter/AliRoot.git] / EMCAL / AliEMCALRecPoint.cxx
index b4d94fc38d868491f2b5ad33cf4dd9d82486845b..0b452642fc1f7f350c8ea51166eac5e299434988 100644 (file)
@@ -16,6 +16,8 @@
 //_________________________________________________________________________
 //  Reconstructed Points for the EMCAL
 //  A RecPoint is a cluster of digits
+//  
+//  
 //*-- Author: Yves Schutz (SUBATECH)
 //*-- Author: Dmitri Peressounko (RRC KI & SUBATECH)
 //*-- Author: Heather Gray (LBL) merged AliEMCALRecPoint and AliEMCALTowerRecPoint 02/04
@@ -29,6 +31,7 @@
 #include "TGeoMatrix.h"
 #include "TGeoManager.h"
 #include "TGeoPhysicalNode.h"
+#include "TRandom.h"
 
 // --- Standard library ---
 #include <Riostream.h>
@@ -43,6 +46,8 @@ class AliEMCAL;
 #include "AliEMCALHit.h"
 #include "AliEMCALDigit.h"
 #include "AliEMCALRecPoint.h"
+#include "AliCaloCalibPedestal.h"
+#include "AliEMCALGeoParams.h"
 
 ClassImp(AliEMCALRecPoint)
 
@@ -50,15 +55,15 @@ ClassImp(AliEMCALRecPoint)
 AliEMCALRecPoint::AliEMCALRecPoint()
   : AliCluster(), fGeomPtr(0),
     fAmp(0), fIndexInList(-1), //to be set when the point is already stored
-    fLocPos(0,0,0), fLocPosM(0),
+    fGlobPos(0,0,0),fLocPos(0,0,0), 
     fMaxDigit(100), fMulDigit(0), fMaxTrack(200),
     fMulTrack(0), fDigitsList(0), fTracksList(0),
     fClusterType(-1), fCoreEnergy(0), fDispersion(0),
-    fEnergyList(0), fTimeList(0), fAbsIdList(0),
+    fEnergyList(0), fAbsIdList(0),
     fTime(0.), fNExMax(0), fCoreRadius(10),  //HG check this 
     fDETracksList(0), fMulParent(0), fMaxParent(0),
     fParentsList(0), fDEParentsList(0), fSuperModuleNumber(0),
-    fDigitIndMax(-1)
+    fDigitIndMax(-1), fDistToBadTower(-1), fSharedCluster(kFALSE)
 {
   // ctor
   fGeomPtr = AliEMCALGeometry::GetInstance();
@@ -72,15 +77,15 @@ AliEMCALRecPoint::AliEMCALRecPoint()
 AliEMCALRecPoint::AliEMCALRecPoint(const char *) 
   : AliCluster(), fGeomPtr(0),
     fAmp(0), fIndexInList(-1), //to be set when the point is already stored
-    fLocPos(0,0,0), fLocPosM(new TMatrixF(3,3)),
+    fGlobPos(0,0,0), fLocPos(0,0,0),
     fMaxDigit(100), fMulDigit(0), fMaxTrack(1000), fMulTrack(0),
     fDigitsList(new Int_t[fMaxDigit]), fTracksList(new Int_t[fMaxTrack]),
     fClusterType(-1), fCoreEnergy(0), fDispersion(0),
-    fEnergyList(new Float_t[fMaxDigit]), fTimeList(new Float_t[fMaxDigit]), 
+    fEnergyList(new Float_t[fMaxDigit]), 
     fAbsIdList(new Int_t[fMaxDigit]), fTime(-1.), fNExMax(0), fCoreRadius(10),
     fDETracksList(new Float_t[fMaxTrack]), fMulParent(0), fMaxParent(1000),
     fParentsList(new Int_t[fMaxParent]), fDEParentsList(new Float_t[fMaxParent]),
-    fSuperModuleNumber(0), fDigitIndMax(-1)
+    fSuperModuleNumber(0), fDigitIndMax(-1), fDistToBadTower(-1),fSharedCluster(kFALSE)
 {
   // ctor
   for (Int_t i = 0; i < fMaxTrack; i++)
@@ -99,18 +104,19 @@ AliEMCALRecPoint::AliEMCALRecPoint(const char *)
 AliEMCALRecPoint::AliEMCALRecPoint(const AliEMCALRecPoint & rp) 
   : AliCluster(rp), fGeomPtr(rp.fGeomPtr),
     fAmp(rp.fAmp), fIndexInList(rp.fIndexInList),
-    fLocPos(rp.fLocPos), fLocPosM(rp.fLocPosM),
+    fGlobPos(rp.fGlobPos),fLocPos(rp.fLocPos),
     fMaxDigit(rp.fMaxDigit), fMulDigit(rp.fMulDigit),
     fMaxTrack(rp.fMaxTrack), fMulTrack(rp.fMaxTrack),
     fDigitsList(new Int_t[rp.fMaxDigit]), fTracksList(new Int_t[rp.fMaxTrack]),
     fClusterType(rp.fClusterType), fCoreEnergy(rp.fCoreEnergy), 
     fDispersion(rp.fDispersion),
-    fEnergyList(new Float_t[rp.fMaxDigit]), fTimeList(new Float_t[rp.fMaxDigit]), 
+    fEnergyList(new Float_t[rp.fMaxDigit]),  
     fAbsIdList(new Int_t[rp.fMaxDigit]), fTime(rp.fTime), fNExMax(rp.fNExMax),fCoreRadius(rp.fCoreRadius),
     fDETracksList(new Float_t[rp.fMaxTrack]), fMulParent(rp.fMulParent), 
     fMaxParent(rp.fMaxParent), fParentsList(new Int_t[rp.fMaxParent]), 
     fDEParentsList(new Float_t[rp.fMaxParent]),
-    fSuperModuleNumber(rp.fSuperModuleNumber), fDigitIndMax(rp.fDigitIndMax)
+    fSuperModuleNumber(rp.fSuperModuleNumber), fDigitIndMax(rp.fDigitIndMax), 
+    fDistToBadTower(rp.fDistToBadTower), fSharedCluster(rp.fSharedCluster)
 {
   //copy ctor
   fLambda[0] = rp.fLambda[0];
@@ -118,8 +124,7 @@ AliEMCALRecPoint::AliEMCALRecPoint(const AliEMCALRecPoint & rp)
 
   for(Int_t i = 0; i < rp.fMulDigit; i++) {
     fEnergyList[i] = rp.fEnergyList[i];
-    fTimeList[i] = rp.fTimeList[i];
-    fAbsIdList[i] = rp.fAbsIdList[i];
+    fAbsIdList[i]  = rp.fAbsIdList[i];
   }
 
   for(Int_t i = 0; i < rp.fMulTrack; i++) fDETracksList[i] = rp.fDETracksList[i];
@@ -136,8 +141,6 @@ AliEMCALRecPoint::~AliEMCALRecPoint()
   // dtor
   if ( fEnergyList )
     delete[] fEnergyList ; 
-  if ( fTimeList )
-    delete[] fTimeList ; 
   if ( fAbsIdList )
     delete[] fAbsIdList ; 
    if ( fDETracksList)
@@ -146,8 +149,7 @@ AliEMCALRecPoint::~AliEMCALRecPoint()
     delete[] fParentsList;
    if ( fDEParentsList)
     delete[] fDEParentsList;
-
-   delete fLocPosM ;
+       
    delete [] fDigitsList ;
    delete [] fTracksList ;
 }
@@ -159,111 +161,125 @@ AliEMCALRecPoint& AliEMCALRecPoint::operator= (const AliEMCALRecPoint &rp)
 
   if(&rp == this) return *this;
 
-  fGeomPtr = rp.fGeomPtr;
-  fAmp = rp.fAmp;
+  fGeomPtr     = rp.fGeomPtr;
+  fAmp         = rp.fAmp;
   fIndexInList = rp.fIndexInList;
-  fLocPos = rp.fLocPos;
-  fLocPosM = rp.fLocPosM;
-  fMaxDigit = rp.fMaxDigit;
-  fMulDigit = rp.fMulDigit;
-  fMaxTrack = rp.fMaxTrack;
-  fMulTrack = rp.fMaxTrack;
+  fGlobPos     = rp.fGlobPos;
+  fLocPos      = rp.fLocPos;
+  fMaxDigit    = rp.fMaxDigit;
+  fMulDigit    = rp.fMulDigit;
+  fMaxTrack    = rp.fMaxTrack;
+  fMulTrack    = rp.fMulTrack;
+  
+  if(fDigitsList) delete [] fDigitsList;
+  fDigitsList = new Int_t[rp.fMaxDigit];
+  if(fTracksList) delete [] fTracksList;
+  fTracksList = new Int_t[rp.fMaxTrack];
   for(Int_t i = 0; i<fMaxDigit; i++) fDigitsList[i] = rp.fDigitsList[i];
   for(Int_t i = 0; i<fMaxTrack; i++) fTracksList[i] = rp.fTracksList[i];
+  
   fClusterType = rp.fClusterType;
-  fCoreEnergy = rp.fCoreEnergy; 
-  fDispersion = rp.fDispersion;
+  fCoreEnergy  = rp.fCoreEnergy; 
+  fDispersion  = rp.fDispersion;
+  
+  
+  if(fEnergyList) delete [] fEnergyList;
+  fEnergyList = new Float_t[rp.fMaxDigit];
+  if(fAbsIdList) delete [] fAbsIdList;
+  fAbsIdList = new Int_t[rp.fMaxDigit];  
   for(Int_t i = 0; i<fMaxDigit; i++) {
     fEnergyList[i] = rp.fEnergyList[i];
-    fTimeList[i] = rp.fTimeList[i]; 
-    fAbsIdList[i] = rp.fAbsIdList[i];
+    fAbsIdList[i]  = rp.fAbsIdList[i];
   }
-  fTime = rp.fTime;
-  fNExMax = rp.fNExMax;
+  
+  fTime       = rp.fTime;
+  fNExMax     = rp.fNExMax;
   fCoreRadius = rp.fCoreRadius;
+  
+  if(fDETracksList) delete [] fDETracksList;
+  fDETracksList = new Float_t[rp.fMaxTrack];
   for(Int_t i = 0; i < fMaxTrack; i++) fDETracksList[i] = rp.fDETracksList[i];
+
   fMulParent = rp.fMulParent;
   fMaxParent = rp.fMaxParent;
+  
+  if(fParentsList) delete [] fParentsList;
+  fParentsList = new Int_t[rp.fMaxParent];
+  if(fDEParentsList) delete [] fDEParentsList;
+  fDEParentsList = new Float_t[rp.fMaxParent];
   for(Int_t i = 0; i < fMaxParent; i++) {
-    fParentsList[i] = rp.fParentsList[i]; 
+    fParentsList[i]   = rp.fParentsList[i]; 
     fDEParentsList[i] = rp.fDEParentsList[i];
   }
+  
   fSuperModuleNumber = rp.fSuperModuleNumber;
-  fDigitIndMax = rp.fDigitIndMax;
+  fDigitIndMax       = rp.fDigitIndMax;
 
   fLambda[0] = rp.fLambda[0];
   fLambda[1] = rp.fLambda[1];
-
+       
+  fDistToBadTower = rp.fDistToBadTower;
+  fSharedCluster  = rp.fSharedCluster;
+       
   return *this;
 
 }
 
 //____________________________________________________________________________
-void AliEMCALRecPoint::AddDigit(AliEMCALDigit & digit, Float_t Energy)
+void AliEMCALRecPoint::AddDigit(AliEMCALDigit & digit, const Float_t energy, const Bool_t shared)
 {
   // Adds a digit to the RecPoint
   // and accumulates the total amplitude and the multiplicity 
   
   if(fEnergyList == 0)
     fEnergyList =  new Float_t[fMaxDigit]; 
-  if(fTimeList == 0)
-    fTimeList =  new Float_t[fMaxDigit]; 
   if(fAbsIdList == 0) {
-    fAbsIdList =  new Int_t[fMaxDigit];
+    fAbsIdList  =  new Int_t  [fMaxDigit];
   }
 
   if ( fMulDigit >= fMaxDigit ) { // increase the size of the lists 
     fMaxDigit*=2 ; 
-    Int_t * tempo = new Int_t[fMaxDigit]; 
-    Float_t * tempoE =  new Float_t[fMaxDigit];
-    Float_t * tempoT =  new Float_t[fMaxDigit];
-    Int_t * tempoId = new Int_t[fMaxDigit]; 
+    Int_t   * tempo   = new Int_t  [fMaxDigit]; 
+    Float_t * tempoE  = new Float_t[fMaxDigit];
+    Int_t   * tempoId = new Int_t  [fMaxDigit]; 
 
     Int_t index ;     
     for ( index = 0 ; index < fMulDigit ; index++ ){
-      tempo[index]   = fDigitsList[index] ;
-      tempoE[index]  = fEnergyList[index] ; 
-      tempoT[index]  = fTimeList[index] ; 
-      tempoId[index] = fAbsIdList[index] ; 
+      tempo  [index] = fDigitsList[index] ;
+      tempoE [index] = fEnergyList[index] ; 
+      tempoId[index] = fAbsIdList [index] ; 
     }
     
     delete [] fDigitsList ;
     delete [] fEnergyList ;
-    delete [] fTimeList ;
     delete [] fAbsIdList ;
 
     fDigitsList = tempo;
     fEnergyList = tempoE; 
-    fTimeList = tempoT;
-    fAbsIdList = tempoId;
+    fAbsIdList  = tempoId;
   } // if
   
   fDigitsList[fMulDigit]   = digit.GetIndexInList()  ; 
-  fEnergyList[fMulDigit]   = Energy ;
-  fTimeList[fMulDigit]     = digit.GetTimeR() ;
-  fAbsIdList[fMulDigit]    = digit.GetId();
+  fEnergyList[fMulDigit]   = energy ;
+  fAbsIdList [fMulDigit]   = digit.GetId();
   fMulDigit++ ; 
-  fAmp += Energy ; 
-
-  //JLK 10-Oct-2007 this hasn't been filled before because it was in
-  //the wrong place in previous versions.
-  //Now we evaluate it only if the supermodulenumber for this recpoint
-  //has not yet been set (or is the 0th one)
-  if(fSuperModuleNumber == 0)
-    fSuperModuleNumber = fGeomPtr->GetSuperModuleNumber(digit.GetId());
-
+  fAmp += energy ; 
+       
+  if(shared) fSharedCluster = kTRUE;
 }
 //____________________________________________________________________________
 Bool_t AliEMCALRecPoint::AreNeighbours(AliEMCALDigit * digit1, AliEMCALDigit * digit2 ) const
 {
   // Tells if (true) or not (false) two digits are neighbours
   // A neighbour is defined as being two digits which share a corner
-  
-  static Bool_t areNeighbours = kFALSE ;
-  static Int_t nSupMod=0, nModule=0, nIphi=0, nIeta=0;
-  static int nSupMod1=0, nModule1=0, nIphi1=0, nIeta1=0;
-  static Int_t relid1[2] , relid2[2] ; // ieta, iphi
-  static Int_t rowdiff=0, coldiff=0;
+  // ONLY USED IN CASE OF UNFOLDING 
+       
+  Bool_t areNeighbours = kFALSE ;
+  Int_t nSupMod=0, nModule=0, nIphi=0, nIeta=0;
+  Int_t nSupMod1=0, nModule1=0, nIphi1=0, nIeta1=0;
+  Int_t relid1[2] , relid2[2] ; // ieta, iphi
+  Int_t rowdiff=0, coldiff=0;
 
   areNeighbours = kFALSE ;
 
@@ -273,11 +289,23 @@ Bool_t AliEMCALRecPoint::AreNeighbours(AliEMCALDigit * digit1, AliEMCALDigit * d
   fGeomPtr->GetCellIndex(digit2->GetId(), nSupMod1,nModule1,nIphi1,nIeta1);
   fGeomPtr->GetCellPhiEtaIndexInSModule(nSupMod1,nModule1,nIphi1,nIeta1, relid2[0],relid2[1]);
   
+  // In case of a shared cluster, index of SM in C side, columns start at 48 and ends at 48*2-1
+  // C Side impair SM, nSupMod%2=1; A side pair SM nSupMod%2=0
+  if(fSharedCluster){
+    //printf("Shared cluster in 2 SMs!\n");
+
+    //    if(nSupMod1%2) relid1[1]+=AliEMCALGeoParams::fgkEMCALCols;//bad
+    //    else           relid2[1]+=AliEMCALGeoParams::fgkEMCALCols;//bad
+    if(nSupMod1%2) relid2[1]+=AliEMCALGeoParams::fgkEMCALCols;
+    else           relid1[1]+=AliEMCALGeoParams::fgkEMCALCols;
+  }
+       
   rowdiff = TMath::Abs( relid1[0] - relid2[0] ) ;  
   coldiff = TMath::Abs( relid1[1] - relid2[1] ) ;  
 
-  if (( coldiff <= 1 )  && ( rowdiff <= 1 ) && (coldiff + rowdiff > 0)) 
-  areNeighbours = kTRUE ;
+  //if (( coldiff <= 1 )  && ( rowdiff <= 1 ) && (coldiff + rowdiff > 0)) 
+  if ((coldiff + rowdiff == 1 )) 
+    areNeighbours = kTRUE ;
   
   return areNeighbours;
 }
@@ -287,10 +315,9 @@ Int_t AliEMCALRecPoint::Compare(const TObject * obj) const
 {
   // Compares two RecPoints according to their position in the EMCAL modules
 
-  Float_t delta = 1 ; //Width of "Sorting row". If you change this 
-                      //value (what is senseless) change as well delta in
-                      //AliEMCALTrackSegmentMakerv* and other RecPoints...
-  Int_t rv ; 
+  Float_t delta = 1 ; //Width of "Sorting row". 
+       
+  Int_t rv = 2 ; 
 
   AliEMCALRecPoint * clu = (AliEMCALRecPoint *)obj ; 
 
@@ -312,27 +339,6 @@ Int_t AliEMCALRecPoint::Compare(const TObject * obj) const
   return rv ; 
 }
 
-//____________________________________________________________________________
-Int_t AliEMCALRecPoint::DistancetoPrimitive(Int_t px, Int_t py)
-{
-  // Compute distance from point px,py to  a AliEMCALRecPoint considered as a Tmarker
-  // Compute the closest distance of approach from point px,py to this marker.
-  // The distance is computed in pixels units.
-  // HG Still need to update -> Not sure what this should achieve
-
-  TVector3 pos(0.,0.,0.) ;
-  GetLocalPosition(pos) ;
-  Float_t x =  pos.X() ;
-  Float_t y =  pos.Y() ;
-  const Int_t kMaxDiff = 10;
-  Int_t pxm  = gPad->XtoAbsPixel(x);
-  Int_t pym  = gPad->YtoAbsPixel(y);
-  Int_t dist = (px-pxm)*(px-pxm) + (py-pym)*(py-pym);
-  
-  if (dist > kMaxDiff) return 9999;
-  return dist;
-}
-
 //___________________________________________________________________________
  void AliEMCALRecPoint::Draw(Option_t *option)
  {
@@ -341,98 +347,35 @@ Int_t AliEMCALRecPoint::DistancetoPrimitive(Int_t px, Int_t py)
    AppendPad(option);
  }
 
-//______________________________________________________________________________
-void AliEMCALRecPoint::ExecuteEvent(Int_t /*event*/, Int_t, Int_t)
+//____________________________________________________________________________
+void AliEMCALRecPoint::EvalAll(Float_t logWeight,TClonesArray * digits, const Bool_t justClusters) 
 {
-  // Execute action corresponding to one event
-  // This member function is called when a AliEMCALRecPoint is clicked with the locator
-  //
-  // If Left button is clicked on AliEMCALRecPoint, the digits are switched on    
-  // and switched off when the mouse button is released.
-
-  //  static Int_t pxold, pyold;
-
-  /*  static TGraph *  digitgraph = 0 ;
-  static TPaveText* clustertext = 0 ;
-  
-  if (!gPad->IsEditable()) return;
-  
-  switch (event) {
-    
-    
-  case kButton1Down:{
-    AliEMCALDigit * digit ;
-
-    Int_t iDigit;
-    Int_t relid[2] ;
-  
-    const Int_t kMulDigit=AliEMCALRecPoint::GetDigitsMultiplicity() ;
-    Float_t * xi = new Float_t [kMulDigit] ; 
-    Float_t * zi = new Float_t [kMulDigit] ;
-    
-    for(iDigit = 0; iDigit < kMulDigit; iDigit++) {
-      Fatal("AliEMCALRecPoint::ExecuteEvent", " -> Something wrong with the code"); 
-      digit = 0 ; //dynamic_cast<AliEMCALDigit *>((fDigitsList)[iDigit]);
-      fGeomPtr->AbsToRelNumbering(digit->GetId(), relid) ;
-      fGeomPtr->PosInAlice(relid, xi[iDigit], zi[iDigit]) ;
-    }
+  // Evaluates cluster parameters
+       
+  // First calculate the index of digit with maximum amplitude and get 
+  // the supermodule number where it sits.
     
-    if (!digitgraph) {
-      digitgraph = new TGraph(fMulDigit,xi,zi);
-      digitgraph-> SetMarkerStyle(5) ; 
-      digitgraph-> SetMarkerSize(1.) ;
-      digitgraph-> SetMarkerColor(1) ;
-      digitgraph-> Draw("P") ;
-    }
-    if (!clustertext) {
-      
-      TVector3 pos(0.,0.,0.) ;
-      GetLocalPosition(pos) ;
-      clustertext = new TPaveText(pos.X()-10,pos.Z()+10,pos.X()+50,pos.Z()+35,"") ;
-      Text_t  line1[40] ;
-      Text_t  line2[40] ;
-      sprintf(line1,"Energy=%1.2f GeV",GetEnergy()) ;
-      sprintf(line2,"%d Digits",GetDigitsMultiplicity()) ;
-      clustertext ->AddText(line1) ;
-      clustertext ->AddText(line2) ;
-      clustertext ->Draw("");
-    }
-    gPad->Update() ; 
-    Print("") ;
-    delete[] xi ; 
-    delete[] zi ; 
-   }
+  fDigitIndMax       = GetMaximalEnergyIndex();
+  fSuperModuleNumber = fGeomPtr->GetSuperModuleNumber(GetAbsIdMaxDigit());
   
-break;
-  
-  case kButton1Up:
-    if (digitgraph) {
-      delete digitgraph  ;
-      digitgraph = 0 ;
-    }
-    if (clustertext) {
-      delete clustertext ;
-      clustertext = 0 ;
-    }
-    
-    break;
-    
-    }*/
-}
-//____________________________________________________________________________
-void AliEMCALRecPoint::EvalAll(Float_t logWeight,TClonesArray * digits) 
-{
-  // Evaluates all shower parameters
+  //Evaluate global and local position
+  EvalGlobalPosition(logWeight, digits) ;
   EvalLocalPosition(logWeight, digits) ;
+       
+  //Evaluate shower parameters
   EvalElipsAxis(logWeight, digits) ;
   EvalDispersion(logWeight, digits) ;
+
   //EvalCoreEnergy(logWeight, digits);
   EvalTime(digits) ;
   EvalPrimaries(digits) ;
   EvalParents(digits);
-
+       
   //Called last because it sets the global position of the cluster?
-  EvalLocal2TrackingCSTransform();
+  //Do not call it when recalculating clusters out of standard reconstruction
+  if(!justClusters){ 
+    EvalLocal2TrackingCSTransform();
+  }
 
 }
 
@@ -444,8 +387,8 @@ void  AliEMCALRecPoint::EvalDispersion(Float_t logWeight, TClonesArray * digits)
 
   Double_t d = 0., wtot = 0., w = 0.;
   Int_t iDigit=0, nstat=0;
-  AliEMCALDigit * digit ;
-  
+  AliEMCALDigit * digit=0;
+       
   // Calculates the dispersion in cell units 
   Double_t etai, phii, etaMean=0.0, phiMean=0.0; 
   int nSupMod=0, nModule=0, nIphi=0, nIeta=0;
@@ -457,6 +400,11 @@ void  AliEMCALRecPoint::EvalDispersion(Float_t logWeight, TClonesArray * digits)
     if (fAmp>0 && fEnergyList[iDigit]>0) {
       fGeomPtr->GetCellIndex(digit->GetId(), nSupMod,nModule,nIphi,nIeta);
       fGeomPtr->GetCellPhiEtaIndexInSModule(nSupMod,nModule,nIphi,nIeta, iphi,ieta);
+       
+      // In case of a shared cluster, index of SM in C side, columns start at 48 and ends at 48*2
+      // C Side impair SM, nSupMod%2=1; A side pair SM nSupMod%2=0
+      if(fSharedCluster && nSupMod%2) ieta+=AliEMCALGeoParams::fgkEMCALCols;
+               
       etai=(Double_t)ieta;
       phii=(Double_t)iphi;
       w = TMath::Max(0.,logWeight+TMath::Log(fEnergyList[iDigit]/fAmp ) ) ;
@@ -480,6 +428,11 @@ void  AliEMCALRecPoint::EvalDispersion(Float_t logWeight, TClonesArray * digits)
     if (fAmp>0 && fEnergyList[iDigit]>0) {
       fGeomPtr->GetCellIndex(digit->GetId(), nSupMod,nModule,nIphi,nIeta);
       fGeomPtr->GetCellPhiEtaIndexInSModule(nSupMod,nModule,nIphi,nIeta, iphi,ieta);
+               
+      // In case of a shared cluster, index of SM in C side, columns start at 48 and ends at 48*2
+      // C Side impair SM, nSupMod%2=1; A side pair SM, nSupMod%2=0
+      if(fSharedCluster && nSupMod%2) ieta+=AliEMCALGeoParams::fgkEMCALCols;
+      
       etai=(Double_t)ieta;
       phii=(Double_t)iphi;
       w = TMath::Max(0.,logWeight+TMath::Log(fEnergyList[iDigit]/fAmp ) ) ;
@@ -495,43 +448,124 @@ void  AliEMCALRecPoint::EvalDispersion(Float_t logWeight, TClonesArray * digits)
   else                      d = 0. ; 
 
   fDispersion = TMath::Sqrt(d) ;
+  //printf("AliEMCALRecPoint::EvalDispersion() : Dispersion %f \n",fDispersion);
+}
+
+//____________________________________________________________________________
+void AliEMCALRecPoint::EvalDistanceToBadChannels(AliCaloCalibPedestal* caloped)
+{
+  //For each EMC rec. point set the distance to the nearest bad channel.
+  //AliInfo(Form("%d bad channel(s) found.\n", caloped->GetDeadTowerCount()));
+  //It is done in cell units and not in global or local position as before (Sept 2010)
+  
+  if(!caloped->GetDeadTowerCount()) return;
+  
+  //Get channels map of the supermodule where the cluster is.
+  TH2D* hMap  = caloped->GetDeadMap(fSuperModuleNumber);
+  
+  Int_t dRrow, dReta;  
+  Float_t  minDist = 10000.;
+  Float_t  dist    = 0.;
+  Int_t nSupMod, nModule;
+  Int_t nIphi, nIeta;
+  Int_t iphi, ieta;
+  fDigitIndMax  = GetMaximalEnergyIndex();
+  fGeomPtr->GetCellIndex(fAbsIdList[fDigitIndMax], nSupMod,nModule,nIphi,nIeta);
+  fGeomPtr->GetCellPhiEtaIndexInSModule(nSupMod,nModule,nIphi,nIeta, iphi,ieta);
+  
+  //Loop on tower status map 
+  for(Int_t irow = 0; irow < AliEMCALGeoParams::fgkEMCALRows; irow++){
+    for(Int_t icol = 0; icol < AliEMCALGeoParams::fgkEMCALCols; icol++){
+      //Check if tower is bad.
+      if(hMap->GetBinContent(icol,irow)==AliCaloCalibPedestal::kAlive) continue;
+      //printf("AliEMCALRecPoint::EvalDistanceToBadChannels() - Bad channel in SM %d, col %d, row %d\n",iSM,icol, irow);
+      
+      dRrow=TMath::Abs(irow-iphi);
+      dReta=TMath::Abs(icol-ieta);
+      dist=TMath::Sqrt(dRrow*dRrow+dReta*dReta);
+      if(dist < minDist) minDist = dist;
+      
+    }
+  }
+  
+  //In case the cluster is shared by 2 SuperModules, need to check the map of the second Super Module
+  if (fSharedCluster) {
+    TH2D* hMap2 = 0;
+    Int_t nSupMod2 = -1;
+    
+    //The only possible combinations are (0,1), (2,3) ... (10,11)
+    if(fSuperModuleNumber%2) nSupMod2 = fSuperModuleNumber-1;
+    else                     nSupMod2 = fSuperModuleNumber+1;
+    hMap2  = caloped->GetDeadMap(nSupMod2);
+    
+    //Loop on tower status map of second super module
+    for(Int_t irow = 0; irow < AliEMCALGeoParams::fgkEMCALRows; irow++){
+      for(Int_t icol = 0; icol < AliEMCALGeoParams::fgkEMCALCols; icol++){
+       //Check if tower is bad.
+       if(hMap2->GetBinContent(icol,irow)==AliCaloCalibPedestal::kAlive) continue;
+       //printf("AliEMCALRecPoint::EvalDistanceToBadChannels() - Bad channel in SM %d, col %d, row %d\n",iSM,icol, irow);
+        dRrow=TMath::Abs(irow-iphi);
+       
+        if(fSuperModuleNumber%2) {
+         dReta=TMath::Abs(icol-(AliEMCALGeoParams::fgkEMCALCols+ieta));
+       }
+       else {
+        dReta=TMath::Abs(AliEMCALGeoParams::fgkEMCALCols+icol-ieta);
+       }                    
+        
+       dist=TMath::Sqrt(dRrow*dRrow+dReta*dReta);
+        if(dist < minDist) minDist = dist;        
+
+      }
+    }
+    
+  }// shared cluster in 2 SuperModules
+  
+  fDistToBadTower = minDist;
+  //printf("AliEMCALRecPoint::EvalDistanceToBadChannel() - Distance to Bad is %f cm, shared cluster? %d \n",fDistToBadTower,fSharedCluster);
 }
 
+
 //____________________________________________________________________________
 void AliEMCALRecPoint::EvalLocalPosition(Float_t logWeight, TClonesArray * digits)
 {
   // Calculates the center of gravity in the local EMCAL-module coordinates 
   //  Info("Print", " logWeight %f : cluster energy %f ", logWeight, fAmp); // for testing
   
-  static Double_t dist;
-
-  AliEMCALDigit * digit;
-  Int_t i=0, nstat=0, idMax=-1;
+  AliEMCALDigit * digit=0;
+  Int_t i=0, nstat=0;
+  
+  Double_t dist  = TmaxInCm(Double_t(fAmp));
+  //Int_t      idMax = GetAbsIdMaxDigit();// idMax is not used at all in RelPosCellInSModule, why use it?
+  
   Double_t clXYZ[3]={0.,0.,0.}, clRmsXYZ[3]={0.,0.,0.}, xyzi[3], wtot=0., w=0.;
-
+  
   //printf(" dist : %f  e : %f \n", dist, fAmp);
   for(Int_t iDigit=0; iDigit<fMulDigit; iDigit++) {
     digit = dynamic_cast<AliEMCALDigit *>(digits->At(fDigitsList[iDigit])) ;
-    if(iDigit==0) {
-      idMax = digit->GetId(); // is it correct
-      dist  = TmaxInCm(Double_t(fAmp));
+    
+    if(!digit) {
+      AliError("No Digit!!");
+      continue;
     }
-    fGeomPtr->RelPosCellInSModule(digit->GetId(), idMax, dist, xyzi[0], xyzi[1], xyzi[2]);
-    //printf(" Id %i : dist %f Local x,y,z %f %f %f \n", digit->GetId(), dist, xyzi[0], xyzi[1], xyzi[2]);
-
-    //fGeomPtr->RelPosCellInSModule(digit->GetId(), xyzi[0], xyzi[1], xyzi[2]);
-    //printf(" Id %i : dist %f Local x,y,z %f %f %f \n", digit->GetId(), 0.0, xyzi[0], xyzi[1], xyzi[2]);
-    //    if(fAmp>102.) assert(0);
-
+    
+    fGeomPtr->RelPosCellInSModule(digit->GetId(), dist, xyzi[0], xyzi[1], xyzi[2]);
+    
+    //Temporal patch, due to mapping problem, need to swap "y" in one of the 2 SM, although no effect in position calculation. GCB 05/2010
+    if(fSharedCluster && fSuperModuleNumber != fGeomPtr->GetSuperModuleNumber(digit->GetId())) xyzi[1]*=-1;
+    
+    //printf("EvalLocalPosition Cell:  Id %i, SM %i : dist %f Local x,y,z %f %f %f \n", 
+    //         digit->GetId(), fGeomPtr->GetSuperModuleNumber(digit->GetId()), dist, xyzi[0], xyzi[1], xyzi[2]);
+    
     if(logWeight > 0.0)  w = TMath::Max( 0., logWeight + TMath::Log( fEnergyList[iDigit] / fAmp ));
     else  w = fEnergyList[iDigit]; // just energy
-
+    
     if(w>0.0) {
       wtot += w ;
       nstat++;
       for(i=0; i<3; i++ ) {
-        clXYZ[i]    += (w*xyzi[i]);
-        clRmsXYZ[i] += (w*xyzi[i]*xyzi[i]);
+       clXYZ[i]    += (w*xyzi[i]);
+       clRmsXYZ[i] += (w*xyzi[i]*xyzi[i]);
       }
     }
   }
@@ -541,11 +575,11 @@ void AliEMCALRecPoint::EvalLocalPosition(Float_t logWeight, TClonesArray * digit
     for(i=0; i<3; i++ ) {
       clXYZ[i] /= wtot;
       if(nstat>1) {
-        clRmsXYZ[i] /= (wtot*wtot);  
-        clRmsXYZ[i]  =  clRmsXYZ[i] - clXYZ[i]*clXYZ[i];
+       clRmsXYZ[i] /= (wtot*wtot);  
+       clRmsXYZ[i]  =  clRmsXYZ[i] - clXYZ[i]*clXYZ[i];
        if(clRmsXYZ[i] > 0.0) {
-          clRmsXYZ[i] =  TMath::Sqrt(clRmsXYZ[i]);
-        } else      clRmsXYZ[i] = 0;
+         clRmsXYZ[i] =  TMath::Sqrt(clRmsXYZ[i]);
+       } else      clRmsXYZ[i] = 0;
       } else        clRmsXYZ[i] = 0;
     }    
   } else {
@@ -553,40 +587,60 @@ void AliEMCALRecPoint::EvalLocalPosition(Float_t logWeight, TClonesArray * digit
       clXYZ[i] = clRmsXYZ[i] = -1.;
     }
   }
-  // clRmsXYZ[i] ??
+  
+  //   // Cluster of one single digit, smear the position to avoid discrete position
+  //   // smear x and z with +- 3 cm to uniform (avoid discrete effects). Tower size is approx 6 cm.
+  //   // Rndm generates a number in ]0,1]
+  //   if (fMulDigit==1) { 
+  //     clXYZ[0] += fGeomPtr->GetPhiTileSize()*(0.5 - gRandom->Rndm()); 
+  //     clXYZ[2] += fGeomPtr->GetEtaTileSize()*(0.5 - gRandom->Rndm()); 
+  //   }
+  
+  //Set position in local vector
   fLocPos.SetX(clXYZ[0]);
   fLocPos.SetY(clXYZ[1]);
   fLocPos.SetZ(clXYZ[2]);
-    
-//  if (gDebug==2)
-//    printf("EvalLocalPosition: eta,phi,r = %f,%f,%f", fLocPos.X(), fLocPos.Y(), fLocPos.Z()) ; 
-  fLocPosM = 0 ; // covariance matrix
+  
+  if (gDebug==2)
+    printf("EvalLocalPosition Cluster: Local (x,y,z) = (%f,%f,%f) \n", fLocPos.X(), fLocPos.Y(), fLocPos.Z()) ; 
+  
 }
 
+
 //____________________________________________________________________________
-void AliEMCALRecPoint::EvalLocalPositionFit(Double_t deff, Double_t logWeight, 
-Double_t phiSlope, TClonesArray * digits)
+void AliEMCALRecPoint::EvalGlobalPosition(Float_t logWeight, TClonesArray * digits)
 {
-  // Aug 14-16, 2007 - for fit 
-  // Aug 31 - should be static ??
-  static Double_t dist, ycorr;
-  static AliEMCALDigit *digit;
-
-  Int_t i=0, nstat=0, idMax=-1;
-  Double_t clXYZ[3]={0.,0.,0.}, clRmsXYZ[3]={0.,0.,0.}, xyzi[3], wtot=0., w=0.; 
+  // Calculates the center of gravity in the global ALICE coordinates 
+  //  Info("Print", " logWeight %f : cluster energy %f ", logWeight, fAmp); // for testing
+  
+  AliEMCALDigit * digit=0;
+  Int_t i=0, nstat=0;
+       
+  Double_t dist  = TmaxInCm(Double_t(fAmp));
+  //Int_t      idMax = GetAbsIdMaxDigit();// idMax is not used at all in RelPosCellInSModule, why use it?
+       
+  Double_t clXYZ[3]={0.,0.,0.}, clRmsXYZ[3]={0.,0.,0.}, lxyzi[3], xyzi[3], wtot=0., w=0.;
 
-  for(Int_t iDigit=0; iDigit<digits->GetEntries(); iDigit++) {
+  //printf(" dist : %f  e : %f \n", dist, fAmp);
+  for(Int_t iDigit=0; iDigit<fMulDigit; iDigit++) {
     digit = dynamic_cast<AliEMCALDigit *>(digits->At(fDigitsList[iDigit])) ;
-    if(iDigit==0) {
-      idMax = digit->GetId(); // is it correct
-      dist  = TmaxInCm(Double_t(fAmp));
-    }
-
-    dist = deff;
-    fGeomPtr->RelPosCellInSModule(digit->GetId(), idMax, dist, xyzi[0], xyzi[1], xyzi[2]);
 
+    if(!digit) {
+      AliError("No Digit!!");
+      continue;
+    }    
+    
+    //Get the local coordinates of the cell
+    fGeomPtr->RelPosCellInSModule(digit->GetId(), dist, lxyzi[0], lxyzi[1], lxyzi[2]);
+    
+    //Now get the global coordinate
+    fGeomPtr->GetGlobal(lxyzi,xyzi, fGeomPtr->GetSuperModuleNumber(digit->GetId()));
+    //TVector3 pos(xyzi[0], xyzi[1], xyzi[2]);
+    //printf("EvalGlobalPosition Cell:  Id %i, SM %i : dist %f Local (x,y,z) = (%f %f %f), eta %f, phi%f \n", 
+    //    digit->GetId(), fGeomPtr->GetSuperModuleNumber(digit->GetId()),dist, xyzi[0], xyzi[1], xyzi[2],pos.Eta(),pos.Phi()*TMath::RadToDeg());
+         
     if(logWeight > 0.0)  w = TMath::Max( 0., logWeight + TMath::Log( fEnergyList[iDigit] / fAmp ));
-    else                 w = fEnergyList[iDigit]; // just energy
+    else  w = fEnergyList[iDigit]; // just energy
 
     if(w>0.0) {
       wtot += w ;
@@ -615,6 +669,77 @@ Double_t phiSlope, TClonesArray * digits)
       clXYZ[i] = clRmsXYZ[i] = -1.;
     }
   }
+
+//  // Cluster of one single digit, smear the position to avoid discrete position
+//  // smear x and z with +- 3 cm to uniform (avoid discrete effects). Tower size is approx 6 cm.
+//  // Rndm generates a number in ]0,1]
+//  if (fMulDigit==1) { 
+//    clXYZ[0] += fGeomPtr->GetPhiTileSize()*(0.5 - gRandom->Rndm()); 
+//    clXYZ[2] += fGeomPtr->GetEtaTileSize()*(0.5 - gRandom->Rndm());  
+//  }
+       
+  //Set position in global vector
+  fGlobPos.SetX(clXYZ[0]);
+  fGlobPos.SetY(clXYZ[1]);
+  fGlobPos.SetZ(clXYZ[2]);
+               
+  if (gDebug==2)
+       printf("EvalGlobalPosition Cluster: (x ,y ,z) = (%f,%f,%f), eta %f,phi %f\n", 
+                  fGlobPos.X(), fGlobPos.Y(), fGlobPos.Z(),fGlobPos.Eta(),fGlobPos.Phi()*TMath::RadToDeg()) ; 
+}
+
+//____________________________________________________________________________
+void AliEMCALRecPoint::EvalLocalPositionFit(Double_t deff, Double_t logWeight, 
+Double_t phiSlope, TClonesArray * digits)
+{
+  // Evaluates local position of clusters in SM
+  
+  Double_t ycorr=0;
+  AliEMCALDigit *digit=0;
+  Int_t i=0, nstat=0;
+  Double_t clXYZ[3]={0.,0.,0.}, clRmsXYZ[3]={0.,0.,0.}, xyzi[3], wtot=0., w=0.; 
+
+  Double_t dist  = TmaxInCm(Double_t(fAmp));
+  //Int_t      idMax = GetAbsIdMaxDigit();// idMax is not used at all in RelPosCellInSModule, why use it?
+       
+  for(Int_t iDigit=0; iDigit<digits->GetEntries(); iDigit++) {
+    digit = dynamic_cast<AliEMCALDigit *>(digits->At(fDigitsList[iDigit])) ;
+    if(digit){
+      dist = deff;
+      //fGeomPtr->RelPosCellInSModule(digit->GetId(), idMax, dist, xyzi[0], xyzi[1], xyzi[2]);
+      fGeomPtr->RelPosCellInSModule(digit->GetId(), dist, xyzi[0], xyzi[1], xyzi[2]);
+      
+      if(logWeight > 0.0)  w = TMath::Max( 0., logWeight + TMath::Log( fEnergyList[iDigit] / fAmp ));
+      else                 w = fEnergyList[iDigit]; // just energy
+      
+      if(w>0.0) {
+        wtot += w ;
+        nstat++;
+        for(i=0; i<3; i++ ) {
+          clXYZ[i]    += (w*xyzi[i]);
+          clRmsXYZ[i] += (w*xyzi[i]*xyzi[i]);
+        }
+      }
+    }else AliError("Digit null");
+  }//loop
+  //  cout << " wtot " << wtot << endl;
+  if ( wtot > 0 ) { 
+    //    xRMS   = TMath::Sqrt(x2m - xMean*xMean);
+    for(i=0; i<3; i++ ) {
+      clXYZ[i] /= wtot;
+      if(nstat>1) {
+        clRmsXYZ[i] /= (wtot*wtot);  
+        clRmsXYZ[i]  =  clRmsXYZ[i] - clXYZ[i]*clXYZ[i];
+       if(clRmsXYZ[i] > 0.0) {
+          clRmsXYZ[i] =  TMath::Sqrt(clRmsXYZ[i]);
+        } else      clRmsXYZ[i] = 0;
+      } else        clRmsXYZ[i] = 0;
+    }    
+  } else {
+    for(i=0; i<3; i++ ) {
+      clXYZ[i] = clRmsXYZ[i] = -1.;
+    }
+  }
   // clRmsXYZ[i] ??
   if(phiSlope != 0.0 && logWeight > 0.0 && wtot) { 
     // Correction in phi direction (y - coords here); Aug 16;
@@ -623,13 +748,13 @@ Double_t phiSlope, TClonesArray * digits)
     //printf(" y %f : ycorr %f : slope %f \n", clXYZ[1], ycorr, phiSlope); 
     clXYZ[1] = ycorr;
   }
+       
   fLocPos.SetX(clXYZ[0]);
   fLocPos.SetY(clXYZ[1]);
   fLocPos.SetZ(clXYZ[2]);
     
 //  if (gDebug==2)
 //    printf("EvalLocalPosition: eta,phi,r = %f,%f,%f", fLocPos.X(), fLocPos.Y(), fLocPos.Z()) ; 
-  fLocPosM = 0 ; // covariance matrix
 }
 
 //_____________________________________________________________________________
@@ -645,9 +770,9 @@ Bool_t AliEMCALRecPoint::EvalLocalPosition2(TClonesArray * digits, TArrayD &ed)
 Bool_t AliEMCALRecPoint::EvalLocalPositionFromDigits(TClonesArray *digits, TArrayD &ed, TVector3 &locPos)
 {
   // Used when digits should be recalibrated
-  static Double_t deff, w0, esum;
-  static Int_t iDigit;
-  //  static AliEMCALDigit *digit;
+  Double_t deff=0, w0=0, esum=0;
+  Int_t iDigit=0;
+  //  AliEMCALDigit *digit;
 
   if(ed.GetSize() && (digits->GetEntries()!=ed.GetSize())) return kFALSE;
 
@@ -663,31 +788,35 @@ Bool_t AliEMCALRecPoint::EvalLocalPositionFromDigits(TClonesArray *digits, TArra
 //_____________________________________________________________________________
 Bool_t AliEMCALRecPoint::EvalLocalPositionFromDigits(const Double_t esum, const Double_t deff, const Double_t w0, TClonesArray *digits, TArrayD &ed, TVector3 &locPos)
 {
-  static AliEMCALDigit *digit;
+  //Evaluate position of digits in supermodule.
+  AliEMCALDigit *digit=0;
 
-  Int_t i=0, nstat=0, idMax=-1;
+  Int_t i=0, nstat=0;
   Double_t clXYZ[3]={0.,0.,0.}, xyzi[3], wtot=0., w=0.; 
-
+  //Int_t      idMax = GetAbsIdMaxDigit();// idMax is not used at all in RelPosCellInSModule, why use it?
+       
   // Get pointer to EMCAL geometry
   // (can't use fGeomPtr in static method)
   AliEMCALGeometry* geo = AliEMCALGeometry::GetInstance(); 
 
   for(Int_t iDigit=0; iDigit<digits->GetEntries(); iDigit++) {
     digit = dynamic_cast<AliEMCALDigit *>(digits->At(iDigit));
-
-    geo->RelPosCellInSModule(digit->GetId(), idMax, deff, xyzi[0], xyzi[1], xyzi[2]);
-
-    if(w0 > 0.0)  w = TMath::Max( 0., w0 + TMath::Log(ed[iDigit] / esum));
-    else          w = ed[iDigit]; // just energy
-
-    if(w>0.0) {
-      wtot += w ;
-      nstat++;
-      for(i=0; i<3; i++ ) {
-        clXYZ[i] += (w*xyzi[i]);
+    if(digit){
+      //geo->RelPosCellInSModule(digit->GetId(), idMax, deff, xyzi[0], xyzi[1], xyzi[2]);
+      geo->RelPosCellInSModule(digit->GetId(), deff, xyzi[0], xyzi[1], xyzi[2]);
+      
+      if(w0 > 0.0)  w = TMath::Max( 0., w0 + TMath::Log(ed[iDigit] / esum));
+      else          w = ed[iDigit]; // just energy
+      
+      if(w>0.0) {
+        wtot += w ;
+        nstat++;
+        for(i=0; i<3; i++ ) {
+          clXYZ[i] += (w*xyzi[i]);
+        }
       }
-    }
-  }
+    }else AliError("Digit null");
+  }//loop
   //  cout << " wtot " << wtot << endl;
   if (wtot > 0) { 
     for(i=0; i<3; i++ ) {
@@ -713,7 +842,7 @@ void AliEMCALRecPoint::GetDeffW0(const Double_t esum , Double_t &deff,  Double_t
   // for coordinate calculation; 0.5 GeV < esum <100 GeV.
   // Look to:  http://rhic.physics.wayne.edu/~pavlinov/ALICE/SHISHKEBAB/RES/CALIB/GEOMCORR/deffandW0VaEgamma_2.gif
   //
-  static Double_t e=0.0;
+  Double_t e=0.0;
   const  Double_t kdp0=9.25147, kdp1=1.16700; // Hard coded now
   const  Double_t kwp0=4.83713, kwp1=-2.77970e-01, kwp2 = 4.41116;
 
@@ -736,9 +865,9 @@ void AliEMCALRecPoint::EvalCoreEnergy(Float_t logWeight, TClonesArray * digits)
   // Unfinished - Nov 15,2006
   // Distance is calculate in (phi,eta) units
 
-  AliEMCALDigit * digit ;
+  AliEMCALDigit * digit = 0 ;
 
-  Int_t iDigit;
+  Int_t iDigit=0;
 
   if (!fLocPos.Mag()) {
     EvalLocalPosition(logWeight, digits);
@@ -765,7 +894,7 @@ void  AliEMCALRecPoint::EvalElipsAxis(Float_t logWeight,TClonesArray * digits)
   // Calculates the axis of the shower ellipsoid in eta and phi
   // in cell units
 
-  static TString gn(fGeomPtr->GetName());
+  TString gn(fGeomPtr->GetName());
 
   Double_t wtot = 0.;
   Double_t x    = 0.;
@@ -775,8 +904,8 @@ void  AliEMCALRecPoint::EvalElipsAxis(Float_t logWeight,TClonesArray * digits)
   Double_t dxz  = 0.;
 
   AliEMCALDigit * digit = 0;
-
-  Double_t etai , phii, w
+       
+  Double_t etai =0, phii=0, w=0
   int nSupMod=0, nModule=0, nIphi=0, nIeta=0;
   int iphi=0, ieta=0;
   for(Int_t iDigit=0; iDigit<fMulDigit; iDigit++) {
@@ -789,9 +918,14 @@ void  AliEMCALRecPoint::EvalElipsAxis(Float_t logWeight,TClonesArray * digits)
 
     fGeomPtr->GetCellIndex(digit->GetId(), nSupMod,nModule,nIphi,nIeta);
     fGeomPtr->GetCellPhiEtaIndexInSModule(nSupMod,nModule,nIphi,nIeta, iphi,ieta);
+         
+    // In case of a shared cluster, index of SM in C side, columns start at 48 and ends at 48*2
+    // C Side impair SM, nSupMod%2=1; A side pair SM, nSupMod%2=0
+    if(fSharedCluster && nSupMod%2) ieta+=AliEMCALGeoParams::fgkEMCALCols;
+    
     etai=(Double_t)ieta;
     phii=(Double_t)iphi;
-
+         
     w = TMath::Max(0.,logWeight+TMath::Log(fEnergyList[iDigit]/fAmp ) ) ;
     // fAmp summed amplitude of digits, i.e. energy of recpoint 
     // Gives smaller value of lambda than log weight  
@@ -834,7 +968,7 @@ void  AliEMCALRecPoint::EvalElipsAxis(Float_t logWeight,TClonesArray * digits)
     fLambda[1]= 0. ;
   }
 
-  //  printf("Evalaxis: lambdas  = %f,%f", fLambda[0],fLambda[1]) ; 
+  //printf("AliEMCALRecPoint::EvalElipsAxis() lambdas  = %f,%f \n", fLambda[0],fLambda[1]) ; 
 
 }
 
@@ -844,42 +978,49 @@ void  AliEMCALRecPoint::EvalPrimaries(TClonesArray * digits)
   // Constructs the list of primary particles (tracks) which 
   // have contributed to this RecPoint and calculate deposited energy 
   // for each track
-  
-  AliEMCALDigit * digit ;
+    
+  AliEMCALDigit * digit =0;
   Int_t * primArray = new Int_t[fMaxTrack] ;
+  memset(primArray,-1,sizeof(Int_t)*fMaxTrack);
   Float_t * dEPrimArray = new Float_t[fMaxTrack] ;
-
+  memset(dEPrimArray,-1,sizeof(Int_t)*fMaxTrack);
+  
   Int_t index ;  
   for ( index = 0 ; index < GetDigitsMultiplicity() ; index++ ) { // all digits
     digit = dynamic_cast<AliEMCALDigit *>(digits->At( fDigitsList[index] )) ; 
+    if(!digit) {
+      AliError("No Digit!!");
+      continue;
+    }
+    
     Int_t nprimaries = digit->GetNprimary() ;
     if ( nprimaries == 0 ) continue ;
     Int_t jndex ;
     for ( jndex = 0 ; jndex < nprimaries ; jndex++ ) { // all primaries in digit
       if ( fMulTrack > fMaxTrack ) {
-       fMulTrack = fMaxTrack ;
-       Error("EvalPrimaries", "increase fMaxTrack ")  ;
-       break ;
+        fMulTrack = fMaxTrack ;
+        Error("EvalPrimaries", "increase fMaxTrack ")  ;
+        break ;
       }
       Int_t newPrimary = digit->GetPrimary(jndex+1);
       Float_t dEPrimary = digit->GetDEPrimary(jndex+1);
       Int_t kndex ;
       Bool_t already = kFALSE ;
       for ( kndex = 0 ; kndex < fMulTrack ; kndex++ ) { //check if not already stored
-       if ( newPrimary == primArray[kndex] ){
-         already = kTRUE ;
-         dEPrimArray[kndex] += dEPrimary; 
-         break ;
-       }
+        if ( newPrimary == primArray[kndex] ){
+          already = kTRUE ;
+          dEPrimArray[kndex] += dEPrimary; 
+          break ;
+        }
       } // end of check
       if ( !already && (fMulTrack < fMaxTrack)) { // store it
-       primArray[fMulTrack] = newPrimary ; 
-       dEPrimArray[fMulTrack] = dEPrimary ; 
-       fMulTrack++ ;
+        primArray[fMulTrack] = newPrimary ; 
+        dEPrimArray[fMulTrack] = dEPrimary ; 
+        fMulTrack++ ;
       } // store it
     } // all primaries in digit
   } // all digits
-
+  
   Int_t *sortIdx = new Int_t[fMulTrack];
   TMath::Sort(fMulTrack,dEPrimArray,sortIdx); 
   for(index = 0; index < fMulTrack; index++) {
@@ -889,52 +1030,59 @@ void  AliEMCALRecPoint::EvalPrimaries(TClonesArray * digits)
   delete [] sortIdx;
   delete [] primArray ;
   delete [] dEPrimArray ;
-
+  
 }
 
 //______________________________________________________________________________
 void  AliEMCALRecPoint::EvalParents(TClonesArray * digits)
 {
   // Constructs the list of parent particles (tracks) which have contributed to this RecPoint
-  AliEMCALDigit * digit ;
+
+  AliEMCALDigit * digit=0 ;
   Int_t * parentArray = new Int_t[fMaxTrack] ;
+  memset(parentArray,-1,sizeof(Int_t)*fMaxTrack);
   Float_t * dEParentArray = new Float_t[fMaxTrack] ;
-
+  memset(dEParentArray,-1,sizeof(Int_t)*fMaxTrack);
+  
   Int_t index ;  
   for ( index = 0 ; index < GetDigitsMultiplicity() ; index++ ) { // all digits
     if (fDigitsList[index] >= digits->GetEntries() || fDigitsList[index] < 0)
-       AliError(Form("Trying to get invalid digit %d (idx in WriteRecPoint %d)",fDigitsList[index],index));
+      AliError(Form("Trying to get invalid digit %d (idx in WriteRecPoint %d)",fDigitsList[index],index));
     digit = dynamic_cast<AliEMCALDigit *>(digits->At( fDigitsList[index] )) ; 
+    if(!digit) {
+      AliError("No Digit!!");
+      continue;
+    }
+    
     Int_t nparents = digit->GetNiparent() ;
     if ( nparents == 0 ) continue ;
-
+    
     Int_t jndex ;
     for ( jndex = 0 ; jndex < nparents ; jndex++ ) { // all primaries in digit
       if ( fMulParent > fMaxParent ) {
-       fMulTrack = - 1 ;
-       Error("EvalParents", "increase fMaxParent")  ;
-       break ;
+        fMulTrack = - 1 ;
+        Error("EvalParents", "increase fMaxParent")  ;
+        break ;
       }
       Int_t newParent = digit->GetIparent(jndex+1) ;
       Float_t newdEParent = digit->GetDEParent(jndex+1) ;
       Int_t kndex ;
       Bool_t already = kFALSE ;
       for ( kndex = 0 ; kndex < fMulParent ; kndex++ ) { //check if not already stored
-       if ( newParent == parentArray[kndex] ){
-         dEParentArray[kndex] += newdEParent;
-         already = kTRUE ;
-         break ;
-       }
+        if ( newParent == parentArray[kndex] ){
+          dEParentArray[kndex] += newdEParent;
+          already = kTRUE ;
+          break ;
+        }
       } // end of check
       if ( !already && (fMulParent < fMaxParent)) { // store it
-       parentArray[fMulParent] = newParent ; 
-       dEParentArray[fMulParent] = newdEParent ; 
-       fMulParent++ ;
+        parentArray[fMulParent] = newParent ; 
+        dEParentArray[fMulParent] = newdEParent ; 
+        fMulParent++ ;
       } // store it
     } // all parents in digit
   } // all digits
-
+  
   if (fMulParent>0) {
     Int_t *sortIdx = new Int_t[fMulParent];
     TMath::Sort(fMulParent,dEParentArray,sortIdx); 
@@ -944,7 +1092,7 @@ void  AliEMCALRecPoint::EvalParents(TClonesArray * digits)
     }
     delete [] sortIdx;
   }
+  
   delete [] parentArray;
   delete [] dEParentArray;
 }
@@ -964,21 +1112,22 @@ void AliEMCALRecPoint::GetGlobalPosition(TVector3 & gpos) const
   // returns the position of the cluster in the global reference system of ALICE
   // These are now the Cartesian X, Y and Z
   //  cout<<" geom "<<geom<<endl;
-  fGeomPtr->GetGlobal(fLocPos, gpos, fSuperModuleNumber);
-
+  // fGeomPtr->GetGlobal(fLocPos, gpos, fSuperModuleNumber);
+  gpos = fGlobPos;
+       
 }
 
 //____________________________________________________________________________
-void AliEMCALRecPoint::GetGlobalPosition(TVector3 & gpos, TMatrixF & gmat) const
-{
-  // returns the position of the cluster in the global reference system of ALICE
-  // These are now the Cartesian X, Y and Z
-  //  cout<<" geom "<<geom<<endl;
-
-  //To be implemented
-  fGeomPtr->GetGlobalEMCAL(this, gpos, gmat);
-
-}
+//void AliEMCALRecPoint::GetGlobalPosition(TVector3 & gpos, TMatrixF & gmat) const
+//{
+//  // returns the position of the cluster in the global reference system of ALICE
+//  // These are now the Cartesian X, Y and Z
+//  //  cout<<" geom "<<geom<<endl;
+//
+//  //To be implemented
+//  fGeomPtr->GetGlobalEMCAL(this, gpos, gmat);
+//
+//}
 
 //_____________________________________________________________________________
 void AliEMCALRecPoint::EvalLocal2TrackingCSTransform()
@@ -1001,8 +1150,10 @@ void AliEMCALRecPoint::EvalLocal2TrackingCSTransform()
   SetX(txyz[0]); SetY(txyz[1]); SetZ(txyz[2]);
 
   if(AliLog::GetGlobalDebugLevel()>0) {
-    TVector3 gpos; TMatrixF gmat;
-    GetGlobalPosition(gpos,gmat);
+    TVector3 gpos; //TMatrixF gmat;
+    //GetGlobalPosition(gpos,gmat); //Not doing anythin special, replace by next line. 
+    fGeomPtr->GetGlobal(fLocPos, gpos, GetSuperModuleNumber());
+    
     Float_t gxyz[3];
     GetGlobalXYZ(gxyz);
     AliInfo(Form("lCS-->(%.3f,%.3f,%.3f), tCS-->(%.3f,%.3f,%.3f), gCS-->(%.3f,%.3f,%.3f),  gCScalc-\
@@ -1023,7 +1174,6 @@ Float_t AliEMCALRecPoint::GetMaximalEnergy(void) const
   Float_t menergy = 0. ;
 
   Int_t iDigit;
-
   for(iDigit=0; iDigit<fMulDigit; iDigit++) {
  
     if(fEnergyList[iDigit] > menergy) 
@@ -1032,6 +1182,27 @@ Float_t AliEMCALRecPoint::GetMaximalEnergy(void) const
   return menergy ;
 }
 
+//____________________________________________________________________________
+Int_t AliEMCALRecPoint::GetMaximalEnergyIndex(void) const
+{
+  // Finds the maximum energy in the cluster
+  
+  Float_t menergy = 0. ;
+  Int_t mid       = 0  ;
+  Int_t iDigit;
+  
+  for(iDigit=0; iDigit<fMulDigit; iDigit++) {
+    
+    if(fEnergyList[iDigit] > menergy){ 
+      menergy = fEnergyList[iDigit] ;
+      mid     = iDigit ;
+    }
+  }//loop on cluster digits
+  
+  return mid ;
+}
+
+
 //____________________________________________________________________________
 Int_t AliEMCALRecPoint::GetMultiplicityAtLevel(Float_t H) const
 {
@@ -1054,11 +1225,11 @@ Int_t  AliEMCALRecPoint::GetNumberOfLocalMax(AliEMCALDigit **  maxAt, Float_t *
   // Calculates the number of local maxima in the cluster using fLocalMaxCut as the minimum
   // energy difference between two local maxima
 
-  AliEMCALDigit * digit ;
-  AliEMCALDigit * digitN ;
+  AliEMCALDigit * digit  = 0;
+  AliEMCALDigit * digitN = 0;
   
-  Int_t iDigitN ;
-  Int_t iDigit ;
+  Int_t iDigitN = 0 ;
+  Int_t iDigit  = 0 ;
 
   for(iDigit = 0; iDigit < fMulDigit; iDigit++)
     maxAt[iDigit] = (AliEMCALDigit*) digits->At(fDigitsList[iDigit])  ;
@@ -1068,7 +1239,8 @@ Int_t  AliEMCALRecPoint::GetNumberOfLocalMax(AliEMCALDigit **  maxAt, Float_t *
       digit = maxAt[iDigit] ;
           
       for(iDigitN = 0; iDigitN < fMulDigit; iDigitN++) {       
-       digitN = (AliEMCALDigit *) digits->At(fDigitsList[iDigitN]) ; 
+       if(iDigitN == iDigit) continue;//the same digit
+        digitN = (AliEMCALDigit *) digits->At(fDigitsList[iDigitN]) ; 
        
        if ( AreNeighbours(digit, digitN) ) {
          if (fEnergyList[iDigit] > fEnergyList[iDigitN] ) {    
@@ -1076,8 +1248,7 @@ Int_t  AliEMCALRecPoint::GetNumberOfLocalMax(AliEMCALDigit **  maxAt, Float_t *
            // but may be digit too is not local max ?
            if(fEnergyList[iDigit] < fEnergyList[iDigitN] + locMaxCut) 
              maxAt[iDigit] = 0 ;
-         }
-         else {
+         } else {
            maxAt[iDigit] = 0 ;
            // but may be digitN too is not local max ?
            if(fEnergyList[iDigit] > fEnergyList[iDigitN] - locMaxCut) 
@@ -1096,6 +1267,7 @@ Int_t  AliEMCALRecPoint::GetNumberOfLocalMax(AliEMCALDigit **  maxAt, Float_t *
       iDigitN++ ; 
     }
   }
+
   return iDigitN ;
 }
 
@@ -1114,11 +1286,11 @@ Int_t AliEMCALRecPoint::GetPrimaryIndex() const
 void AliEMCALRecPoint::EvalTime(TClonesArray * digits){
   // time is set to the time of the digit with the maximum energy
 
-  Float_t maxE = 0;
-  Int_t maxAt = 0;
+  Float_t maxE  = 0;
+  Int_t   maxAt = 0;
   for(Int_t idig=0; idig < fMulDigit; idig++){
     if(fEnergyList[idig] > maxE){
-      maxE = fEnergyList[idig] ;
+      maxE  = fEnergyList[idig] ;
       maxAt = idig;
     }
   }
@@ -1136,7 +1308,7 @@ void AliEMCALRecPoint::Paint(Option_t *)
   Coord_t x = pos.X()     ;
   Coord_t y = pos.Z()     ;
   Color_t markercolor = 1 ;
-  Size_t  markersize = 1. ;
+  Size_t  markersize  = 1.;
   Style_t markerstyle = 5 ;
   
   if (!gPad->IsBatch()) {
@@ -1154,11 +1326,13 @@ Double_t AliEMCALRecPoint::TmaxInCm(const Double_t e , const Int_t key)
   // e energy in GeV)
   // key  =  0(gamma, default)
   //     !=  0(electron)
-  static Double_t ca = 4.82;  // shower max parameter - first guess; ca=TMath::Log(1000./8.07)
-  static Double_t x0 = 1.23;  // radiation lenght (cm)
-  static Double_t tmax = 0.;   // position of electromagnetic shower max in cm
+  const Double_t ca   = 4.82;  // shower max parameter - first guess; ca=TMath::Log(1000./8.07)
+  Double_t tmax = 0.;    // position of electromagnetic shower max in cm
+
+  Double_t x0   = 1.31;  // radiation lenght (cm)
+  //If old geometry in use
+  if(!((fGeomPtr->GetEMCGeometry()->GetGeoName()).Contains("V1"))) x0 = 1.28;
 
-  tmax = 0.0;
   if(e>0.1) {
     tmax = TMath::Log(e) + ca;
     if      (key==0) tmax += 0.5; 
@@ -1190,24 +1364,24 @@ void AliEMCALRecPoint::Print(Option_t *opt) const
   TString message ; 
   message  = "AliEMCALRecPoint:\n" ;
   message +=  " digits # = " ; 
-  Info("Print", message.Data()) ; 
+  AliInfo(message.Data()) ; 
 
   Int_t iDigit;
   for(iDigit=0; iDigit<fMulDigit; iDigit++)
     printf(" %d ", fDigitsList[iDigit] ) ;  
   printf("\n");
 
-  Info("Print", " Energies = ") ;
+  AliInfo(" Energies = ") ;
   for(iDigit=0; iDigit<fMulDigit; iDigit++) 
     printf(" %f ", fEnergyList[iDigit] ) ;
   printf("\n");
 
-  Info("Print", "\n Abs Ids = ") ;
+  AliInfo("\n Abs Ids = ") ;
   for(iDigit=0; iDigit<fMulDigit; iDigit++) 
     printf(" %i ", fAbsIdList[iDigit] ) ;
   printf("\n");
 
-  Info("Print", " Primaries  ") ;
+  AliInfo(" Primaries  ") ;
   for(iDigit = 0;iDigit < fMulTrack; iDigit++)
     printf(" %d ", fTracksList[iDigit]) ;
 
@@ -1220,15 +1394,14 @@ void AliEMCALRecPoint::Print(Option_t *opt) const
   message += "       Core radius     = %f" ; 
   message += "       Number of primaries %d" ; 
   message += "       Stored at position %d" ; 
-  Info("Print", message.Data(), fClusterType, fMulDigit, fAmp, fCoreEnergy, fCoreRadius, fMulTrack, GetIndexInList() ) ;  
+  AliInfo(Form(message.Data(), fClusterType, fMulDigit, fAmp, fCoreEnergy, fCoreRadius, fMulTrack, GetIndexInList()) ) ;  
 }
 
 //___________________________________________________________
 Double_t  AliEMCALRecPoint::GetPointEnergy() const
 {
   //Returns energy ....
-  static double e;
-  e=0.0;
+  Double_t e=0.0;
   for(int ic=0; ic<GetMultiplicity(); ic++) e += double(fEnergyList[ic]);
   return e;
 }