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 156
    • Merge requests 156
  • 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
  • !59419

ATLASRECTS-7438 LArHVCondAlg . Do not retrieve CaloDetMgr from store. We have it already from Cond

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Christos Anastopoulos requested to merge ATLAS-EGamma/athena:LArHVCondAlg_ATLASRECTS-7438 into master Dec 17, 2022
  • Overview 9
  • Commits 2
  • Pipelines 2
  • Changes 2

LArHVCondAlg . Do not retrieve CaloDetMgr from store. We have it already from Cond

this prb escaped the cleanup we did with @tsulaia

So lets make sure we consistently retrieve from CondStore (actually we do this already in one method but we do not pass it down).

This relates to the cxx issues in ATLASRECTS-7438 .

The second commit should be the final fix and we should be getting a green CI after it.

also ping @fwinkl , @ssnyder , @wlampl

Edited Dec 17, 2022 by Christos Anastopoulos
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: LArHVCondAlg_ATLASRECTS-7438