Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • Phys Phys
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Jira
    • Jira
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCb
  • PhysPhys
  • Merge requests
  • !302

Add Particle to RelatedInfo cloner needed for Turbo processing

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Alex Pearce requested to merge apearce-lbhlt-421 into 2018-patches Apr 03, 2018
  • Overview 16
  • Commits 4
  • Changes 6

Add the CopyParticle2RelatedInfoFromLinePersistenceLocations algorithm, which is necessary for us to clone Particle to RelatedInfo maps in Moore and Tesla. Part of LBHLT-421.

Rather than writing a whole new algorithm, I've only derived from the CopyParticle2RelatedInfo class and then overriden the MicroDSTAlgorithm::inputTESLocations method. This is because the parent RelationsFromParticleLocationsFromClonerAlg algorithm, which does the heavy lifting, calls inputTESLocations to know which locations to clone, which is exactly the behaviour we need to override in Moore.

Sorry for targeting 2017-patches; I'm working with that locally and it seems the various cherry-picks mean I can't target 2018-patches cleanly. Please also port these commits to 2018-patches. (I can fixup and target 2018-patches myself if that's preferred.) Now targets 2018-patches. /cc @rmatev @sstahl

Edited Apr 04, 2018 by Alex Pearce
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: apearce-lbhlt-421