]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - FMD/AliFMDReconstructor.cxx
Fix last coverity defects (Jochen)
[u/mrichter/AliRoot.git] / FMD / AliFMDReconstructor.cxx
index 0bd09864de5deed6c4e14c04d1423618a5b458bb..b6f4f5f8ec86d1827870bc3b7724894b7a7b08e8 100644 (file)
@@ -39,7 +39,9 @@
 #include "AliFMDParameters.h"              // ALIFMDPARAMETERS_H
 #include "AliFMDAltroMapping.h"            // ALIFMDALTROMAPPING_H
 #include "AliFMDDigit.h"                   // ALIFMDDIGIT_H
+#include "AliFMDSDigit.h"                  // ALIFMDDIGIT_H
 #include "AliFMDReconstructor.h"           // ALIFMDRECONSTRUCTOR_H
+#include "AliFMDRecoParam.h"               // ALIFMDRECOPARAM_H
 #include "AliFMDRawReader.h"               // ALIFMDRAWREADER_H
 #include "AliFMDRecPoint.h"               // ALIFMDMULTNAIIVE_H
 #include "AliESDEvent.h"                  // ALIESDEVENT_H
 #include <TH2.h>
 #include <TFile.h>
 #include <climits>
-// Import revertexer into a private namespace (to prevent conflicts) 
-namespace { 
-# include "AliFMDESDRevertexer.h"
-}
+#include "AliFMDESDRevertexer.h"
 
 
 class AliRawReader;
@@ -77,7 +76,7 @@ AliFMDReconstructor::AliFMDReconstructor()
     fAngleCorrect(kTRUE),
     fVertexType(kNoVertex),
     fESD(0x0),
-    fDiagnostics(kTRUE),
+    fDiagnostics(kFALSE),
     fDiagStep1(0), 
     fDiagStep2(0),
     fDiagStep3(0),
@@ -88,53 +87,10 @@ AliFMDReconstructor::AliFMDReconstructor()
   SetNoiseFactor();
   SetAngleCorrect();
   if (AliDebugLevel() > 0) fDiagnostics = kTRUE;
-}
-  
-
-//____________________________________________________________________
-AliFMDReconstructor::AliFMDReconstructor(const AliFMDReconstructor& other) 
-  : AliReconstructor(), 
-    fMult(other.fMult),
-    fNMult(other.fNMult),
-    fTreeR(other.fTreeR),
-    fCurrentVertex(other.fCurrentVertex),
-    fESDObj(other.fESDObj),
-    fNoiseFactor(other.fNoiseFactor),
-    fAngleCorrect(other.fAngleCorrect),
-    fVertexType(other.fVertexType),
-    fESD(other.fESD),
-    fDiagnostics(other.fDiagnostics),
-    fDiagStep1(other.fDiagStep1), 
-    fDiagStep2(other.fDiagStep2),
-    fDiagStep3(other.fDiagStep3),
-    fDiagStep4(other.fDiagStep4),
-    fDiagAll(other.fDiagAll) 
-{
-  // Copy constructor 
-}
-  
-
-//____________________________________________________________________
-AliFMDReconstructor&
-AliFMDReconstructor::operator=(const AliFMDReconstructor& other) 
-{
-  // Assignment operator
-  fMult          = other.fMult;
-  fNMult         = other.fNMult;
-  fTreeR         = other.fTreeR;
-  fCurrentVertex = other.fCurrentVertex;
-  fESDObj        = other.fESDObj;
-  fNoiseFactor   = other.fNoiseFactor;
-  fAngleCorrect  = other.fAngleCorrect;
-  fVertexType    = other.fVertexType;
-  fESD           = other.fESD;
-  fDiagnostics   = other.fDiagnostics;
-  fDiagStep1     = other.fDiagStep1;
-  fDiagStep2     = other.fDiagStep2;
-  fDiagStep3     = other.fDiagStep3;
-  fDiagStep4     = other.fDiagStep4;
-  fDiagAll       = other.fDiagAll;
-  return *this;
+  for(Int_t det = 1; det<=3; det++) {
+    fZS[det-1]       = kFALSE;
+    fZSFactor[det-1] = 0;
+  }
 }
 
 //____________________________________________________________________
@@ -164,7 +120,7 @@ AliFMDReconstructor::Init()
   // Current vertex position
   fCurrentVertex = 0;
   // Create array of reconstructed strip multiplicities 
