Skip to content
Snippets Groups Projects

DerivationFramework: Addition of UTT chains to TRIG8

Merged Ondrej Kovanda requested to merge okovanda/athena:trig8-amendments into main
All threads resolved!

Adding Unconventional Tracking Trigger + reference HLT_mu60_L1MU14FCH chains into TRIG8 derivation format to allow for trigger-level matching and efficiency studies in the UTT triggers.

UTT triggers added: HLT_xe80_tcpufit_isotrk120_medium_iaggrmedium_L1XE55 HLT_xe80_tcpufit_isotrk140_medium_iaggrmedium_L1XE55 HLT_xe80_tcpufit_dedxtrk50_medium_L1XE50 HLT_xe80_tcpufit_distrk20_medium_L1XE50 HLT_xe80_tcpufit_distrk20_tight_L1XE50

Tested on data (2k events of 2022 run 439927), the amended derivation selects 10.9% more events (now 11.7%), with the per-event size increasing by 1% to 100.6 kb.

cc: @sutt

Merge request reports

Pipeline #6811124 passed

Pipeline passed for a819a25c on okovanda:trig8-amendments

Merged by Adam Edward BartonAdam Edward Barton 1 year ago (Jan 30, 2024 4:44pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Mark Sutton
  • Mark Sutton
  • Ondrej Kovanda added 1 commit

    added 1 commit

    • a819a25c - Addition of UTT triggers and removing hanging commas in lists

    Compare with previous version

  • Mark Sutton resolved all threads

    resolved all threads

  • This merge request affects 1 package:

    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkTrigger

    This merge request affects 1 file:

    • PhysicsAnalysis/DerivationFramework/DerivationFrameworkTrigger/python/TRIG8.py

    Adding @emmat ,@jcatmore as watchers

  • :white_check_mark: CI Result SUCCESS (hash 2aa7fa15)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3806]

  • :white_check_mark: CI Result SUCCESS (hash a819a25c)

    Athena
    externals :white_check_mark:
    cmake :white_check_mark:
    make :white_check_mark:
    tests :white_check_mark:

    Full details available on this CI monitor view. Check the JIRA CI status board for known problems
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 3809]

  • The changes are straightforward and there are no issues with the CI. Approving from L1.

  • added review-approved label and removed review-pending-level-1 label

  • mentioned in commit f5edfb8a

  • Please register or sign in to reply
    Loading