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 203
    • Merge requests 203
  • 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
  • !60081

DerivationFrameworkTrainsART: Add lightweight athena reading checks to PHYS+PHYSLITE data18 and mc20 tests

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Alaettin Serhan Mete requested to merge amete/athena:master-update-derivation-tests into master Jan 25, 2023
  • Overview 7
  • Commits 3
  • Pipelines 2
  • Changes 8

This is a follow-up to !60047 (merged). Here we add an additional check to the PHYS+PHYSLITE data18 and mc20 ART tests. These new checks basically highlight shortcomings in reading these files in athena, which are not always caught in vanilla ROOT validation checks.

I also updated the data18 tests such that we now run 4 workers instead of 8. This stems from the fact that the input has only 6k events, which is not enough to fully test the new SharedWriter with that many workers.

cc: @gemmeren @mnowak @calpigia @jcatmore

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: master-update-derivation-tests