-  fMult = new TClonesArray("AliFMDRecPoint", 51200);
+  // fMult = new TClonesArray("AliFMDRecPoint", 51200);
   // Create ESD output object 
   fESDObj = new AliESDFMD;
   
@@ -205,49 +161,141 @@ AliFMDReconstructor::ConvertDigits(AliRawReader* reader,
                                   TTree* digitsTree) const
 {
   // Convert Raw digits to AliFMDDigit's in a tree 
-  AliFMDDebug(2, ("Reading raw data into digits tree"));
+  AliFMDDebug(1, ("Reading raw data into digits tree"));
+  if (!digitsTree) { 
+    AliError("No digits tree passed");
+    return;
+  }
+  static TClonesArray* array = new TClonesArray("AliFMDDigit");
+  digitsTree->Branch("FMD", &array);
+  array->Clear();
+  
   AliFMDRawReader rawRead(reader, digitsTree);
   // rawRead.SetSampleRate(fFMD->GetSampleRate());
-  rawRead.Exec();
+  // rawRead.Exec();
+  rawRead.ReadAdcs(array);
+
+  Int_t nWrite = digitsTree->Fill();
+  AliFMDDebug(1, ("Got a grand total of %d digits, wrote %d bytes to tree", 
+                  array->GetEntriesFast(), nWrite));
+
+  
   AliFMDAltroMapping* map = AliFMDParameters::Instance()->GetAltroMap();
   for (size_t i = 1; i <= 3; i++) { 
-    fZS[i]       = rawRead.IsZeroSuppressed(map->Detector2DDL(i));
-    fZSFactor[i] = rawRead.NoiseFactor(map->Detector2DDL(i));
+    fZS[i-1]       = rawRead.IsZeroSuppressed(map->Detector2DDL(i));
+    fZSFactor[i-1] = rawRead.NoiseFactor(map->Detector2DDL(i));
   }
 }
 
 //____________________________________________________________________
 void 
