Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • M MooreAnalysis
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 13
    • Issues 13
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
  • Merge requests 2
    • Merge requests 2
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • 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
  • MooreAnalysis
  • Merge requests
  • !51

Calculate TOS efficiencies for Allen in HltEfficiencyChecker

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Ross John Hunter requested to merge TOS-for-Allen-HltEfficiencyChecker into master Sep 29, 2021
  • Overview 4
  • Commits 21
  • Pipelines 11
  • Changes 7

FYI @rmatev @sstahl @mvesteri

Configure HltEfficiencyChecker to add MCTupleToolTOSHLT1 to the control flow + dependencies that have recently become available after SelReports were added to Allen. Many thanks to @raaij @thboettc for outlining how to do this.

Also fix the default FT decoding version to be the same (4) if using the "wizard" or "by-hand". If unspecified, you got v6 in the yaml and v4 when writing your options file by hand.

TODO:

  • Understand why MCTupleToolTOSHLT1 is occasionally finding empty vertex-like HltObjectSummary objects in the SelReports when it appears to be running single track lines. Fixed by @thboettc in Allen!681 (merged)

Requires Allen!673 (merged), Allen!681 (merged) and Moore!1048 (merged), and required if we want to show TOS efficiencies for Allen in the detector construction paper.

Edited Oct 07, 2021 by Ross John Hunter
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: TOS-for-Allen-HltEfficiencyChecker