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 158
    • Merge requests 158
  • 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
  • !44005

Don't use btaggingLink in FlavorTagDiscriminants

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Dan Guest requested to merge dguest/athena:unbtaglink into master May 31, 2021
  • Overview 19
  • Commits 4
  • Pipelines 4
  • Changes 35

Keeping track of data dependencies is easier if we retrieve containers directly rather than getting them via a link from another container. This merge makes some changes in FlavorTagDiscriminants to avoid accessing btagging via the link we store in jets.

Part of this is changing the tool interface for b-tag decorations so that we pass around BTagging objects rather than Jet objects. I'm also deleting a few wrapper algorithms in favor of BTagDecoratorAlg, which is slightly more generic.

This shouldn't change anything in the output files: I compared a hundred or so events and saw no differences in any branches.

Edited Jun 01, 2021 by Dan Guest
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: unbtaglink