GSI Forum
GSI Helmholtzzentrum für Schwerionenforschung

Home » PANDA » PandaRoot » EMC » Bump splitting: new endcap vs. old endcap
Bump splitting: new endcap vs. old endcap [message #7833] Wed, 04 February 2009 19:21 Go to next message
Bertram Kopf is currently offline  Bertram Kopf
Messages: 110
Registered: March 2006
continuous participant
From: *ep1.ruhr-uni-bochum.de
Dear all,
I tested the bump splitting for a couple of different scenarios. 5 GeV/c pi0's have been simulated separately for the forward endcap (10-20 deg) and for the forward barrel part (30-90 deg) for two different geometries. I used in the simulation macro the initialization
Emc->SetGeometryFileName("emc_module12345.dat");
for the old geometry;
and Emc->SetGeometryFileNameDouble("emc_module1245.dat","emc_module3new.root ");
for the new geometry and changed moreover the mapper version number to 1 (old) and to 2 (new) in all.par and emc.par.

I got the following results with the bump analysis qa macro provided by Dima:

1. Everything looks more or less o.k. for the barrel part (30-90 deg) (black=old, red=new)

index.php?t=getfile&id=4999&private=0

2.In comparison to this the results for the forward part (10-20 deg) look completely different (black=old, red=new) and it seems that there is only something wrong with the new geometry.

index.php?t=getfile&id=5000&private=0

My assumption is that something is not properly initialized in the new EMC geometry setup (e.g. EmcMapper, TwoCoordinateIndex, etc.) and that the bump splitting algorithm works fine.

Due to this I have a few questions / remarks:

1. Is there a tool available with which one can easily check whether the crystal neighbour list, TwoCoordinateIndex, etc. are o.k.?

2. All the geometry information will be handled via the singleton objects "PndEmcMapper" and "PndEmcStructure".
The static function instance has to be called with arguments here: e.g. static PndEmcMapper* Instance(Int_t, TString geoName="");
This suggests that one can get different mappers by steering it via these arguments. But this is not the case: Once initialized, the instance will never be changed afterwards.
In the code the instance of the mapper is sometimes called with fixed arguments: e.g. PndEmcMapper::Instance(0), etc.. Therefore my question: Who takes care of the initialization for such global objects. How is it guaranteed that the instance will be called there the first time in the application?

3. Would it make sense to introduce an EMC initialization sequence where everything will be initialized centrally at one place.

4. Does it make sense to introduce a global Emc environment class where all global emc environment objects are collected?

Best regards,
Bertram.
Re: Bump splitting: new endcap vs. old endcap [message #7857 is a reply to message #7833] Mon, 09 February 2009 13:08 Go to previous messageGo to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *physik.uni-giessen.de
Hi,
only today I can answer to this post.

Quote:

I got the following results with the bump analysis qa macro provided by Dima:


First of all, which qa macro have you used to produce those plots? So that I can also try to reproduce the distributions.

Quote:

My assumption is that something is not properly initialized in the new EMC geometry setup (e.g. EmcMapper, TwoCoordinateIndex, etc.) and that the bump splitting algorithm works fine.


When the new geometry for the endcap was put inside svn, Ola has showed plots with cluster residuals for energy and angular distributions, and they looked fine. For this reason I think that the mapping should be ok, considering that the clusterization depends on the map. Maybe there is something just after the cluster and before the bump, one should investigate.

Quote:

1. Is there a tool available with which one can easily check whether the crystal neighbour list, TwoCoordinateIndex, etc. are o.k.?


The EMC reconstruction code was substantially ported from the fast simulation framework, thus you could use all your tools to check the mapping in pandaroot, the classes are pratically the same.
Which tools have you developed, so that we can also implement them in pandaroot? In general one can use the GetXPad()/GetYPad() functions and correlate them to the x/y/z. But a real "tool" was never developed by us.

Quote:

2. All the geometry information will be handled via the singleton objects...

[cut]

Could you please show an example on how the structure could be messed up? In theory the mapper is needed only for the reconstruction tasks, when looping inside the data after the reconstruction the mapper could only screw the TwoCoordinateIndex objects, but at this stage they are not needed anymore. So maybe I have not well understood which could be the possible problems. And if I remember well, the Instance(0) are called only when no previous mapping is defined and then the code does not know what to use. Maybe Dima can comment on it.

Quote:

3. Would it make sense to introduce an EMC initialization sequence where everything will be initialized centrally at one place.


In theory the function static PndEmcMapper* Instance(Int_t, TString geoName=""); is done with this purpose, to setup the environment only one time. Or am I missing something?

Quote:

4. Does it make sense to introduce a global Emc environment class where all global emc environment objects are collected?


I have still not understood well what do you call "environment objects". In theory there is just the map and all the correlated classes. I know the structure of the code is a bit complicated, but we have prefered to just port the code and to not modify it a lot.

Which are exaclty your suggestions?
Re: Bump splitting: new endcap vs. old endcap [message #7858 is a reply to message #7857] Mon, 09 February 2009 13:34 Go to previous messageGo to next message
Dima Melnychuk is currently offline  Dima Melnychuk
Messages: 213
Registered: April 2004
Location: National Centre for Nucle...
first-grade participant
From: *fuw.edu.pl
Hi Stefano,

Actually the emc classes PndEmcMapper and PndEmcStructure
were not ported from Babar framework, but created from scratch because of different representation of geometry. In Babar framework the geometry was based on XML description and these two classes provide similar functionality to what exists in Babar framework but they are not identical.

The macro, which shows the plots on bump splitting is bump_analysis.C.

Concerning the PndEmcMapper::Instance(0) with parameter 0, it does not create the instance of PndEmcMapper, but only get the instance of the PndEmcMapper if it was created previously. Otherwise it gives the warning.

And at the moment PndEmcMapper is instantiated with proper input parameter in the Init() methods of the tasks where it is used, for example PndEmcMakeCluster, PndEmcMakeBump.

But I support the idea of Bertram for emc initialization sequence, let's say the task class PndEmcInit, which would be inside digitization/reconstruction sequence before any other tasks and where PndEmcMapper would be initialised only once.
But it's matter of discussion.

Dima
Re: Bump splitting: new endcap vs. old endcap [message #7927 is a reply to message #7858] Thu, 19 February 2009 15:43 Go to previous messageGo to next message
Aleksandra Biegun is currently offline  Aleksandra Biegun
Messages: 64
Registered: May 2007
Location: Groningen
continuous participant
From: *KVI.nl
Hi,

../trunk/emc/EmcMC/PndEmc.cxx

and

../trunk/emc/EmcTools/PndEmcStructure.cxx

are posted to svn /pandaroot/trunk/emc/ .

The part corresponding the creation of detectorID
for the forward endcap is changed in both cases.


Best regards,
Ola.


Aleksandra Biegun
University of Groningen/KVI
Zernikelaan 25, 9747 AA Groningen
tel. +31 50 363 3630
fax. +31 50 363 4003
Re: Bump splitting: new endcap vs. old endcap [message #7935 is a reply to message #7833] Fri, 20 February 2009 16:32 Go to previous message
Dima Melnychuk is currently offline  Dima Melnychuk
Messages: 213
Registered: April 2004
Location: National Centre for Nucle...
first-grade participant
From: *fuw.edu.pl
Hi all,

With the new changes, made by Ola in creation of emc forward endcap geometry, the bump splitting gives reasonable results.

For the 500 events 5 GeV pi0 generated in theta [10;20] degree an phi in range [0;360] degree.

pi0 opening angle:
index.php?t=getfile&id=5059&private=0

and reconstructed invariant mass:
index.php?t=getfile&id=5060&private=0


Dima
  • Attachment: pi0_angle.png
    (Size: 15.01KB, Downloaded 591 times)
  • Attachment: pi0_mass.png
    (Size: 11.55KB, Downloaded 475 times)
Previous Topic: correction in emc bump splitting
Next Topic: New geometry of the Backward EndCap
Goto Forum:
  


Current Time: Tue Jul 23 08:38:02 CEST 2024

Total time taken to generate the page: 0.00701 seconds