]> git.uio.no Git - u/mrichter/AliRoot.git/blobdiff - FMD/AliFMDReconstructor.cxx
Load pythia libraries.
[u/mrichter/AliRoot.git] / FMD / AliFMDReconstructor.cxx
index 935a2f4b40f8ab41bf7d662dc0f0fbc1493915de..bb1717fa1d9921e00cd7d247b2c735cc9a10b563 100644 (file)
  * about the suitability of this software for any purpose. It is          *
  * 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 
+*/
 //____________________________________________________________________
 //
-// This is a class that constructs AliFMDMult (reconstructed
-// multiplicity) from of Digits
-//
-// This class reads either digits from a TClonesArray or raw data from
+// 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). 
-//
-// From the cached values it then calculates the number of particles
-// that hit a region of the FMDs, as specified by the user. 
-//
-// The reconstruction can be done in two ways: Either via counting the
-// number of empty strips (Poisson method), or by converting the ADC
-// signal to an energy deposition, and then dividing by the typical
-// energy loss of a particle.
-// 
-//      +---------------------+       +---------------------+
-//      | AliFMDReconstructor |<>-----| AliFMDMultAlgorithm |
-//      +---------------------+       +---------------------+
-//                                               ^
-//                                               |
-//                                   +-----------+---------+
-//                                   |                     |
-//                         +-------------------+   +------------------+
-//                         | AliFMDMultPoisson |   | AliFMDMultNaiive |
-//                         +-------------------+   +------------------+
-//
-// AliFMDReconstructor acts as a manager class.  It contains a list of
-// AliFMDMultAlgorithm objects.  The call graph looks something like 
-//
+// internal cache (fAdcs).   The rec-points are made via the naiive
+// method. 
 //
-//       +----------------------+            +----------------------+
-//       | :AliFMDReconstructor |            | :AliFMDMultAlgorithm |
-//       +----------------------+            +----------------------+
-//                  |                                  |
-//    Reconstruct  +-+                                 |
-//    ------------>| |                         PreRun +-+
-//                 | |------------------------------->| |   
-//                 | |                                +-+
-//                 | |-----+ (for each event)          |
-//                 | |     | *ProcessEvent             |
-//                 |+-+    |                           |
-//                 || |<---+                 PreEvent +-+
-//                 || |------------------------------>| |      
-//                 || |                               +-+
-//                 || |-----+                          |
-//                 || |     | ProcessDigits            |
-//                 ||+-+    |                          |
-//                 ||| |<---+                          |
-//                 ||| |         *ProcessDigit(digit) +-+
-//                 ||| |----------------------------->| |
-//                 ||| |                              +-+
-//                 ||+-+                               |
-//                 || |                     PostEvent +-+
-//                 || |------------------------------>| |
-//                 || |                               +-+
-//                 |+-+                                |
-//                 | |                        PostRun +-+
-//                 | |------------------------------->| |
-//                 | |                                +-+
-//                 +-+                                 |
-//                  |                                  |
-//
-//
-// 
 //-- 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
-#include <AliRunLoader.h>                  // ALIRUNLOADER_H
-#include <AliLoader.h>                     // ALILOADER_H
-#include <AliHeader.h>                     // ALIHEADER_H
-#include <AliRawReader.h>                  // ALIRAWREADER_H
-#include <AliGenEventHeader.h>             // ALIGENEVENTHEADER_H
-#include "AliFMD.h"                       // ALIFMD_H
+// #include <AliLog.h>                        // ALILOG_H
+// #include <AliRun.h>                        // ALIRUN_H
+#include "AliFMDDebug.h"
 #include "AliFMDGeometry.h"                // ALIFMDGEOMETRY_H
 #include "AliFMDParameters.h"              // ALIFMDPARAMETERS_H
-#include "AliFMDDetector.h"                // ALIFMDDETECTOR_H
-#include "AliFMDRing.h"                    // ALIFMDRING_H
 #include "AliFMDDigit.h"                   // ALIFMDDIGIT_H
 #include "AliFMDReconstructor.h"           // ALIFMDRECONSTRUCTOR_H
-#include "AliFMDRawStream.h"               // ALIFMDRAWSTREAM_H
 #include "AliFMDRawReader.h"               // ALIFMDRAWREADER_H
