Skip to content

PrLongLivedTracking to functional including use of new MultiIndexedHitContainer

Christoph Hasse requested to merge PrLongLivedTracking2Functional into future

This branch is a dev branch currently used by @ldufour , @rquaglia and @chasse.
The overall goal is to work on this together while we are all at CERN and merge this by the end of this week.

@decianm We were wondering if it would be ok in the future branch to start getting rid of older algorithms like the PrDownstream and PrMatch which is superseded by the @sesen PrMatchNN.
The problem here is that these require us to keep the old state of the PrUTHit etc..


Included in this branch so far:

  • MultiIndexedHitContainer is added.
  • The IndexedHitContainer1D is just an alias to the MultiIndexedHitContainer
  • The PrFTHitHandler uses the MultiIndexedHitContainer.
  • PrUTHitHandler supports now the MultiIndexedHitContainer as well.
  • PrLongLivedTracking & PrVeloUT are both still working (use the hit handler)
  • PRLongLivedTracking is now ported to the new framework BUT no support for flagging and PrDebugTool still WIP

Things to DO :

  • PrUTHit lives together with UTHit, get rid of duplicated temporary hacks like this. [ UTHIt get rid of inheritances from ST/OT/IT]
  • AddUtHitTool should be adapted to use the PrUTHitHandler , once it'is done, we can can move to a single UTHit class [ WIP in !530 (closed)]
  • Check Forward and Matching if they need some modification of UTHit. [WIP in !530 (closed)]
  • PrDebugTool needs to be ported/checked, and in general all the consumers in PrMCTools using it.
  • We should update the functions in TfKernel/RecoFunc.h to handle the new Hit and then move to using these generic functions. [Should be fixed in !530 (closed)]

Merge request reports