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 148
    • Merge requests 148
  • 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
    • Repository
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • atlasatlas
  • athenaathena
  • Merge requests
  • !48282

Prevent CP algorithms from copying the EventInfo collection

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nils Erik Krumnack requested to merge krumnack/athena:sweep_pileup into master Nov 16, 2021
  • Overview 16
  • Commits 2
  • Pipelines 5
  • Changes 18

This is a cherry-pick of a24324e3 from 21.2.

I had to comment out some config files that don't seem to work in R22, that should be fixed separately. For now I'd rather have consistent code between 21.2 and master. From what I can tell this doesn't actually break anything, but the tests that are failing are essentially new.

I also added a second commit that allows to call configure on the pileup sequence (again) like for any other CP algorithm sequence. It is a no-op right now, but since it does more than just manage the temporary copies, I figured it is better to keep it around. This will have to be (manually) swept back into 21.2 once this is merged.

cc @jburr

Edited Nov 18, 2021 by Nils Erik Krumnack
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: sweep_pileup