]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - FMD/AliFMDReconstructor.cxx
coverity #24442, add check on nullness of primary particles pointer
[u/mrichter/AliRoot.git] / FMD / AliFMDReconstructor.cxx
index 2f655375dcd67c2f246e397ae2973d4261a5ad72..38bf2945ade2ca479d7c0f32854d2571995a3076 100644 (file)
@@ -1,3 +1,16 @@
+//____________________________________________________________________
+//
+// This is a class that constructs AliFMDRecPoint objects from of Digits
+// This class reads either digits from a TClonesArray or raw data from 
+// a DDL file (or similar), and stores the read ADC counts in an
+// internal cache (fAdcs).   The rec-points are made via the naiive
+// method. 
+//
+//-- Authors: Evgeny Karpechev(INR) and Alla Maevsksia
+//  Latest changes by Christian Holm Christensen <cholm@nbi.dk>
+//
+//
+//____________________________________________________________________
 /**************************************************************************
  * Copyright(c) 1998-1999, ALICE Experiment at CERN, All rights reserved. *
  *                                                                        *
  * provided "as is" without express or implied warranty.                  *
  **************************************************************************/
 /* $Id$ */
-/** @file    AliFMDReconstructor.cxx
-    @author  Christian Holm Christensen <cholm@nbi.dk>
-    @date    Mon Mar 27 12:47:09 2006
-    @brief   FMD reconstruction 
+/** 
+ * @file    AliFMDReconstructor.cxx
+ * @author  Christian Holm Christensen <cholm@nbi.dk>
+ * @date    Mon Mar 27 12:47:09 2006
+ * @brief   FMD reconstruction 
 */
-//____________________________________________________________________
-//
-// This is a class that constructs AliFMDRecPoint objects from of Digits
-// This class reads either digits from a TClonesArray or raw data from 
-// a DDL file (or similar), and stores the read ADC counts in an
-// internal cache (fAdcs).   The rec-points are made via the naiive
-// method. 
-//
-//-- Authors: Evgeny Karpechev(INR) and Alla Maevsksia
-//  Latest changes by Christian Holm Christensen <cholm@nbi.dk>
-//
-//
-//____________________________________________________________________
 
 // #include <AliLog.h>                        // ALILOG_H
 // #include <AliRun.h>                        // ALIRUN_H
@@ -76,12 +77,14 @@ AliFMDReconstructor::AliFMDReconstructor()
     fAngleCorrect(kTRUE),
     fVertexType(kNoVertex),
     fESD(0x0),
-    fDiagnostics(kTRUE),
+    fDiagnostics(kFALSE),
     fDiagStep1(0), 
     fDiagStep2(0),
     fDiagStep3(0),
     fDiagStep4(0),
