Skip to content
GitLab
Projects Groups Topics 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
    • Contributor statistics
    • Graph
    • Compare revisions
    • Locked files
  • Jira
    • Jira
  • Merge requests 137
    • Merge requests 137
  • 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
  • !61546

Bring back dual-use SUSYTools

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Sara Alderweireldt requested to merge salderwe/athena:dev/ST_AthAna into master Mar 15, 2023
  • Overview 61
  • Commits 28
  • Pipelines 13
  • Changes 28

See ATLSUSYBGF-654 for further discussion.

This Draft MR includes development to restore dual-use SUSYTools, testing with AthAnalysis,master,r11.

  • general fixes to CMake files, output types, EventInfo calls, and b-tagging handling
  • reshuffle in test JO
  • example of AA ART test version

Non-SUSYTools issues under investigation, highlighting that we have a few more tools that aren't dual-use yet:

  • locked container issue with NNJvt recalculation --> !62030 (merged)
  • Configuration of OverlapRemovalTools: fixing init in configuration (--> leave it to the parent) --> !62031 (merged)
  • DecorWriteHandleKey issues where not initialised in init functions, e.g. in if-conditions in the BoostedJetTaggers package --> !62032 (merged)
  • AsgTools fixes --> !61986 (merged) We'll need some discussion with the developers of the other tools on these complications, I'll make this MR dependent, but hopefully we can merge a first functional solution soon.

Would probably merge after unrelated SUSYTools update !62029 (merged).

To do (not necessarily in this MR):

  • development of 2nd Alg to separate plain workflow and count validation (more like the original SUSYToolsTester) and histogram validation (more like the existing ART setup)

tagging: @will, @mrimoldi, @cmerlass, @miholzbo

Edited Mar 31, 2023 by Sara Alderweireldt
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: dev/ST_AthAna