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 157
    • Merge requests 157
  • 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
  • !29907

Run3 adaptions to CSC condition access (ATLASRECTS-5276)

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nicolas Koehler requested to merge nkoehler/athena:rec5276 into master Jan 29, 2020
  • Overview 5
  • Commits 4
  • Pipelines 2
  • Changes 14

Hi,

this MR is a follow-up MR of !26583 (merged) which completed the thread-safe CSC condition access migration. Since !26583 (merged) was opened for a long time, it went out of sync with the Run3 developments wrt to the addition of 1(2) NSW and thus, the removal of CSC chambers. This MR adds back these developements (made in !26610 (merged)) to the CscCondDbData and CscCondDbAlg. Furthermore, the MuonIdHelperSvc is used instead of the MuonIdHelperTool and the CSC/PSLOPE parameter (which is the same float for all CSC channels) is not read from the conditions database anymore by default but just taken from a const float instead to safe computing time.

These changes should also fix ATLASRECTS-5276.

Adding @cheidegg for his information

Best, Nico

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: rec5276