-    fDiagAll(0)
+    fDiagAll(0),
+    fBad(0), 
+    fZombie(false)
 {
   // Make a new FMD reconstructor object - default CTOR.  
   SetNoiseFactor();
@@ -115,44 +118,59 @@ AliFMDReconstructor::Init()
 
   // Initialize the parameters
   AliFMDParameters* param = AliFMDParameters::Instance();
-  param->Init();
+  if (param->Init() != 0) {
+    AliError("Failed to initialize parameters, making zombie");
+    fZombie = true;
+  }
+  else 
+    fZombie = false;
   
   // 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;
   
   // Check if we need diagnostics histograms 
   if (!fDiagnostics) return;
   AliInfo("Making diagnostics histograms");
-  fDiagStep1   = new TH2I("diagStep1", "Read ADC vs. Noise surpressed ADC",
-                       1024, -.5, 1023.5, 1024, -.5, 1023.5);
-  fDiagStep1->SetDirectory(0);
-  fDiagStep1->GetXaxis()->SetTitle("ADC (read)");
-  fDiagStep1->GetYaxis()->SetTitle(Form("ADC (noise surpressed %4.f)", 
-                                       fNoiseFactor));
-  fDiagStep2  = new TH2F("diagStep2",  "ADC vs Edep deduced",
-                       1024, -.5, 1023.5, 100, 0, 2);
-  fDiagStep2->SetDirectory(0);
-  fDiagStep2->GetXaxis()->SetTitle("ADC (noise surpressed)");
-  fDiagStep2->GetYaxis()->SetTitle("#Delta E [GeV]");
-  fDiagStep3  = new TH2F("diagStep3",  "Edep vs Edep path corrected",
-                       100, 0., 2., 100, 0., 2.);
-  fDiagStep3->SetDirectory(0);
-  fDiagStep3->GetXaxis()->SetTitle("#Delta E [GeV]");
-  fDiagStep3->GetYaxis()->SetTitle("#Delta E/#Delta x #times #delta x [GeV]");
-  fDiagStep4  = new TH2F("diagStep4",  "Edep vs Multiplicity deduced", 
-                       100, 0., 2., 100, -.1, 19.9);
-  fDiagStep4->SetDirectory(0);
-  fDiagStep4->GetXaxis()->SetTitle("#Delta E/#Delta x #times #delta x [GeV]");
-  fDiagStep4->GetYaxis()->SetTitle("Multiplicity");
-  fDiagAll    = new TH2F("diagAll",    "Read ADC vs Multiplicity deduced", 
-                        1024, -.5, 1023.5, 100, -.1, 19.9);
-  fDiagAll->SetDirectory(0);
-  fDiagAll->GetXaxis()->SetTitle("ADC (read)");
-  fDiagAll->GetYaxis()->SetTitle("Multiplicity");
+  if (!fDiagStep1) {
+    fDiagStep1   = new TH2I("diagStep1", "Read ADC vs. Noise surpressed ADC",
+                           1024, -.5, 1023.5, 1024, -.5, 1023.5);
+    fDiagStep1->SetDirectory(0);
+    fDiagStep1->GetXaxis()->SetTitle("ADC (read)");
+    fDiagStep1->GetYaxis()->SetTitle(Form("ADC (noise surpressed %4.f)", 
+                                         fNoiseFactor));
+  }
+  if (!fDiagStep2) {
+    fDiagStep2  = new TH2F("diagStep2",  "ADC vs Edep deduced",
+                          1024, -.5, 1023.5, 100, 0, 2);
+    fDiagStep2->SetDirectory(0);
+    fDiagStep2->GetXaxis()->SetTitle("ADC (noise surpressed)");
+    fDiagStep2->GetYaxis()->SetTitle("#Delta E [GeV]");
+  }
+  if (!fDiagStep3) {
+    fDiagStep3  = new TH2F("diagStep3",  "Edep vs Edep path corrected",
+                          100, 0., 2., 100, 0., 2.);
+    fDiagStep3->SetDirectory(0);
+    fDiagStep3->GetXaxis()->SetTitle("#Delta E [GeV]");
+    fDiagStep3->GetYaxis()->SetTitle("#Delta E/#Delta x #times #delta x [GeV]");
+  }
+  if (!fDiagStep4) {
+    fDiagStep4  = new TH2F("diagStep4",  "Edep vs Multiplicity deduced", 
+                          100, 0., 2., 100, -.1, 19.9);
+    fDiagStep4->SetDirectory(0);
+    fDiagStep4->GetXaxis()->SetTitle("#Delta E/#Delta x #times #delta x [GeV]");
+    fDiagStep4->GetYaxis()->SetTitle("Multiplicity");
+    fDiagAll    = new TH2F("diagAll",    "Read ADC vs Multiplicity deduced", 
+                          1024, -.5, 1023.5, 100, -.1, 19.9);
+  }
+  if (fDiagAll) {
+    fDiagAll->SetDirectory(0);
+    fDiagAll->GetXaxis()->SetTitle("ADC (read)");
+    fDiagAll->GetYaxis()->SetTitle("Multiplicity");
+  }
 }
 
 //____________________________________________________________________
@@ -161,14 +179,35 @@ AliFMDReconstructor::ConvertDigits(AliRawReader* reader,
                                   TTree* digitsTree) const
 {
   // Convert Raw digits to AliFMDDigit's in a tree 
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
   AliFMDDebug(1, ("Reading raw data into digits tree"));
+  if (!digitsTree) { 
+    AliError("No digits tree passed");
+    return;
+  }
+  static TClonesArray* array = 0;
+  if (!array) 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();
+  AliDebugF(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));
+  for (UShort_t i = 1; i <= 3; i++) { 
+    fZS[i-1]       = rawRead.IsZeroSuppressed(map->Detector2DDL(i));
+    fZSFactor[i-1] = rawRead.NoiseFactor(map->Detector2DDL(i));
+    AliDebugF(2, "Noise factor for FMD%hu: %d", i, fZSFactor[i-1]);
   }
 }
 
