Skip to content
Snippets Groups Projects

Remove unused b-tagging configuration

Merged Remove unused b-tagging configuration
dguest/athena:cleanup-config into master
Merged Dan Guest requested to merge dguest/athena:cleanup-config into master

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 by Dan Guest

Merge request reports

Pipeline #2590551 passed

Stage: external

Pipeline passed for 5cd6a4c9 on dguest:cleanup-config

Approval is optional

Merged by Walter LamplWalter Lampl 4 years ago (May 10, 2021 7:08pm UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply