Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • R Rec
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 282
    • Issues 282
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 48
    • Merge requests 48
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • Rec
  • Merge requests
  • !2482

Speed up and clean up PrAddUTHitsTool

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Andre Gunther requested to merge gunther_addUTHitsToolImprovements into master Jun 23, 2021
  • Overview 58
  • Commits 33
  • Pipelines 7
  • Changes 4

The tool that adds UT hits to long tracks (mainly used by the Forward Tracking and the Matching) is improved by avoiding memory (de)allocations and redundant accesses to the TES. The code is also cleaned up a bit. The tool gains a factor 2.5 in speed without changing the hit finding performance.

  • replace some magic numbers with meaningful ones
  • remove useless status code return

/cc @decianm @sstahl @peilian @chasse @lohenry

Edited Jun 25, 2021 by Rosen Matev
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: gunther_addUTHitsToolImprovements