@@ -208,14 +247,22 @@ AliFMDReconstructor::GetVertex(AliESDEvent* esd) const
 Int_t
 AliFMDReconstructor::GetIdentifier() const
 {
-  return AliReconstruction::GetDetIndex(GetDetectorName());
+  // 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
 {
-  Int_t iDet = 12; // GetIdentifier();
+  // 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);
@@ -225,6 +272,14 @@ AliFMDReconstructor::GetParameters() const
 void 
 AliFMDReconstructor::UseRecoParam(Bool_t set) const
 {
+  // 
+  // Set-up reconstructor to use values from reconstruction
+  // parameters, if present, for this event.   If the argument @a set
+  // is @c false, then restore preset values. 
+  // 
+  // Parameters:
+  //    set 
+  //  
   static Float_t savedNoiseFactor  = fNoiseFactor;
   static Bool_t  savedAngleCorrect = fAngleCorrect;
   if (set) { 
@@ -240,19 +295,108 @@ AliFMDReconstructor::UseRecoParam(Bool_t set) const
 }
   
   
+//____________________________________________________________________
+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++) {
+         // A strip can be marked `bad' for two reasons: 
+         // 
+         // - The raw reader fails to read the value 
+         // - The strip is marked in OCDB as bad (IsDead) 
+         // 
+         // Hence, a dead strip will always be marked kInvalid here, 
+         // while a non-dead bad-read strip will be filled with 0. 
+         if (fBad(d, r, s, t)) { 
+           AliDebugF(5, "Marking FMD%d%c[%2d,%3d] as bad", d, r, s, t);
+           esd->SetMultiplicity(d, r, s, t, AliESDFMD::kInvalidMult);
+         }         
+         if (param->IsDead(d, r, s, t)) { 
+           AliDebugF(5, "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) {
+           AliDebugF(20, "Setting null signal in FMD%d%c[%2d,%3d]", 
+                     d, r, s, t);
+           esd->SetMultiplicity(d, r, s, t, 0);
+         }
+         else {
+           AliDebugF(10, "Setting FMD%d%c[%2d,%3d]=%f",d,r,s,t,
+                     esd->Multiplicity(d, r, s, t));
+         }
+       }
+      }
+    }
+  }
+}
+
+//____________________________________________________________________
+Bool_t
+AliFMDReconstructor::PreReconstruct() const
+{
+  AliFMDDebug(1, ("Before reoconstructing"));
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return false;
+  }
+
+  // Get our vertex
+  GetVertex(fESD);
+
+  // Reset bad flags 
+  fBad.Reset(false);
+
+  // Reset the output ESD 
+  if (fESDObj) {
+    fESDObj->Clear();
+    
+    // Pre-set eta values 
+    for (UShort_t d=1; d<=3; d++) { 
+      UShort_t nQ = (d == 1 ? 1 : 2);
+      for (UShort_t q=0; q<nQ; q++) { 
+       UShort_t nStr = (q == 0 ? 512 : 256);
+       Char_t   r    = (q == 0 ? 'I' : 'O');
+       
+       for (UShort_t t = 0; t < nStr; t++) { 
+         Float_t eta, phi;
+         // Always use sector 0
+         PhysicalCoordinates(d, r, 0, t, eta, phi);
+         fESDObj->SetEta(d, r, 0, t, eta);
+       }
+      }
+    }
+  }
+
+
+  return true;
+}
 
 //____________________________________________________________________
 void 
-AliFMDReconstructor::Reconstruct(AliRawReader* reader, TTree*) const
+AliFMDReconstructor::Reconstruct(AliFMDRawReader& rawReader) const
 {
   // Reconstruct directly from raw data (no intermediate output on
   // digit tree or rec point tree).  
   // 
   // Parameters: 
-  //   reader  Raw event reader 
-  //   ctree    Not used. 
-  AliFMDDebug(1, ("Reconstructing from raw reader"));
-  AliFMDRawReader rawReader(reader, 0);
+  //   rawReader       FMD Raw event reader 
+  AliFMDDebug(1, ("Reconstructing from FMD raw reader"));
+  if (!PreReconstruct()) return;
 
   UShort_t det, sec, str, fac;
   Short_t  adc, oldDet = -1;
@@ -269,7 +413,26 @@ AliFMDReconstructor::Reconstruct(AliRawReader* reader, TTree*) const
     ProcessSignal(det, rng, sec, str, adc);
   }
   UseRecoParam(kFALSE);
