GSI Forum
GSI Helmholtzzentrum für Schwerionenforschung

Home » PANDA » PandaRoot » Tracking » Updates in PndTrack - PndTrackCand
Updates in PndTrack - PndTrackCand [message #11095] Wed, 13 October 2010 19:41 Go to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *180-80-r.retail.telecomitalia.it
Dear all,
after the EVO session of this morning, we have decided to apply the following scheme to the tracking code:

  • PndTrackCand will store ONLY hit informations (no momentum anymore)
  • PndTrack will store only the track informations (i.e. track parameters at the first and last hit), and an index of the corresponding PndTrackCand
  • each tracking algorithm will register two separated TClonesArray, one for PndTrack objects, the other for PndTrackCand
  • the link between PndTrack and PndTrackCand will be done using FairLink code


We have decided to apply the changes in three steps:

  1. Each tracking algorithm will write both PndTrack and PndTrackCand TClonesArray
  2. The momentum information will be removed from PndTrackCand
  3. The PndTrackCand object will be removed from PndTrack, leaving only the TCA index


We will proceed step by step, so that we will always be able to use track informations for analysis and tests.
Let's start with step 1.

The people involved should be:

MVD: Tobias/Ralf
STT: Gianluigi/Lia
TPC: Felix/Sebastian(?)/Christian(?)
GEM: Radek
LHE: Stefano
STT+MVD: Gianluigi
Global: Radex

I hope I have forgotten nobody.
When the first step change (i.e. writing two different TClonesArray made of PndTrack and PndTrackCand) will be applied, we cpuld move to the second step.

I would like to ask to developers to write in this forum the "OK" once the change will be done in the trunk, so that we could go on. It would be nice to have step 1 ready before next evo meeting (3rd november).



Re: Updates in PndTrack - PndTrackCand [message #11106 is a reply to message #11095] Tue, 19 October 2010 09:43 Go to previous messageGo to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: 140.109.127*
Step 1 - LHETRACK done (r10117)

It took me less than 30 minutes.
Waiting for all the other tracking algorythms...
Re: Updates in PndTrack - PndTrackCand [message #11159 is a reply to message #11106] Wed, 03 November 2010 12:32 Go to previous messageGo to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *to.infn.it
Updates on 3/11/2010

MVD riem: OK - Tobias
STT real: ongoing - Gianluigi
STT idea: OK - Lia
TPC real: ongoing - Sebastian
GEM real: OK - Radek
LHE idea: OK - Stefano
LHE real: OK - Stefano
STT+MVD: ongoing - Gianluigi
Global: OK - Radek
Re: Updates in PndTrack - PndTrackCand [message #11239 is a reply to message #11159] Sun, 28 November 2010 13:23 Go to previous messageGo to next message
Sebastian Neubert is currently offline  Sebastian Neubert
Messages: 282
Registered: March 2006
Location: Munich
first-grade participant

From: *natpool.mwn.de
Dear colleagues,

I have implemented the PndTrackCand output for the PndTpcRiemannTrackingTask.
Now I am about to add the PndTracks. However I think I do not yet fully understand how this all will work out in the end.

There are still seed values for the momentum and position in the PndTrackCand ... right?
The PndTracks are filled with content in a common TrackFitting task... right?

Is there a an agreed Track Merging strategy, yet? In that case I suppose there should be a mechanism to join several PndTrackCands. As far as I can tell this is not there ... right?

Anyhow. My current picture is this: The pattern reco (for each subsystem) will spit out a PndTrackCand array plus an PndTrack array. The Tracks will be linked too the cands but they will not contain fitted parameters yet.

Cheers! Sebastian.


Sebastian Neubert
Technische Universität München
Department Physik E18
sneubert@e18.physik.tu-muenchen.de
tel: +49-8928912592
Re: Updates in PndTrack - PndTrackCand [message #11240 is a reply to message #11239] Sun, 28 November 2010 14:08 Go to previous messageGo to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *4-87-r.retail.telecomitalia.it
Hi Sebastian,

Sebastian Neubert wrote on Sun, 28 November 2010 13:23

There are still seed values for the momentum and position in the PndTrackCand ... right?



At present yes,
Christian has added them on 20/07/2009 regardless the original design, but I don't know for which reason.
They will be removed in a short time, because useless for our design.

Quote:


The PndTracks are filled with content in a common TrackFitting task... right?



Not exactly. PndTracks are filled by several track finding tasks which should also give prefit parameters (i.e. from helix assumption).

Quote:


Is there a an agreed Track Merging strategy, yet? In that case I suppose there should be a mechanism to join several PndTrackCands. As far as I can tell this is not there ... right?



There is a track merging strategy, but at present there is no code doing it for all the detectors because each track finding/fitting is giving different track/candidate objects, even if the object design was fixed three years ago. This is what we are trying to solve.

Quote:


Anyhow. My current picture is this: The pattern reco (for each subsystem) will spit out a PndTrackCand array plus an PndTrack array. The Tracks will be linked too the cands but they will not contain fitted parameters yet.



This I have not understood.
Your finding must give also the prefit values, and put these values in the PndTrack (if not you cannot extrapolate and merge to different detectors)

Hoping it helps a bit...
Re: Updates in PndTrack - PndTrackCand [message #11241 is a reply to message #11240] Sun, 28 November 2010 15:10 Go to previous messageGo to next message
Sebastian Neubert is currently offline  Sebastian Neubert
Messages: 282
Registered: March 2006
Location: Munich
first-grade participant

From: *natpool.mwn.de
Hi Stefano!

Thank you for the quick reply...

Ok I begin to see the picture. I will thus put the prefit values to the PndTrack objects.

For backwards compatibility I will retain the direct Genfit output of the task until we have a common fitting task.

Cheers! Sebastian.


Sebastian Neubert
Technische Universität München
Department Physik E18
sneubert@e18.physik.tu-muenchen.de
tel: +49-8928912592
Re: Updates in PndTrack - PndTrackCand [message #11242 is a reply to message #11241] Sun, 28 November 2010 19:14 Go to previous messageGo to next message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *8-87-r.retail.telecomitalia.it
For backward compatibility, probably a "persistance" flag could be helpful, so that you can set if you want to store a GTrack, a PndTrack or both.
Removing Seed from PndTrackCand [message #11290 is a reply to message #11095] Thu, 09 December 2010 10:33 Go to previous message
StefanoSpataro is currently offline  StefanoSpataro
Messages: 2736
Registered: June 2005
Location: Torino
first-grade participant

From: *to.infn.it
Dear all,
considering that almost everybody has updated the tracking classes to store a PndtrackCand and also a PndTrack TClonesArrays, I would move to the second step -> removing the seeds from PndTrackCand.

This means deleting the following data members:

       TVector3 fPosSeed;
       TVector3 fDirSeed;
       double fQoverPseed;


and the correlated member functions.
I would like to ask to tracking developers to apply this modification in their code, considering that the seed should be taken from the PndTrack object.
If not, I can just delete all the calls to SetSeed and GetSeed, but it is important that the code knows where to take those values properly.

Waiting for feedback, this is the responsible list:

MVD: Tobias/Ralf
STT: Gianluigi/Lia
TPC: Felix/Sebastian
GEM: Radek
LHE: Stefano
STT+MVD: Gianluigi
Global: Radek

LHE is not using seeds as originally planned, therefore it is ready for 3rd step.

[Updated on: Thu, 09 December 2010 10:36]

Report message to a moderator

Previous Topic: Lhe Tracking: Kalman Task with STT, worse results
Next Topic: Template for association track from PR to MC track
Goto Forum:
  


Current Time: Mon Apr 29 01:57:59 CEST 2024

Total time taken to generate the page: 0.01010 seconds