-#include "AliFMDMultStrip.h"              // ALIFMDMULTNAIIVE_H
-#include "AliESD.h"                       // ALIESD_H
+#include "AliFMDRecPoint.h"               // ALIFMDMULTNAIIVE_H
+#include "AliESDEvent.h"                  // ALIESDEVENT_H
+#include "AliESDVertex.h"                 // ALIESDVERTEX_H
 #include <AliESDFMD.h>                    // ALIESDFMD_H
+#include <TMath.h>
+#include <TH1.h>
+#include <TH2.h>
 #include <TFile.h>
+class AliRawReader;
 
 //____________________________________________________________________
 ClassImp(AliFMDReconstructor)
@@ -117,10 +59,26 @@ ClassImp(AliFMDReconstructor)
 //____________________________________________________________________
 AliFMDReconstructor::AliFMDReconstructor() 
   : AliReconstructor(),
-    fMult(0), 
-    fESDObj(0)
+    fMult(0x0), 
+    fNMult(0),
+    fTreeR(0x0),
+    fCurrentVertex(0),
+    fESDObj(0x0),
+    fNoiseFactor(0),
+    fAngleCorrect(kTRUE),
+    fVertexType(kNoVertex),
+    fESD(0x0),
+    fDiagnostics(kFALSE),
+    fDiagStep1(0), 
+    fDiagStep2(0),
+    fDiagStep3(0),
+    fDiagStep4(0),
+    fDiagAll(0)
 {
   // Make a new FMD reconstructor object - default CTOR.  
+  SetNoiseFactor();
+  SetAngleCorrect();
+  if (AliDebugLevel() > 0) fDiagnostics = kTRUE;
 }
   
 
@@ -128,7 +86,20 @@ AliFMDReconstructor::AliFMDReconstructor()
 AliFMDReconstructor::AliFMDReconstructor(const AliFMDReconstructor& other) 
   : AliReconstructor(), 
     fMult(other.fMult),
-    fESDObj(other.fESDObj)
+    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 
 }
@@ -139,8 +110,21 @@ AliFMDReconstructor&
 AliFMDReconstructor::operator=(const AliFMDReconstructor& other) 
 {
   // Assignment operator
-  fMult   = other.fMult;
-  fESDObj = other.fESDObj;
+  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;
 }
 
@@ -155,46 +139,55 @@ AliFMDReconstructor::~AliFMDReconstructor()
 
 //____________________________________________________________________
 void 