-  
+
+}
+
+//____________________________________________________________________
+void 
+AliFMDReconstructor::Reconstruct(AliRawReader* reader, TTree*) const
+{
+  // Reconstruct directly from raw data (no intermediate output on
+  // digit tree or rec point tree).  
+  // 
+  // Parameters: 
+  //   reader  Raw event reader 
+  //   ctree    Not used - 'cluster tree' to store rec-points in. 
+  AliFMDDebug(1, ("Reconstructing from raw reader"));
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
+  AliFMDRawReader rawReader(reader, 0);
+  Reconstruct(rawReader);
 }
 
 //____________________________________________________________________
@@ -282,6 +445,10 @@ AliFMDReconstructor::Digitize(AliRawReader* reader, TClonesArray* sdigits) const
   // Parameters: 
   //   reader  Raw event reader 
   //   ctree    Not used. 
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
   AliFMDRawReader rawReader(reader, 0);
 
   UShort_t det, sec, str, sam, rat, fac;
@@ -316,40 +483,70 @@ AliFMDReconstructor::Reconstruct(TTree* digitsTree,
   //   digitsTree      Pointer to a tree containing digits 
   //   clusterTree     Pointer to output tree 
   // 
+  if (!PreReconstruct()) return;
+
+  if (!fMult) fMult = new TClonesArray("AliFMDRecPoint");
+
   AliFMDDebug(1, ("Reconstructing from digits in a tree"));
-  GetVertex(fESD);
 
-  static TClonesArray* digits = new TClonesArray("AliFMDDigit");
-  TBranch *digitBranch = digitsTree->GetBranch("FMD");
+  // Get the digitis array 
+  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();
-  
+
+  // Create rec-point output branch 
   fNMult = 0;
   fTreeR = clusterTree;
   fTreeR->Branch("FMD", &fMult);
   
-  AliFMDDebug(5, ("Getting entry 0 from digit branch"));
+  AliDebug(5, "Getting entry 0 from digit branch");
   digitBranch->GetEntry(0);
   
-  AliFMDDebug(5, ("Processing digits"));
+  AliDebugF(5, "Processing %d digits", digits->GetEntriesFast());
   UseRecoParam(kTRUE);
   ProcessDigits(digits);
   UseRecoParam(kFALSE);
 
   Int_t written = clusterTree->Fill();
   AliFMDDebug(10, ("Filled %d bytes into cluster tree", written));
-  digits->Delete();
+  // digits->Delete();
   // delete digits;
 }
  
 
+//____________________________________________________________________
+void
+AliFMDReconstructor::ProcessDigits(TClonesArray* digits, 
+                                  const AliFMDRawReader& rawRead) const
+{
+  // For each digit, find the pseudo rapdity, azimuthal angle, and
+  // number of corrected ADC counts, and pass it on to the algorithms
+  // used. 
+  // 
+  // Parameters: 
+  //    digits Array of digits
+  // 
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
+  AliFMDAltroMapping* map = AliFMDParameters::Instance()->GetAltroMap();
+  for (size_t i = 1; i <= 3; i++) { 
+    fZS[i-1]       = rawRead.IsZeroSuppressed(map->Detector2DDL(i));
+    fZSFactor[i-1] = rawRead.NoiseFactor(map->Detector2DDL(i));
+  }
+  UseRecoParam(kTRUE);
+  ProcessDigits(digits);
+  UseRecoParam(kFALSE);
+}
+
 //____________________________________________________________________
 void
 AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
@@ -365,6 +562,7 @@ AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
   AliFMDDebug(2, ("Got %d digits", nDigits));
   fESDObj->SetNoiseFactor(fNoiseFactor);
   fESDObj->SetAngleCorrected(fAngleCorrect);
