Skip to content

GitLab

  • Menu
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 137
    • Merge requests 137
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • Code review
    • Issue
    • Repository
  • Activity
  • Graph
  • Commits
Collapse sidebar
  • atlas
  • athenaathena
  • Merge requests
  • !33194

Fix ITk irreproducibility [ATLITKSW-117]

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Thomas Strebler requested to merge tstreble/athena:ATLITKSW-117-fix into 21.9 May 26, 2020
  • Overview 15
  • Commits 6
  • Pipelines 2
  • Changes 2

This MR addresses an irreproducibility issue observed in ATLITKSW-117

A check for invalid SiCellId is added in the phiPitch functions.

The negative value of the pitch at the middle of the sensor is returned to allow for potential error catching while still being usable in cases such as

https://gitlab.cern.ch/atlas/athena/-/blob/21.9/InnerDetector/InDetRecTools/SiClusterOnTrackTool/src/SCT_ClusterOnTrackTool.cxx#L289

where the value is squared.

FYI @ncalace @nstyles @hessey

Edited May 26, 2020 by Thomas Strebler
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: ATLITKSW-117-fix