|
Re: emc data flow diagram [message #5365 is a reply to message #5350] |
Tue, 06 November 2007 09:39 |
Jens Sören Lange
Messages: 193 Registered: June 2005
|
first-grade participant |
From: *physik.uni-giessen.de
|
|
Hi Dima,
as no emc expert replied, I would like to say that I think that an empty PndEmcBump to PndEmcRecoHit convertor for now is a good idea. I think we will postpone the release 2.0 for a few more days, maybe until the weekend, so there is still some time and maybe you can try. There are many things to think about, and maybe some things even have to be discussed on the next evo meeting, so a few days is not a problem, I think. Many people will be at GSI tomorrow, so we will try to talk to Mohammad and Florian to see, what they think.
So why do I think that the empty convertor is sufficient for now? As far as I can see it is easy to get the bump positions (e.g. TVector3 v1 in macro/emc/bump_analys.C). So if it turns out in the next few weeks that the energy and coordinate correction is absolutely needed immediately (I still think that at first we would have to check the g4 vs. g3 comparison again _before_ any applied correction which - expressed in a simplified way - "pushes" the cluster energy where we want to) then it seems that it is not so difficult to implement a preliminary correction (just for checking) not as a task, but just as a few lines into the macro bump_analys.C. Would you agree?
best regards, -- Soeren
|
|
|
Re: emc data flow diagram [message #5366 is a reply to message #5365] |
Tue, 06 November 2007 11:30 |
StefanoSpataro
Messages: 2736 Registered: June 2005 Location: Torino
|
first-grade participant |
From: *physik.uni-giessen.de
|
|
Hi,
I think that if the PndEmcRecoHit should be our "final product" and almost a copy of PndEmcBump, it makes no sense to create an additional object: let's rename PndEmcBump into PndEmcRecoHit and that will be all, maybe with some modifications. I do not think we have to duplicate the same information in the stream, I think it is not necessary.
However, for the energy calibration, what one could do is to add a fEnergyCorrected as data member for the Bumb(RecoHit), that at the beginning will be equal to the normal fEnergy. Then one calls a Calibrater task which overwrites only the fEnergyCorrected, without needs of additional objects. In this way we have in the same object both the raw energy and the calibrated one.
In each case energy calibration depends on the transport model, so this step could be tricky before a good physics validation.
Opinions are welcome...
|
|
|
Re: emc data flow diagram [message #5370 is a reply to message #5366] |
Wed, 07 November 2007 16:07 |
Dima Melnychuk
Messages: 213 Registered: April 2004 Location: National Centre for Nucle...
|
first-grade participant |
From: *fuw.edu.pl
|
|
Hi all,
I added to emc 2 classes PndEmcRecoHit and PndEmcMakeRecoHit.
I understand arguments of Stefano that it's almost a copy of PndEmcBump but nevertheless I created it as an additional class and not repalced the PndEmcBump.
The argument is that in PndEmcBump methods energy() and position() calculated correspondent values but in PndEmcRecoHit GetEnergy() and GetPosition() simply return correspondent data member so it can be more convenient for the following analysis. And in addition PndEmcRecoHit doesn't contain vector of PndEmcDigi's while PndEmcBump does.
But all this should be discussed and modified if necessary.
And I modified full_emc.C macro to produce at the end PndEmcRecoHit's objects.
Best regards,
Dima
|
|
|