Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • 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 172
    • Merge requests 172
  • 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
  • !42484

Merged
Created Apr 12, 2021 by Nicholas Styles@nstylesOwner

Have SCT_Digitization classes inherit from Interfaces in SiDigitization

  • Overview 4
  • Commits 2
  • Pipelines 2
  • Changes 16

Following discussions with @jchapman and @csebasti, move Interface class headers out of SCT_Digitization and into SiDigitization (and make them more generic), and update the implementations of these classes to inherit from them.

This will allow:

  • The same Interfaces to be used outside of SCT_Digitization in dedicated Digitization packages for e.g. ITkStrip and HGTD, to be added in future MRs.
  • The same implementations of e.g. SCT_FrontEnd to be used in such new packages (and new DigitizationTools), until they likely get their own dedicated class implementaions in future (once updates mean that they would need to diverge from the SCT implementation) also inheriting from the Interfaces in SiDigitization.

The motivation is to give the necessary flexibility for dedicated tools for new detectors to be added in future, while avoiding code duplication in the meantime.

NB, SCT_InducedChargeModel is also moved to SiDigitization (as InducedChargeModel) for similar reason, but in future should possibly be re-written as a ConditionsStore object.

FYI @hhayward @tadej @cohm @tawang

Edited Apr 12, 2021 by Nicholas Styles
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: master-SCT_DigiInheritanceUpdate