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 151
    • Merge requests 151
  • 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
  • !46863

configs, bugfixes and patching of rdo identifiers to have reco running with full digi input

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Alexander Leopold requested to merge atlas-hgtd/athena:hgtd_fix_digi_positions_21p9 into 21.9 Oct 01, 2021
  • Overview 76
  • Commits 18
  • Pipelines 0
  • Changes 15

I am proposing here a fix to keep working with the current sim structures to have RDO identifiers after full digi being usable for reco. This way we can keep using the produced HITS files and keep working in R21, while it should be cleaned up in master.

I ran on 500 single muons to check if the chi2 I get from the extension look good (so the HGTD_Cluster are in the right position after retrieving the RDOs), and that worked fine as far as I can tell. More in depth validation is still needed.

In addition, I am adding a couple of fixes to have everything running (similar to what was in MR 46526.

Tagging @tadej @ncalace @nstyles @cohm @dshope for feedback.

Edited Oct 02, 2021 by Alexander Leopold
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: hgtd_fix_digi_positions_21p9