-AliFMDReconstructor::GetVertex() const
+AliFMDReconstructor::GetVertex(AliESDEvent* esd) const
 {
   // Return the vertex to use. 
   // This is obtained from the ESD object. 
   // If not found, a warning is issued.
   fVertexType    = kNoVertex;
   fCurrentVertex = 0;
-  if (fESD) {
-    const AliESDVertex* vertex = fESD->GetPrimaryVertex();
-    if (!vertex)        vertex = fESD->GetPrimaryVertexSPD();
-    if (!vertex)        vertex = fESD->GetPrimaryVertexTPC();
-    if (!vertex)        vertex = fESD->GetVertex();
-
-    if (vertex) {
-      AliFMDDebug(2, ("Got %s (%s) from ESD: %f", 
-                     vertex->GetName(), vertex->GetTitle(), vertex->GetZv()));
-      fCurrentVertex = vertex->GetZv();
-      fVertexType    = kESDVertex;
-      return;
-    }
-    else if (fESD->GetESDTZERO()) { 
-      AliFMDDebug(2, ("Got primary vertex from T0: %f", fESD->GetT0zVertex()));
-      fCurrentVertex = fESD->GetT0zVertex();
-      fVertexType    = kESDVertex;
-      return;
-    }
+  if (!esd) return;
+  
+  const AliESDVertex* vertex = esd->GetPrimaryVertex();
+  if (!vertex)        vertex = esd->GetPrimaryVertexSPD();
+  if (!vertex)        vertex = esd->GetPrimaryVertexTPC();
+  if (!vertex)        vertex = esd->GetVertex();
+
+  if (vertex) {
+    AliFMDDebug(2, ("Got %s (%s) from ESD: %f", 
+                   vertex->GetName(), vertex->GetTitle(), vertex->GetZv()));
+    fCurrentVertex = vertex->GetZv();
+    fVertexType    = kESDVertex;
+    return;
+  }
+  else if (esd->GetESDTZERO()) { 
+    AliFMDDebug(2, ("Got primary vertex from T0: %f", esd->GetT0zVertex()));
+    fCurrentVertex = esd->GetT0zVertex();
+    fVertexType    = kESDVertex;
+    return;
   }
   AliWarning("Didn't get any vertex from ESD or generator");
 }
   
+//____________________________________________________________________
+Int_t
+AliFMDReconstructor::GetIdentifier() const
+{
+  // Get the detector identifier. 
+  // Note the actual value is cached so that we do not 
+  // need to do many expensive string comparisons. 
+  static Int_t idx = AliReconstruction::GetDetIndex(GetDetectorName());
+  return idx;
+}
+
+//____________________________________________________________________
+const AliFMDRecoParam*
+AliFMDReconstructor::GetParameters() const
+{
+  // Get the reconstruction parameters. 
+  // 
+  // Return: 
+  //   Pointer to reconstruction parameters or null if not found or wrong type
+  Int_t iDet = GetIdentifier(); // Was 12 - but changed on Cvetans request
+  const AliDetectorRecoParam* params = AliReconstructor::GetRecoParam(iDet);
+  if (!params || params->IsA() != AliFMDRecoParam::Class()) return 0;
+  return static_cast<const AliFMDRecoParam*>(params);
+}
+
+//____________________________________________________________________
+void 
+AliFMDReconstructor::UseRecoParam(Bool_t set) const
+{
+  static Float_t savedNoiseFactor  = fNoiseFactor;
+  static Bool_t  savedAngleCorrect = fAngleCorrect;
+  if (set) { 
+    const AliFMDRecoParam* params  = GetParameters();
+    if (params) { 
+      fNoiseFactor  = params->NoiseFactor();
+      fAngleCorrect = params->AngleCorrect();
+    }
+    return;
+  }
+  fNoiseFactor  = savedNoiseFactor;
+  fAngleCorrect = savedAngleCorrect;
+}
+  
+  
+//____________________________________________________________________
+void 
+AliFMDReconstructor::MarkDeadChannels(AliESDFMD* esd) const
+{
+  // Loop over all entries of the ESD and mark 
+  // those that are dead as such 
+  // - otherwise put in the zero signal. 
+  AliFMDParameters* param = AliFMDParameters::Instance();
+
+  for (UShort_t d = 1; d <= 3; d++) { 
+    UShort_t nR = (d == 1 ? 1 : 2);
+
+    for (UShort_t q = 0; q < nR; q++) {
+      Char_t   r  = (q == 0 ? 'I' : 'O');
+      UShort_t nS = (q == 0 ?  20 :  40);
+      UShort_t nT = (q == 0 ? 512 : 256);
+
+      for (UShort_t s = 0; s < nS; s++) { 
+       for (UShort_t t = 0; t < nT; t++) {
+         if (param->IsDead(d, r, s, t)) { 
+           AliDebug(5, Form("Marking FMD%d%c[%2d,%3d] as dead", d, r, s, t));
+           esd->SetMultiplicity(d, r, s, t, AliESDFMD::kInvalidMult);
+           esd->SetEta(d, r, s, t, AliESDFMD::kInvalidEta);
+         }
+         else if (esd->Multiplicity(d, r, s, t) == AliESDFMD::kInvalidMult) {
+           AliDebug(10, Form("Setting null signal in FMD%d%c[%2d,%3d]", d, r, s, t));
+           esd->SetMultiplicity(d, r, s, t, 0);
+         }
+       }
+      }
+    }
+  }
+}
 
 //____________________________________________________________________
 void 
@@ -258,14 +306,16 @@ AliFMDReconstructor::Reconstruct(AliRawReader* reader, TTree*) const
   // 
   // Parameters: 
   //   reader  Raw event reader 
-  //   ctree    Not used. 
+  //   ctree    Not used - 'cluster tree' to store rec-points in. 
+  AliFMDDebug(1, ("Reconstructing from raw reader"));
   AliFMDRawReader rawReader(reader, 0);
 
   UShort_t det, sec, str, fac;
   Short_t  adc, oldDet = -1;
   Bool_t   zs;
   Char_t   rng;
-    
+  UseRecoParam(kTRUE);
   while (rawReader.NextSignal(det, rng, sec, str, adc, zs, fac)) { 
     if (det != oldDet) { 
       fZS[det-1]       = zs;
@@ -274,6 +324,38 @@ AliFMDReconstructor::Reconstruct(AliRawReader* reader, TTree*) const
     }
     ProcessSignal(det, rng, sec, str, adc);
   }
+  UseRecoParam(kFALSE);
+  
+}
+
+//____________________________________________________________________
+void 
+AliFMDReconstructor::Digitize(AliRawReader* reader, TClonesArray* sdigits) const
+{
+  // Reconstruct directly from raw data (no intermediate output on
+  // digit tree or rec point tree).  
+  // 
+  // Parameters: 
+  //   reader  Raw event reader 
+  //   ctree    Not used. 
+  AliFMDRawReader rawReader(reader, 0);
+
+  UShort_t det, sec, str, sam, rat, fac;
+  Short_t  adc, oldDet = -1;
+  Bool_t   zs;
+  Char_t   rng;
+    
+  UseRecoParam(kTRUE);
+  while (rawReader.NextSample(det, rng, sec, str, sam, rat, adc, zs, fac)) { 
+    if (!rawReader.SelectSample(sam, rat)) continue;
+    if (det != oldDet) { 
+      fZS[det-1]       = zs;
+      fZSFactor[det-1] = fac;
+      oldDet           = det;
+    }
+    DigitizeSignal(sdigits, det, rng, sec, str, sam, adc);
+  }
+  UseRecoParam(kFALSE);
 }
 
 //____________________________________________________________________
