Skip to content
GitLab
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 170
    • Merge requests 170
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !27538

Zeroing MC event weights on old-style EventInfo

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Zach Marshall requested to merge zmarshal/athena:21.2_OldEventInfoZeroWeight into 21.2 Oct 27, 2019
  • Overview 8
  • Commits 1
  • Pipelines 1
  • Changes 1

This solves the duplication of information in derivations between the old-style EventInfo and the xAOD::EventInfo. The latter should be the exclusive source for the information, and is used by all the PMG tools. Therefore, we're clearing the event weights on the old-style EventInfo. In release 22, we can simply outright drop the old-style EventInfo, but in release 21 that will result in files that can't be read by athena (or the derivation framework, for example). By putting this change into the TP converter in 21.2, we get the change for all derivations immediately.

This change was discussed with @jcatmore and @elmsheus . FYI: @ewelina , @lcorpe , @amoroso , @cgutscho .

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 21.2_OldEventInfoZeroWeight