-AliFMDReconstructor::Init(AliRunLoader* runLoader
+AliFMDReconstructor::Init() 
 {
   // Initialize the reconstructor 
-  AliDebug(1, Form("Init called with runloader 0x%x", runLoader));
+
+  // Initialize the geometry 
+  AliFMDGeometry* geom = AliFMDGeometry::Instance();
+  geom->Init();
+  geom->InitTransformations();
+
+  // Initialize the parameters
+  AliFMDParameters* param = AliFMDParameters::Instance();
+  param->Init();
+  
   // Current vertex position
   fCurrentVertex = 0;
   // Create array of reconstructed strip multiplicities 
-  fMult = new TClonesArray("AliFMDMultStrip", 51200);
+  fMult = new TClonesArray("AliFMDRecPoint", 51200);
   // Create ESD output object 
   fESDObj = new AliESDFMD;
   
-  // Check that we have a run loader
-  if (!runLoader) { 
-    Warning("Init", "No run loader");
-    return;
-  }
-
-  // Check if we can get the header tree 
-  AliHeader* header = runLoader->GetHeader();
-  if (!header) {
-    Warning("Init", "No header");
-    return;
-  }
-
-  // Check if we can get a simulation header 
-  AliGenEventHeader* eventHeader = header->GenEventHeader();
-  if (eventHeader) {
-    TArrayF vtx;
-    eventHeader->PrimaryVertex(vtx);
-    fCurrentVertex = vtx[2];
-    AliDebug(1, Form("Primary vertex Z coordinate for event # %d/%d is %f", 
-                    header->GetRun(), header->GetEvent(), fCurrentVertex));
-    Warning("Init", "no generator event header");
-  }
-  else {
-    Warning("Init", "No generator event header - "
-           "perhaps we get the vertex from ESD?");
-  }
-  // Get the ESD tree 
-  SetESD(new AliESD);
+  // 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");
 }
 
 //____________________________________________________________________
@@ -203,7 +196,7 @@ AliFMDReconstructor::ConvertDigits(AliRawReader* reader,
                                   TTree* digitsTree) const
 {
   // Convert Raw digits to AliFMDDigit's in a tree 
-  AliDebug(1, "Reading raw data into digits tree");
+  AliFMDDebug(2, ("Reading raw data into digits tree"));
   AliFMDRawReader rawRead(reader, digitsTree);
   // rawRead.SetSampleRate(fFMD->GetSampleRate());
   rawRead.Exec();
@@ -211,23 +204,58 @@ AliFMDReconstructor::ConvertDigits(AliRawReader* reader,
 
 //____________________________________________________________________
 void 
-AliFMDReconstructor::Reconstruct(TTree* digitsTree, 
-                                TTree* clusterTree) const 
+AliFMDReconstructor::GetVertex() const
 {
-  // Reconstruct event from digits in tree 
-  // Get the FMD branch holding the digits. 
-  // FIXME: The vertex may not be known yet, so we may have to move
-  // some of this to FillESD. 
-  AliDebug(1, "Reconstructing from digits in a tree");
-#if 0
+  // 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->GetVertex();
     if (vertex) {
-      AliDebug(1, Form("Got vertex from ESD: %f", vertex->GetZv()));
+      AliFMDDebug(2, ("Got vertex from ESD: %f", vertex->GetZv()));
       fCurrentVertex = vertex->GetZv();
+      fVertexType    = kESDVertex;
+      return;
     }
   }
-#endif  
+  AliWarning("Didn't get any vertex from ESD or generator");
+}
+  
+
+//____________________________________________________________________
+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. 
+  AliError("Method is not used");
+#if 0
+  TClonesArray*   array = new TClonesArray("AliFMDDigit");
+  AliFMDRawReader rawRead(reader, 0);
+  rawRead.ReadAdcs(array);
+  ProcessDigits(array);
+  array->Delete();
+  delete array;
+#endif
+}
+
+//____________________________________________________________________
+void 
+AliFMDReconstructor::Reconstruct(TTree* digitsTree, 
+                                TTree* clusterTree) const 
+{
+  // Reconstruct event from digits in tree 
+  // Get the FMD branch holding the digits. 
+  // FIXME: The vertex may not be known yet, so we may have to move
+  // some of this to FillESD. 
+  AliFMDDebug(2, ("Reconstructing from digits in a tree"));
+  GetVertex();
+
   TBranch *digitBranch = digitsTree->GetBranch("FMD");
   if (!digitBranch) {
     Error("Exec", "No digit branch for the FMD found");
@@ -236,13 +264,6 @@ AliFMDReconstructor::Reconstruct(TTree* digitsTree,
   TClonesArray* digits = new TClonesArray("AliFMDDigit");
   digitBranch->SetAddress(&digits);
 
-  // TIter next(&fAlgorithms);
-  // AliFMDMultAlgorithm* algorithm = 0;
-  // while ((algorithm = static_cast<AliFMDMultAlgorithm*>(next()))) {
-  //   AliDebug(10, Form("PreEvent called for algorithm %s", 
-  //                     algorithm->GetName()));    
-  //   algorithm->PreEvent(clusterTree, fCurrentVertex);
-  // }
   if (fMult)   fMult->Clear();
   if (fESDObj) fESDObj->Clear();
   
@@ -250,21 +271,14 @@ AliFMDReconstructor::Reconstruct(TTree* digitsTree,
   fTreeR = clusterTree;
   fTreeR->Branch("FMD", &fMult);
   
-  AliDebug(5, "Getting entry 0 from digit branch");
+  AliFMDDebug(5, ("Getting entry 0 from digit branch"));
   digitBranch->GetEntry(0);
   
-  AliDebug(5, "Processing digits");
+  AliFMDDebug(5, ("Processing digits"));
   ProcessDigits(digits);
 
-  // next.Reset();
-  // algorithm = 0;
-  // while ((algorithm = static_cast<AliFMDMultAlgorithm*>(next()))) {
-  //   AliDebug(10, Form("PostEvent called for algorithm %s", 
-  //                     algorithm->GetName()));
-  // algorithm->PostEvent();
-  // }
   Int_t written = clusterTree->Fill();
-  AliDebug(10, Form("Filled %d bytes into cluster tree", written));
+  AliFMDDebug(10, ("Filled %d bytes into cluster tree", written));
   digits->Delete();
   delete digits;
 }
@@ -278,13 +292,19 @@ AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
   // number of corrected ADC counts, and pass it on to the algorithms
   // used. 
   Int_t nDigits = digits->GetEntries();
-  AliDebug(1, Form("Got %d digits", nDigits));
+  AliFMDDebug(1, ("Got %d digits", nDigits));
+  fESDObj->SetNoiseFactor(fNoiseFactor);
+  fESDObj->SetAngleCorrected(fAngleCorrect);
   for (Int_t i = 0; i < nDigits; i++) {
     AliFMDDigit* digit = static_cast<AliFMDDigit*>(digits->At(i));
     AliFMDParameters* param  = AliFMDParameters::Instance();
     // Check that the strip is not marked as dead 
     if (param->IsDead(digit->Detector(), digit->Ring(), 
-                     digit->Sector(), digit->Strip())) continue;
+                     digit->Sector(), digit->Strip())) {
+      AliFMDDebug(10, ("FMD%d%c[%2d,%3d] is dead", digit->Detector(), 
+                       digit->Ring(), digit->Sector(), digit->Strip()));
+      continue;
+    }
 
     // digit->Print();
     // Get eta and phi 
@@ -300,27 +320,30 @@ AliFMDReconstructor::ProcessDigits(TClonesArray* digits) const
     // Make rough multiplicity 
     Double_t mult     = Energy2Multiplicity(digit, edep);
     
-    AliDebug(10, Form("FMD%d%c[%2d,%3d]: "
+    AliFMDDebug(10, ("FMD%d%c[%2d,%3d]: "
                      "ADC: %d, Counts: %d, Energy: %f, Mult: %f", 
                      digit->Detector(), digit->Ring(), digit->Sector(),
                      digit->Strip(), digit->Counts(), counts, edep, mult));
     
     // Create a `RecPoint' on the output branch. 
-    AliFMDMultStrip* m = 
-      new ((*fMult)[fNMult]) AliFMDMultStrip(digit->Detector(), 
-                                            digit->Ring(), 
-                                            digit->Sector(),
-                                            digit->Strip(),
-                                            eta, phi, 
-                                            edep, mult,
-                                            AliFMDMult::kNaiive);
-    (void)m; // Suppress warnings about unused variables. 
-    fNMult++;
-
+    if (fMult) {
+      AliFMDRecPoint* m = 
+       new ((*fMult)[fNMult]) AliFMDRecPoint(digit->Detector(), 
+                                             digit->Ring(), 
+                                             digit->Sector(),
+                                             digit->Strip(),
+                                             eta, phi, 
+                                             edep, mult);
+      (void)m; // Suppress warnings about unused variables. 
+      fNMult++;
+    }
+    
     fESDObj->SetMultiplicity(digit->Detector(), digit->Ring(), 
                             digit->Sector(),  digit->Strip(), mult);
     fESDObj->SetEta(digit->Detector(), digit->Ring(), 
                    digit->Sector(),  digit->Strip(), eta);
+
+    if (fDiagAll) fDiagAll->Fill(digit->Counts(), mult);  
   }
 }
 
@@ -333,19 +356,25 @@ AliFMDReconstructor::SubtractPedestal(AliFMDDigit* digit) const
   // load this to subtract a pedestal that was given in a database or
   // something like that. 
 
-  Int_t             counts = 0;
   AliFMDParameters* param  = AliFMDParameters::Instance();
-  Float_t           pedM   = param->GetPedestal(digit->Detector(), 
+  Float_t           ped    = param->GetPedestal(digit->Detector(), 
                                                digit->Ring(), 
                                                digit->Sector(), 
                                                digit->Strip());
-  AliDebug(10, Form("Subtracting pedestal %f from signal %d", 
-                  pedM, digit->Counts()));
-  if (digit->Count3() > 0)      counts = digit->Count3();
-  else if (digit->Count2() > 0) counts = digit->Count2();
-  else                          counts = digit->Count1();
-  counts = TMath::Max(Int_t(counts - pedM), 0);
-  if (counts > 0) AliDebug(10, "Got a hit strip");
+  Float_t           noise  = param->GetPedestalWidth(digit->Detector(), 
+                                                    digit->Ring(), 
+                                                    digit->Sector(), 
+                                                    digit->Strip());
+  AliFMDDebug(15, ("Subtracting pedestal %f from signal %d", 
+                  ped, digit->Counts()));
+  // if (digit->Count3() > 0)      adc = digit->Count3();
+  // else if (digit->Count2() > 0) adc = digit->Count2();
+  // else                          adc = digit->Count1();
+  Int_t adc    = digit->Counts();
+  Int_t counts = TMath::Max(Int_t(adc - ped), 0);
+  if (counts < noise * fNoiseFactor) counts = 0;
+  if (counts > 0) AliFMDDebug(15, ("Got a hit strip"));
+  if (fDiagStep1) fDiagStep1->Fill(adc, counts);
   
   return  UShort_t(counts);
 }
@@ -353,7 +382,7 @@ AliFMDReconstructor::SubtractPedestal(AliFMDDigit* digit) const
 //____________________________________________________________________
 Float_t
 AliFMDReconstructor::Adc2Energy(AliFMDDigit* digit, 
-                               Float_t      /* eta */
+                               Float_t      eta
                                UShort_t     count) const
 {
   // Converts number of ADC counts to energy deposited. 
@@ -372,17 +401,26 @@ AliFMDReconstructor::Adc2Energy(AliFMDDigit* digit,
   //           ADC_channel_size    
   // 
   // is constant and the same for all strips.
-
-  // Double_t theta = 2 * TMath::ATan(TMath::Exp(-eta));
-  // Double_t edep  = TMath::Abs(TMath::Cos(theta)) * fGain * count;
+  if (count <= 0) return 0;
   AliFMDParameters* param = AliFMDParameters::Instance();
   Float_t           gain  = param->GetPulseGain(digit->Detector(), 
                                                digit->Ring(), 
                                                digit->Sector(), 
                                                digit->Strip());
-  Double_t          edep  = count * gain;
-  AliDebug(10, Form("Converting counts %d to energy via factor %f", 
+  AliFMDDebug(15, ("Converting counts %d to energy via factor %f", 
                    count, gain));
+
+  Double_t edep  = count * gain;
+  if (fDiagStep2) fDiagStep2->Fill(count, edep);  
+  if (fAngleCorrect) {
+    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));
+    if (fDiagStep3) fDiagStep3->Fill(edep, cedep);  
+    edep = cedep;
+  }
   return edep;
 }
 
@@ -409,8 +447,9 @@ AliFMDReconstructor::Energy2Multiplicity(AliFMDDigit* /* digit */,
   Double_t          edepMIP = param->GetEdepMip();
   Float_t           mult    = edep / edepMIP;
   if (edep > 0) 
-    AliDebug(10, Form("Translating energy %f to multiplicity via "
+    AliFMDDebug(15, ("Translating energy %f to multiplicity via "
                     "divider %f->%f", edep, edepMIP, mult));
+  if (fDiagStep4) fDiagStep4->Fill(edep, mult);  
   return mult;
 }
 
@@ -423,32 +462,16 @@ AliFMDReconstructor::PhysicalCoordinates(AliFMDDigit* digit,
   // Get the eta and phi of a digit 
   // 
   // Get geometry. 
-  AliFMDGeometry* fmd = AliFMDGeometry::Instance();
-  AliFMDDetector* subDetector = fmd->GetDetector(digit->Detector());
-  if (!subDetector) { 
-    Warning("ProcessDigits", "Unknown detector: FMD%d" , digit->Detector());
-    return;
-  }
-    
-  // Get the ring - we should really use geometry->Detector2XYZ(...)
-  // here. 
-  AliFMDRing* ring  = subDetector->GetRing(digit->Ring());
-  Float_t     ringZ = subDetector->GetRingZ(digit->Ring());
-  if (!ring) {
-    Warning("ProcessDigits", "Unknown ring: FMD%d%c", digit->Detector(), 
-           digit->Ring());
-    return;
-  }
-    
+  AliFMDGeometry* geom = AliFMDGeometry::Instance();
+  Double_t x, y, z, r, theta;
+  geom->Detector2XYZ(digit->Detector(), digit->Ring(), digit->Sector(), 
+                   digit->Strip(), x, y, z);
   // Correct for vertex offset. 
-  Float_t  realZ    = fCurrentVertex + ringZ;
-  Float_t  stripR   = ((ring->GetHighR() - ring->GetLowR()) 
-                      / ring->GetNStrips() * (digit->Strip() + .5) 
-                      + ring->GetLowR());
-  Float_t  theta    = TMath::ATan2(stripR, realZ);
-  phi               = (2 * TMath::Pi() / ring->GetNSectors() 
-                      * (digit->Sector() + .5));
-  eta               = -TMath::Log(TMath::Tan(theta / 2));
+  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));
 }
 
       
@@ -457,146 +480,56 @@ AliFMDReconstructor::PhysicalCoordinates(AliFMDDigit* digit,
 void 
 AliFMDReconstructor::FillESD(TTree*  /* digitsTree */, 
                             TTree*  /* clusterTree */,
-                            AliESD* esd) const
+                            AliESDEvent* esd) const
 {
   // nothing to be done
   // FIXME: The vertex may not be known when Reconstruct is executed,
   // so we may have to move some of that member function here. 
-  AliDebug(1, Form("Calling FillESD with two trees and one ESD"));
+  AliFMDDebug(2, ("Calling FillESD with two trees and one ESD"));
   // fESDObj->Print();
 
   if (esd) { 
-    AliDebug(1, Form("Writing FMD data to ESD tree"));
+    AliFMDDebug(2, ("Writing FMD data to ESD tree"));
     esd->SetFMDData(fESDObj);
-    // Let's check the data in the ESD
-#if 0
-    AliESDFMD* fromEsd = esd->GetFMDData();
-    if (!fromEsd) {
-      AliWarning("No FMD object in ESD!");
-      return;
-    }
-    for (UShort_t det = 1; det <= fESDObj->MaxDetectors(); det++) {
-      for (UShort_t ir = 0; ir < fESDObj->MaxRings(); ir++) {
-       Char_t ring = (ir == 0 ? 'I' : 'O');
-       for (UShort_t sec = 0; sec < fESDObj->MaxSectors(); sec++) {
-         for (UShort_t str = 0; str < fESDObj->MaxStrips(); str++) {
-           if (fESDObj->Multiplicity(det, ring, sec, str) != 
-               fromEsd->Multiplicity(det, ring, sec, str))
-             AliWarning(Form("Mult for FMD%d%c[%2d,%3d]",det,ring,sec,str));
-           if (fESDObj->Eta(det, ring, sec, str) != 
-               fromEsd->Eta(det, ring, sec, str))
-             AliWarning(Form("Eta for FMD%d%c[%2d,%3d]", det,ring,sec,str));
-           if (fESDObj->Multiplicity(det, ring, sec, str) > 0 && 
-               fESDObj->Multiplicity(det, ring, sec, str) 
-               != AliESDFMD::kInvalidMult) 
-             AliInfo(Form("Mult in FMD%d%c[%2d,%3d] is %f", det,ring,sec,str,
-                          fESDObj->Multiplicity(det, ring, sec, str)));
-         }
-       }
-      }
-    }
-#endif
   }
 
-#if 0  
-  static Int_t evNo = -1;
-  evNo++;
-  if (esd) evNo = esd->GetEventNumber();
-  TString fname(Form("FMD.ESD.%03d.root", evNo));
-  TFile* file = TFile::Open(fname.Data(), "RECREATE");
-  if (!file) {
-    AliError(Form("Failed to open file %s", fname.Data()));
-    return;
-  }
-  fESDObj->Write();
-  file->Close();
-#endif
-    
-#if 0
-  TClonesArray* multStrips  = 0;
-  TClonesArray* multRegions = 0;
-  TTree*        treeR  = fmdLoader->TreeR();
-  TBranch*      branchRegions = treeR->GetBranch("FMDPoisson");
-  TBranch*      branchStrips  = treeR->GetBranch("FMDNaiive");
-  branchRegions->SetAddress(&multRegions);
-  branchStrips->SetAddress(&multStrips);
-  
-  Int_t total = 0;
-  Int_t nEntries  = clusterTree->GetEntries();
-  for (Int_t entry = 0; entry < nEntries; entry++) {
-    AliDebug(5, Form("Entry # %d in cluster tree", entry));
-    treeR->GetEntry(entry);
-    
-    
-    Int_t nMults = multRegions->GetLast();
-    for (Int_t i = 0; i <= nMults; i++) {
-      AliFMDMultRegion* multR =
-       static_cast<AliFMDMultRegion*>(multRegions->UncheckedAt(i));
-      Int_t nParticles=multR->Particles();
-      if (i>=0 && i<=13)   hEtaPoissonI1->AddBinContent(i+1,nParticles);
-      if (i>=14 && i<=27 ) hEtaPoissonI2->AddBinContent(i-13,nParticles);
-      if (i>=28 && i<=33 );
-      if (i>=34 && i<=47 ) hEtaPoissonI3->AddBinContent(48-i,nParticles);
-      if (i>=48 && i<=53)  hEtaPoissonO3->AddBinContent(54-i,nParticles);
-    }
-  }
-#endif   
-}
+  if (!fDiagnostics || !esd) return;
+  static bool first = true;
+  // This is most likely NOT the event number you'd like to use. It
+  // has nothing to do with the 'real' event number. 
+  // - 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));
+  TFile f("FMD.Diag.root", (first ? "RECREATE" : "UPDATE"));
+  first = false;
+  f.cd(); 
+  TDirectory* d = f.mkdir(Form("%03d", evno), 
+                         Form("Diagnostics histograms for event # %d", evno));
+  d->cd();
+  if (fDiagStep1) fDiagStep1->Write();
+  if (fDiagStep2) fDiagStep2->Write();
+  if (fDiagStep3) fDiagStep3->Write();
+  if (fDiagStep4) fDiagStep4->Write();
+  if (fDiagAll)   fDiagAll->Write();
+  d->Write();
+  f.Write();
+  f.Close();
 
-
-//____________________________________________________________________
-void 
-AliFMDReconstructor::Reconstruct(AliRawReader*,TTree*) const 
-{
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with loader and tree"));
-  AliError("MayNotUse");
-}
-//____________________________________________________________________
-void 
-AliFMDReconstructor::Reconstruct(AliRunLoader*) const 
-{
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with loader"));
-  AliError("MayNotUse");
-}
-//____________________________________________________________________
-void 
-AliFMDReconstructor::Reconstruct(AliRunLoader*, AliRawReader*) const 
-{
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with loader and raw reader"));
-  AliError("MayNotUse");
-}
-//____________________________________________________________________
-void 
-AliFMDReconstructor::FillESD(AliRawReader*,TTree*,AliESD*) const 
-{
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with raw reader, tree, and ESD"));
-  AliError("MayNotUse");
-}
-//____________________________________________________________________
-void 
-AliFMDReconstructor::FillESD(AliRunLoader*,AliESD*) const
-{
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with loader and ESD"));
-  AliError("MayNotUse");
+  if (fDiagStep1) fDiagStep1->Reset();
+  if (fDiagStep2) fDiagStep2->Reset();
+  if (fDiagStep3) fDiagStep3->Reset();
+  if (fDiagStep4) fDiagStep4->Reset();
+  if (fDiagAll)   fDiagAll->Reset();
 }
+
 //____________________________________________________________________
-void 
-AliFMDReconstructor::FillESD(AliRunLoader*,AliRawReader*,AliESD*) const 
+void
+AliFMDReconstructor::FillESD(AliRawReader*, TTree* clusterTree, 
+                            AliESDEvent* esd) const
 {
-  // Cannot be used.  See member function with same name but with 2
-  // TTree arguments.   Make sure you do local reconstrucion 
-  AliDebug(1, Form("Calling FillESD with loader, raw reader, and ESD"));
-  AliError("MayNotUse");
+  TTree* dummy = 0;
+  FillESD(dummy, clusterTree, esd);
 }
 
 //____________________________________________________________________