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 150
    • Merge requests 150
  • 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
  • !35991

Updated BeamPipeBuilder to pick physical radius

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Thomas Strebler requested to merge tstreble/athena:BeamPipeBuilder into master Aug 28, 2020
  • Overview 30
  • Commits 7
  • Pipelines 4
  • Changes 3

This MR updates the way the beam pipe radius is retrieved to build the InDetTrackingGeometry, based on the observation in Phase 2 studies that it is wrongly set at 27mm in tracking material map, instead of the expected 24mm. This is believed to be due to the fact that the radius retrieved is the one of the logical volume associated to the beam pipe which does not correspond to the physical volume. This updates returns the expected 24mm and it was checked to have a minimal impact when used together with the existing tracking material maps of ATLAS-P2-ITK-22-02-00.

FYI: @ncalace @nstyles @goblirsc @npetters

Edited Sep 02, 2020 by Thomas Strebler
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: BeamPipeBuilder