Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • L LHCb
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Locked Files
  • Issues 193
    • Issues 193
    • List
    • Boards
    • Service Desk
    • Milestones
    • Iterations
    • Requirements
  • Jira
    • Jira
  • Merge requests 46
    • Merge requests 46
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
    • Test Cases
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Code review
    • Insights
    • Issue
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • LHCbLHCb
  • LHCb
  • Merge requests
  • !2246

Cleanups and changes in DetDesc to streamline the code and prepare DD4hep introduction

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Sebastien Ponce requested to merge sponce_detdescchanges into master Dec 05, 2019
  • Overview 11
  • Commits 12
  • Pipelines 17
  • Changes 16

Among other things :

  • Dropped unused VPTrackingInfo
  • Merged VPGeometry into the VPDetectorElement
  • Improved the VP DetectorElement interface to not expose the internals
  • Allow functional algorithms using conditions to have non default base class

One can expect tiny changes to the test outputs, but there should be not changes in the counters or histograms.

Related: Lbcom!391 (merged), Rec!1817 (merged), Alignment!107 (merged) and Boole!248 (merged)

Edited Dec 19, 2019 by Alex Pearce
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: sponce_detdescchanges