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 157
    • Merge requests 157
  • 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
  • !43204

Remove unused b-tagging configuration

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Dan Guest requested to merge dguest/athena:cleanup-config into master May 04, 2021
  • Overview 29
  • Commits 11
  • Pipelines 7
  • Changes 86

This removes a substantial fraction of the configuration code for b-tagging. It shouldn't change anything, since all this code is dead.

I wasn't able to remove BTaggingFlags or BTaggingConfiguration completely: these are apparently used to set the AOD output collections somewhere. But I was able to remove 90% of the code in them. I also had to remove some code in the jet configuration that was manipulating BTaggingFlags but not actually doing anything.

I ran a diff over a few hundred events with and without this change, it made no difference (as expected).

Attention @duperrin

Edited May 10, 2021 by Dan Guest
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: cleanup-config