Skip to content
Snippets Groups Projects

CP algorithms: Electron trigger SF calculation improvements

Merged Tadej Novak requested to merge tadej/athena:cp-trig-el into main

Standalone electron trigger SF calculation in CP algorithms improvements:

  • make HLT prefix configurable
  • make output prefixes configurable - useful if multiple WPs are covered (alternatively we could somehow add ID to the name, but current implementation allows more flexibility)
  • support all years for uniform branches - this currently only works either only for Run 2 or for Run 3, I hope this is acceptable
  • add ability to save output with tool key name as it makes the key the same for a Run
  • add a helper function to properly differentiate trilepton legs with others
  • adds extra e-tau (and mu-tau to avoid too many iterations) triggers that were missing

Keeping as draft for now to get feedback but to allow more testing.

/cc @tstreble @jedebevc @ravinab @jmaurer

Edited by Tadej Novak

Merge request reports

Pipeline #9936095 passed

Pipeline passed for 3c0f0d23 on tadej:cp-trig-el

Approval is optional

Merged by Frank WinklmeierFrank Winklmeier 4 months ago (Jan 22, 2025 6:37pm UTC)

Merge details

  • Changes merged into main with 38f9911b (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Tadej Novak added 190 commits

    added 190 commits

    Compare with previous version

  • Tadej Novak changed the description

    changed the description

  • Tadej Novak resolved all threads

    resolved all threads

  • Tadej Novak added 41 commits

    added 41 commits

    Compare with previous version

  • Tadej Novak changed the description

    changed the description

  • Tadej Novak resolved all threads

    resolved all threads

  • Tadej Novak mentioned in merge request !77288 (merged)

    mentioned in merge request !77288 (merged)

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading