Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • athena athena
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Jira
    • Jira
  • Merge requests 155
    • Merge requests 155
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !38629

Cleanup B-tagging variables

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Dan Guest requested to merge dguest/athena:cleanup-btag-auxvar-names into master Nov 26, 2020
  • Overview 30
  • Commits 6
  • Pipelines 6
  • Changes 9

The b-tagging algorithms store some intermediate dynamic variables in the EDM. In general these have a form like ALGORITHM_VARIABLE, where ALGORITHM is a group like JetFitter or IP3D and VARIABLE is some discriminant that algorithm adds.

This merge moves few variables that should be considered part of JetFitterSecondaryVertex into that group. Moving them isn't strictly necessary but should avoid confusion in the future. Specifically the following variables are changed:

minimumTrackRelativeEta -> JetFitterSecondaryVertex_minimumAllJetTrackRelativeEta
maximumTrackRelativeEta -> JetFitterSecondaryVertex_maximumAllJetTrackRelativeEta
averageTrackRelativeEta -> JetFitterSecondaryVertex_averageAllJetTrackRelativeEta

I think @jshlomi originally came up with these variables, maybe he has a better suggestion for what to call them.

As far as I know these variables are only used in two places:

  • monitoring (@cmilke take note, I touched some of your code)
  • training high level taggers like DL1 (should be easy to accommodate in the downstream code)

This is also a minor bugfix: there were some extra variables stored in the trigger outputs that we no longer fill.

Edited Nov 27, 2020 by Dan Guest
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: cleanup-btag-auxvar-names