| Reco hit production with two inputs. [message #10558] |
Wed, 21 April 2010 17:29  |
Lia Lavezzi
Messages: 291
Registered: May 2007
Location: Torino
|
first-grade participant |
From: *pv.infn.it
|
|
Hi Genfitters,
I have a question concerning the reco hit production: is it mandatory to use the GFRecoHitProducer or can I create a PndSttRecoHitProducer, inheriting from GFAbsRecoHitProducer, and use that one?
I ask this because since I started using the STT tube map to access the geometrical information I had to access the parameters from inside the PndSttRecoHit::PndSttRecoHit(PndSttHelixHit *currenthit) and this slows down the code. To speed it up I would do like I do in digi and reco:
1) access the parameters only once per run
2) fill the tubeArray with the objs PndSttTube, which represent the tubes
3) have something like PndSttRecoHit::PndSttRecoHit(PndSttHelixHit *currenthit, TClonesArray *tubeArray) to pass the tubeArray directly to the recohit and avoid accessing the parameters at each hit.
I tried the PndSttRecoHitProducer solution and it seems to me that it works, but I wanted to ask if this is "permitted" before putting it on svn.
Cheers,
Lia.
|
|
|
|
| Re: Reco hit production with two inputs. [message #10561 is a reply to message #10558] |
Wed, 21 April 2010 19:26  |
|
Anonymous Poster
|
|
From: *e18.physik.tu-muenchen.de
|
|
Hi Lia,
it is no problem in terms of using GENFIT. The GFAbsRecoHitProducer allows you to do exactly that. Instead of the template approach of GFRecoHitProducer you solve the issue by inheritance. As far as I can tell now, it is no problem.
The point where it becomes important is at the KalmanTask level. Here you use the HitProducers. Stefano designed the common Kalman task to be used by everybody. So my question to Stefano is:
How do you configure in the KalmanTask, which hit producers are inserted in the GFRecoHitFactory for specific detector IDs? This is where Lia's question points to. Do you see any problem with her proposal?
Cheers, Christian
|
|
|
|
|
|
| Re: Reco hit production with two inputs. [message #10566 is a reply to message #10562] |
Thu, 22 April 2010 10:21 |
Lia Lavezzi
Messages: 291
Registered: May 2007
Location: Torino
|
first-grade participant |
From: *pv.infn.it
|
|
Hi Christian and Stefano,
| Quote: |
Is the class PndSttTube inside pnddata/SttData or stt
|
The PndSttTube is inside the stt directory. Anyway, the CMakeLists.txt in GenfitTools/recohit has the stt directory among the INCLUDE_DIRECTORIES, so there should be no problem.
I will try this (in my first trial I put the PndSttRecoHitProducer inside stt/sttreco) and see if it works. If so, I will put it in svn and give the instructions on the changes needed to use the stt producer and fill the map.
Thank you and ciao,
Lia.
|
|
|
|
GSI Forum
Search
Help
Members
Login
Home