+  fBad.Reset(false);
   for (Int_t i = 0; i < nDigits; i++) {
     AliFMDDigit* digit = static_cast<AliFMDDigit*>(digits->At(i));
     if (!digit) continue;
@@ -376,12 +574,18 @@ AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
 void
 AliFMDReconstructor::ProcessDigit(AliFMDDigit* digit) const
 {
+  // 
+  // Process a single digit 
+  // 
+  // Parameters:
+  //    digit Digiti to process
+  // 
   UShort_t det = digit->Detector();
   Char_t   rng = digit->Ring();
   UShort_t sec = digit->Sector();
   UShort_t str = digit->Strip();
   Short_t  adc = digit->Counts();
+  // if (AliLog::GetDebugLevel("FMD","")>3) digit->Print();
   ProcessSignal(det, rng, sec, str, adc);
 }
 
@@ -402,26 +606,46 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
   //    rng    Strip ID
   //    adc     ADC counts
   // 
-  AliFMDParameters* param  = AliFMDParameters::Instance();
+  Int_t  dbg = AliLog::GetDebugLevel("FMD","");
+  Bool_t dhr = dbg > 3 && dbg < 10;
+  if (dhr) printf("FMD%d%c[%2d,%3d] adc=%4d ", det, rng, sec, str, adc);
+  
+  if (adc >= AliFMDRawReader::kBadSignal) { 
+    AliFMDDebug(10, ("FMD%d%c[%2d,%3d] is marked bad", det, rng, sec, str));
+    if (dhr) Printf("bad");
+    fBad(det,rng,sec,str) = true;
+    return;
+  }
+
   // Check that the strip is not marked as dead 
+  AliFMDParameters* param  = AliFMDParameters::Instance();
   if (param->IsDead(det, rng, sec, str)) {
-    AliFMDDebug(1, ("FMD%d%c[%2d,%3d] is dead", det, rng, sec, str));
+    AliFMDDebug(10, ("FMD%d%c[%2d,%3d] is dead", det, rng, sec, str));
+    if (dhr) Printf("dead");
+    fBad(det,rng,sec,str) = true;
     return;
   }
   
   // digit->Print();
   // Get eta and phi 
-  Float_t eta, phi;
-  PhysicalCoordinates(det, rng, sec, str, eta, phi);
+  Float_t eta = fESDObj->Eta(det, rng, 0, str);
     
   // Substract pedestal. 
   UShort_t counts   = SubtractPedestal(det, rng, sec, str, adc);
-  if(counts == USHRT_MAX) return;
+  if(counts == USHRT_MAX)  { 
+    if (dhr) Printf("invalid");
+    return;
+  }
+  if (dhr) printf("counts=%4d ", counts);
   
     // Gain match digits. 
   Double_t edep     = Adc2Energy(det, rng, sec, str, eta, counts);
   // Get rid of nonsense energy
-  if(edep < 0)  return;
+  if(edep < 0)  { 
+    if (dhr) Printf("zero");
+    return;
+  }
+  if (dhr) printf("edep=%f ", edep);
   
   // Make rough multiplicity 
   Double_t mult     = Energy2Multiplicity(det, rng, sec, str, edep);
@@ -431,13 +655,21 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
   //               "(ADC: %d, Energy: %f)", det, rng, sec, str, mult, 
   //               counts, edep));
   // }
-  if (mult < 0)  return; 
+  if (mult < 0)  {
+    if (dhr) Printf("not hit");
+    return; 
+  }
+
+  if (dhr) Printf("mult=%f ", mult);
   AliFMDDebug(10, ("FMD%d%c[%2d,%3d]: "
                    "ADC: %d, Counts: %d, Energy: %f, Mult: %f",
                  det, rng, sec, str, adc, counts, edep, mult));
   
   // Create a `RecPoint' on the output branch. 
   if (fMult) {
+    Float_t phi;
+    PhysicalCoordinates(det, rng, sec, str, eta, phi);
+
     AliFMDRecPoint* m = 
       new ((*fMult)[fNMult]) AliFMDRecPoint(det, rng, sec, str, 
                                            eta, phi, edep, mult);
@@ -446,7 +678,7 @@ AliFMDReconstructor::ProcessSignal(UShort_t det,
   }
   
   fESDObj->SetMultiplicity(det, rng, sec, str, mult);
-  fESDObj->SetEta(det, rng, sec, str, eta);
+  // fESDObj->SetEta(det, rng, sec, str, eta);
   
   if (fDiagAll) fDiagAll->Fill(adc, mult);  
 
@@ -505,17 +737,36 @@ AliFMDReconstructor::SubtractPedestal(UShort_t det,
                                      Bool_t   zsEnabled, 
                                      UShort_t zsNoiseFactor) const
 {
+  // 
+  // Subtract the pedestal off the ADC counts. 
+  // 
+  // Parameters:
+  //    det           Detector number
+  //    rng           Ring identifier
+  //    sec           Sector number
+  //    str           Strip number
+  //    adc           ADC counts
+  //    noiseFactor   If pedestal substracted pedestal is less then
+  //        this times the noise, then consider this to be 0. 
+  //    zsEnabled     Whether zero-suppression is on.
+  //    zsNoiseFactor Noise factor used in on-line pedestal
+  //        subtraction. 
+  // 
+  // Return:
+  //    The pedestal subtracted ADC counts (possibly 0), or @c
+  //         USHRT_MAX in case of problems.
+  //  
   AliFMDParameters* param  = AliFMDParameters::Instance();
   Float_t           ped    = (zsEnabled ? 0 : 
-                               param->GetPedestal(det, rng, sec, str));
+                             param->GetPedestal(det, rng, sec, str));
   Float_t           noise  = param->GetPedestalWidth(det, rng, sec, str);
-  if(ped < 0 || noise < 0) { 
+  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 "
+  AliDebugClass(10, 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, 
@@ -539,8 +790,8 @@ AliFMDReconstructor::SubtractPedestal(UShort_t det,
   //
   // 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;
+  Float_t nf = zsNoiseFactor; // 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;
@@ -623,12 +874,12 @@ AliFMDReconstructor::Adc2Energy(UShort_t det,
   Float_t           gain  = param->GetPulseGain(det, rng, sec, str);
   // 'Tagging' bad gains as bad energy
   if (gain < 0) { 
-    AliWarning(Form("Invalid gain (%f) for FMD%d%c[%02d,%03d]", 
-                   gain, det, rng, sec, str));
+    AliDebugF(10, "Invalid gain (%f) for FMD%d%c[%02d,%03d]", 
+               gain, det, rng, sec, str);
     return -1;
   }
-  AliFMDDebug(5, ("Converting counts %d to energy (factor=%f, DAC2MIP=%f)", 
-                 count, gain,param->GetDACPerMIP()));
+  AliDebugF(10, "Converting counts %d to energy (factor=%f, DAC2MIP=%f)", 
+           count, gain,param->GetDACPerMIP());
 
   Double_t edep  = ((count * param->GetEdepMip()) 
                    / (gain * param->GetDACPerMIP()));
@@ -687,8 +938,8 @@ AliFMDReconstructor::Adc2Energy(UShort_t det,
     Double_t theta = 2 * TMath::ATan(TMath::Exp(-eta));
     Double_t corr  = TMath::Abs(TMath::Cos(theta));
     Double_t cedep = corr * edep;
-    AliFMDDebug(10, ("correcting for path %f * %f = %f (eta=%f, theta=%f)",
-                     edep, corr, cedep, eta, theta));
+    AliDebugF(10, "correcting for path %f * %f = %f (eta=%f, theta=%f)",
+             edep, corr, cedep, eta, theta);
     if (fDiagStep3) fDiagStep3->Fill(edep, cedep);  
     edep = cedep;
   }
@@ -758,17 +1009,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));
+  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 
@@ -780,8 +1045,15 @@ AliFMDReconstructor::FillESD(TTree*  /* digitsTree */,
   // FIXME: The vertex may not be known when Reconstruct is executed,
   // so we may have to move some of that member function here. 
   AliFMDDebug(2, ("Calling FillESD with two trees and one ESD"));
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
   // fESDObj->Print();
 
+  // Fix up ESD so that only truely dead channels get the kInvalidMult flag. 
+  MarkDeadChannels(fESDObj);
+
   Double_t oldVz = fCurrentVertex;
   GetVertex(esd);
   if (fVertexType != kNoVertex) { 
@@ -790,6 +1062,11 @@ AliFMDReconstructor::FillESD(TTree*  /* digitsTree */,
     AliFMDESDRevertexer revertexer;
     revertexer.Revertex(fESDObj, fCurrentVertex);
   }
+  
+  if (AliDebugLevel() > 10) { 
+    ESDPrinter p;
+    fESDObj->ForEach(p);
+  }
 
   if (esd) { 
     AliFMDDebug(2, ("Writing FMD data to ESD tree"));
@@ -831,6 +1108,13 @@ void
 AliFMDReconstructor::FillESD(AliRawReader*, TTree* clusterTree, 
                             AliESDEvent* esd) const
 {
+  // 
+  // Forwards to above member function 
+  //
+  if (fZombie) { 
+    AliWarning("I'm a zombie - cannot do anything");
+    return;
+  }
   TTree* dummy = 0;
   FillESD(dummy, clusterTree, esd);
 }