@@ -290,17 +372,22 @@ AliFMDReconstructor::Reconstruct(TTree* digitsTree,
   //   digitsTree      Pointer to a tree containing digits 
   //   clusterTree     Pointer to output tree 
   // 
-  AliFMDDebug(2, ("Reconstructing from digits in a tree"));
-  GetVertex();
+  if (!fMult) fMult = new TClonesArray("AliFMDRecPoint");
+
+  AliFMDDebug(1, ("Reconstructing from digits in a tree"));
+  GetVertex(fESD);
 
-  TBranch *digitBranch = digitsTree->GetBranch("FMD");
+
+
+  static TClonesArray* digits = new TClonesArray("AliFMDDigit");
+  TBranch*      digitBranch   = digitsTree->GetBranch("FMD");
   if (!digitBranch) {
     Error("Exec", "No digit branch for the FMD found");
     return;
   }
-  TClonesArray* digits = new TClonesArray("AliFMDDigit");
   digitBranch->SetAddress(&digits);
 
+  if (digits)  digits->Clear();
   if (fMult)   fMult->Clear();
   if (fESDObj) fESDObj->Clear();
   
@@ -312,12 +399,14 @@ AliFMDReconstructor::Reconstruct(TTree* digitsTree,
   digitBranch->GetEntry(0);
   
   AliFMDDebug(5, ("Processing digits"));
+  UseRecoParam(kTRUE);
   ProcessDigits(digits);
+  UseRecoParam(kFALSE);
 
   Int_t written = clusterTree->Fill();
   AliFMDDebug(10, ("Filled %d bytes into cluster tree", written));
-  digits->Delete();
-  delete digits;
+  // digits->Delete();
+  // delete digits;
 }
  
 
@@ -333,7 +422,7 @@ AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
   //    digits Array of digits
   // 
   Int_t nDigits = digits->GetEntries();
-  AliFMDDebug(1, ("Got %d digits", nDigits));
+  AliFMDDebug(2, ("Got %d digits", nDigits));
   fESDObj->SetNoiseFactor(fNoiseFactor);
   fESDObj->SetAngleCorrected(fAngleCorrect);
   for (Int_t i = 0; i < nDigits; i++) {
@@ -352,7 +441,7 @@ AliFMDReconstructor::ProcessDigit(AliFMDDigit* digit) const
   UShort_t sec = digit->Sector();
   UShort_t str = digit->Strip();
   Short_t  adc = digit->Counts();
-  
   ProcessSignal(det, rng, sec, str, adc);
 }
 
@@ -376,7 +465,7 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
   AliFMDParameters* param  = AliFMDParameters::Instance();
   // Check that the strip is not marked as dead 
   if (param->IsDead(det, rng, sec, str)) {
-    AliFMDDebug(10, ("FMD%d%c[%2d,%3d] is dead", det, rng, sec, str));
+    AliFMDDebug(1, ("FMD%d%c[%2d,%3d] is dead", det, rng, sec, str));
     return;
   }
   
@@ -397,8 +486,13 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
   // Make rough multiplicity 
   Double_t mult     = Energy2Multiplicity(det, rng, sec, str, edep);
   // Get rid of nonsense mult
+  //if (mult > 20) { 
+  //  AliWarning(Form("The mutliplicity in FMD%d%c[%2d,%3d]=%f > 20 "
+  //               "(ADC: %d, Energy: %f)", det, rng, sec, str, mult, 
+  //               counts, edep));
+  // }
   if (mult < 0)  return; 
-  AliFMDDebug(5, ("FMD%d%c[%2d,%3d]: "
+  AliFMDDebug(10, ("FMD%d%c[%2d,%3d]: "
                    "ADC: %d, Counts: %d, Energy: %f, Mult: %f",
                  det, rng, sec, str, adc, counts, edep, mult));
   
@@ -418,6 +512,100 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
 
 }
 
+//____________________________________________________________________
+void
+AliFMDReconstructor::DigitizeSignal(TClonesArray* sdigits, 
+                                   UShort_t det, 
+                                   Char_t   rng, 
+                                   UShort_t sec, 
+                                   UShort_t str, 
+                                   UShort_t /* sam */,
+                                   Short_t  adc) const
+{
+  // Process the signal from a single strip 
+  // 
+  // Parameters: 
+  //    det    Detector ID
+  //    rng    Ring ID
+  //    sec    Sector ID
+  //    rng    Strip ID
+  //    adc     ADC counts
+  // 
+  AliFMDParameters* param  = AliFMDParameters::Instance();
+  // Check that the strip is not marked as dead 
+  if (param->IsDead(det, rng, sec, str)) {
+    AliFMDDebug(10, ("FMD%d%c[%2d,%3d] is dead", det, rng, sec, str));
+    return;
+  }
+  
+  // Substract pedestal. 
+  UShort_t counts   = SubtractPedestal(det, rng, sec, str, adc);
+  if(counts == USHRT_MAX || counts == 0) return;
+  
+    // Gain match digits. 
+  Double_t edep     = Adc2Energy(det, rng, sec, str, counts);
+  // Get rid of nonsense energy
+  if(edep < 0)  return;
+
+  Int_t n = sdigits->GetEntriesFast();
+  // AliFMDSDigit* sdigit = 
+  new ((*sdigits)[n]) 
+    AliFMDSDigit(det, rng, sec, str, edep, counts, counts, counts, counts);
+  // sdigit->SetCount(sam, counts);
+}
+
+//____________________________________________________________________
+UShort_t
+AliFMDReconstructor::SubtractPedestal(UShort_t det, 
+                                     Char_t   rng, 
+                                     UShort_t sec, 
+                                     UShort_t str, 
+                                     UShort_t adc, 
+                                     Float_t  noiseFactor,
+                                     Bool_t   zsEnabled, 
+                                     UShort_t zsNoiseFactor) const
+{
+  AliFMDParameters* param  = AliFMDParameters::Instance();
+  Float_t           ped    = (zsEnabled ? 0 : 
+                               param->GetPedestal(det, rng, sec, str));
+  Float_t           noise  = param->GetPedestalWidth(det, rng, sec, str);
+  if(ped < 0 || noise < 0) { 
+    AliWarningClass(Form("Invalid pedestal (%f) or noise (%f) "
+                        "for FMD%d%c[%02d,%03d]", 
+                   ped, noise, det, rng, sec, str));
+    return USHRT_MAX;
+  }
+  AliDebugClass(15, Form("Subtracting pedestal for FMD%d%c[%2d,%3d]=%4d "
+                        "(%s w/factor %d, noise factor %f, "
+                        "pedestal %8.2f+/-%8.2f)",
+                        det, rng, sec, str, adc, 
+                        (zsEnabled ? "zs'ed" : "straight"), 
+                        zsNoiseFactor, noiseFactor, ped, noise));
+
+  Int_t counts = adc + Int_t(zsEnabled ? zsNoiseFactor * noise : - ped);
+  counts =  TMath::Max(Int_t(counts), 0);
+  // Calculate the noise factor for suppressing remenants of the noise
+  // peak.  If we have done on-line zero suppression, we only check
+  // for noise signals that are larger than the suppressed noise.  If
+  // the noise factor used on line is larger than the factor used
+  // here, we do not do this check at all.  
+  // 
+  // For example:
+  //    Online factor  |  Read factor |  Result 
+  //    ---------------+--------------+-------------------------------
+  //           2       |      3       | Check if signal > 1 * noise
+  //           3       |      3       | Check if signal > 0
+  //           3       |      2       | Check if signal > 0
+  //
+  // In this way, we make sure that we do not suppress away too much
+  // data, and that the read-factor is the most stringent cut. 
+  Float_t nf = TMath::Max(0.F, noiseFactor - (zsEnabled ? zsNoiseFactor : 0));
+  if (counts < noise * nf) counts = 0;
+  if (counts > 0) AliDebugClass(15, "Got a hit strip");
+
+  UShort_t ret = counts < 0 ? 0 : counts;
+  return ret;
+}
 
 
 //____________________________________________________________________
@@ -439,29 +627,12 @@ AliFMDReconstructor::SubtractPedestal(UShort_t det,
   // Return:
   //    Pedestal subtracted signal or USHRT_MAX in case of problems 
   //
-  AliFMDParameters* param  = AliFMDParameters::Instance();
-  Bool_t            zs     = fZS[det-1];
-  UShort_t          fac    = fZSFactor[det-1];
-  Float_t           ped    = (zs ? 0 : 
-                             param->GetPedestal(det, rng, sec, str));
-  Float_t           noise  = param->GetPedestalWidth(det, rng, sec, str);
-  if(ped < 0 || noise < 0) { 
-    AliWarning(Form("Invalid pedestal (%f) or noise (%f) "
-                   "for FMD%d%c[%02d,%03d]", ped, noise, det, rng, sec, str));
-    return USHRT_MAX;
-  }
-
-  AliFMDDebug(5, ("Subtracting pedestal %f from signal %d", ped, adc));
-  // if (digit->Count3() > 0)      adc = digit->Count3();
-  // else if (digit->Count2() > 0) adc = digit->Count2();
-  // else                          adc = digit->Count1();
-  Int_t counts = adc + Int_t(zs ? fac * noise : - ped);
-  counts       = TMath::Max(Int_t(counts), 0);
-  if (counts < noise * fNoiseFactor) counts = 0;
-  if (counts > 0) AliFMDDebug(15, ("Got a hit strip"));
+  UShort_t counts = SubtractPedestal(det, rng, sec, str, adc, 
+                                    fNoiseFactor, fZS[det-1], 
+                                    fZSFactor[det-1]);
   if (fDiagStep1) fDiagStep1->Fill(adc, counts);
   
-  return  UShort_t(counts);
+  return counts;
 }
 
 //____________________________________________________________________
@@ -470,7 +641,6 @@ AliFMDReconstructor::Adc2Energy(UShort_t det,
                                Char_t   rng, 
                                UShort_t sec, 
                                UShort_t str, 
-                               Float_t  eta, 
                                UShort_t count) const
 {
   // Converts number of ADC counts to energy deposited. 
@@ -504,7 +674,6 @@ AliFMDReconstructor::Adc2Energy(UShort_t det,
   //    rng    Ring ID
   //    sec    Sector ID
   //    rng    Strip ID
-  //    eta     Psuedo-rapidity
   //    counts  Number of ADC counts over pedestal
   // Return 
   //    The energy deposited in a single strip, or -1 in case of problems
@@ -523,6 +692,56 @@ AliFMDReconstructor::Adc2Energy(UShort_t det,
 
   Double_t edep  = ((count * param->GetEdepMip()) 
                    / (gain * param->GetDACPerMIP()));
+  return edep;
+}
+
+//____________________________________________________________________
+Float_t
+AliFMDReconstructor::Adc2Energy(UShort_t det, 
+                               Char_t   rng, 
+                               UShort_t sec, 
+                               UShort_t str, 
+                               Float_t  eta, 
+                               UShort_t count) const
+{
+  // Converts number of ADC counts to energy deposited. 
+  // Note, that this member function can be overloaded by derived
+  // classes to do strip-specific look-ups in databases or the like,
+  // to find the proper gain for a strip. 
+  // 
+  // In the first simple version, we calculate the energy deposited as 
+  // 
+  //    EnergyDeposited = cos(theta) * gain * count
+  // 
+  // where 
+  // 
+  //           Pre_amp_MIP_Range
+  //    gain = ----------------- * Energy_deposited_per_MIP
+  //           ADC_channel_size    
+  // 
+  // is constant and the same for all strips.
+  //
+  // For the production we use the conversion measured in the NBI lab.
+  // The total conversion is then:
+  // 
+  //    gain = ADC / DAC
+  // 
+  //                  EdepMip * count
+  //      => energy = ----------------
+  //                  gain * DACPerADC
+  // 
+  // Parameters: 
+  //    det    Detector ID
+  //    rng    Ring ID
+  //    sec    Sector ID
+  //    rng    Strip ID
+  //    eta     Psuedo-rapidity
+  //    counts  Number of ADC counts over pedestal
+  // Return 
+  //    The energy deposited in a single strip, or -1 in case of problems
+  //
+  Double_t edep = Adc2Energy(det, rng, sec, str, count);
+  
   if (fDiagStep2) fDiagStep2->Fill(count, edep);  
   if (fAngleCorrect) {
     Double_t theta = 2 * TMath::ATan(TMath::Exp(-eta));
@@ -599,17 +818,31 @@ AliFMDReconstructor::PhysicalCoordinates(UShort_t det,
   //    phi     On return, contains the azimuthal angle of the strip
   // 
   AliFMDGeometry* geom = AliFMDGeometry::Instance();
-  Double_t x, y, z, r, theta;
+  Double_t x, y, z, r, theta, deta, dphi;
   geom->Detector2XYZ(det, rng, sec, str, x, y, z);
+
   // Correct for vertex offset. 
-  z     += fCurrentVertex;
-  phi   =  TMath::ATan2(y, x);
-  r     =  TMath::Sqrt(y * y + x * x);
-  theta =  TMath::ATan2(r, z);
-  eta   = -TMath::Log(TMath::Tan(theta / 2));
+  z     -= fCurrentVertex;
+  AliFMDGeometry::XYZ2REtaPhiTheta(x, y, z, r, deta, dphi, theta);
+  eta = deta;
+  phi = dphi;
+}
+
+namespace { 
+  class ESDPrinter : public AliESDFMD::ForOne
+  {
+  public:
+    ESDPrinter() {}
+    Bool_t operator()(UShort_t d, Char_t r, UShort_t s, UShort_t t, 
+                     Float_t m, Float_t e)
+    {
+      if (m > 0 && m != AliESDFMD::kInvalidMult) 
+       printf("  FMD%d%c[%2d,%3d] = %6.3f / %6.3f\n", d, r, s, t, m, e);
+      return kTRUE;
+    }
+  };
 }
 
-      
 
 //____________________________________________________________________
 void 
@@ -623,14 +856,22 @@ AliFMDReconstructor::FillESD(TTree*  /* digitsTree */,
   AliFMDDebug(2, ("Calling FillESD with two trees and one ESD"));
   // fESDObj->Print();
 
+  // Fix up ESD so that only truely dead channels get the kInvalidMult flag. 
+  MarkDeadChannels(fESDObj);
+
   Double_t oldVz = fCurrentVertex;
-  GetVertex();
+  GetVertex(esd);
   if (fVertexType != kNoVertex) { 
     AliFMDDebug(2, ("Revertexing the ESD data to vz=%f (was %f)",
                    fCurrentVertex, oldVz));
     AliFMDESDRevertexer revertexer;
     revertexer.Revertex(fESDObj, fCurrentVertex);
   }
+  
+  if (AliDebugLevel() > 10) { 
+    ESDPrinter p;
+    fESDObj->ForEach(p);
+  }
 
   if (esd) { 
     AliFMDDebug(2, ("Writing FMD data to ESD tree"));
@@ -644,7 +885,7 @@ AliFMDReconstructor::FillESD(TTree*  /* digitsTree */,
   // - That's OK.  We just use it for the name of the directory -
   // nothing else.  Christian
   Int_t evno = esd->GetEventNumberInFile(); 
-  AliFMDDebug(1, ("Writing diagnostics histograms to FMD.Diag.root/%03d",evno));
+  AliFMDDebug(3, ("Writing diagnostics histograms to FMD.Diag.root/%03d",evno));
   TFile f("FMD.Diag.root", (first ? "RECREATE" : "UPDATE"));
   first = false;
   f.cd(); 
@@ -675,7 +916,6 @@ AliFMDReconstructor::FillESD(AliRawReader*, TTree* clusterTree,
   TTree* dummy = 0;
   FillESD(dummy, clusterTree, esd);
 }
-
 //____________________________________________________________________
 //
 // EOF