Skip to content
Snippets Groups Projects

Add an algorithm to decorate BTagging object

Merged Dan Guest requested to merge dguest/athena:add-btagdeco into master

This adds a new algorithm to run b-tag decorators. Since it relies on some b-tag -> jet links that don't exist in release 21, I'm also adding an algorithm that creates the b-tag to jet links.

The immediate use-case is supporting soft-muon based taggers in release 21, but in the longer term this algorithm can also replace a few Algorithms in BTagging which do very similar things:

The advantage of using the algorithm here (beyond removing some boilerplate code) is that it should track the data dependencies for flavor tagging more accurately.

Edited by Dan Guest

Merge request reports

Loading
Loading

Activity

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

    added 3 commits

    Compare with previous version

  • Dan Guest added 1 commit

    added 1 commit

    Compare with previous version

  • Dan Guest added 1 commit

    added 1 commit

    Compare with previous version

  • Dan Guest added 3 commits

    added 3 commits

    • 48a4b258 - Add an algorithm to run btag decorators
    • d03b2a48 - Add an algorithm to build btag -> jet links
    • 7b06d6de - Readme updates

    Compare with previous version

  • Dan Guest marked this merge request as ready

    marked this merge request as ready

  • Dan Guest changed the description

    changed the description

  • This merge request affects 1 package:

    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants

    This merge request affects 18 files:

    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/CMakeLists.txt
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagAugmenterTool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagJetAugmenter.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagJetDecoratorAlg.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagMuonAugmenter.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagMuonAugmenterTool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagToJetLinkerAlg.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/DL2Tool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/ISingleJetDecorator.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/README.md
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagAugmenterTool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagJetAugmenter.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagMuonAugmenter.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagMuonAugmenterTool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/DL2Tool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/BTagJetDecoratorAlg.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/BTagToJetLinkerAlg.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/components/FlavorTagDiscriminants_entries.cxx
  • Dan Guest changed title from Add a flavor tag discriminant writer to Add an algorithm to decorate flavor tagging discriminants to BTagging

    changed title from Add a flavor tag discriminant writer to Add an algorithm to decorate flavor tagging discriminants to BTagging

  • Dan Guest changed title from Add an algorithm to decorate flavor tagging discriminants to BTagging to Add an algorithm to decorate BTagging object

    changed title from Add an algorithm to decorate flavor tagging discriminants to BTagging to Add an algorithm to decorate BTagging object

  • :white_check_mark: CI Result SUCCESS (hash 7b06d6de)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :warning: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :warning: Athena: number of compilation errors 0, warnings 1
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31167]

  • Dan Guest added 1 commit

    added 1 commit

    • 87548d66 - Add missing event context (thanks helpful warning!)

    Compare with previous version

  • This merge request affects 1 package:

    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants

    This merge request affects 18 files:

    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/CMakeLists.txt
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagAugmenterTool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagJetAugmenter.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagJetDecoratorAlg.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagMuonAugmenter.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagMuonAugmenterTool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/BTagToJetLinkerAlg.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/DL2Tool.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/FlavorTagDiscriminants/ISingleJetDecorator.h
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/README.md
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagAugmenterTool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagJetAugmenter.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagMuonAugmenter.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/BTagMuonAugmenterTool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/Root/DL2Tool.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/BTagJetDecoratorAlg.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/BTagToJetLinkerAlg.cxx
    • PhysicsAnalysis/JetTagging/FlavorTagDiscriminants/src/components/FlavorTagDiscriminants_entries.cxx
  • :white_check_mark: CI Result SUCCESS (hash 87548d66)

    Athena AthSimulation AthGeneration AnalysisBase AthAnalysis DetCommon
    externals :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    cmake :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    make :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    required tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:
    optional tests :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark: :white_check_mark:

    Full details available on this CI monitor view
    :white_check_mark: Athena: number of compilation errors 0, warnings 0
    :white_check_mark: AthSimulation: number of compilation errors 0, warnings 0
    :white_check_mark: AthGeneration: number of compilation errors 0, warnings 0
    :white_check_mark: AnalysisBase: number of compilation errors 0, warnings 0
    :white_check_mark: AthAnalysis: number of compilation errors 0, warnings 0
    :white_check_mark: DetCommon: number of compilation errors 0, warnings 0
    :pencil: For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31170]

  • A bunch of new code, easy to read and otherwise well commented. Approved, -- L1

  • mentioned in commit 7917ab37

Please register or sign in to reply
Loading