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 175
    • Merge requests 175
  • 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
  • !60036

Create Valid SiHitIdentifiers in SQLite workflows

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Nicholas Styles requested to merge nstyles/athena:SiHitIdsFromName into master Jan 24, 2023
  • Overview 29
  • Commits 5
  • Pipelines 3
  • Changes 12

When using geometries coming from externally-generated (using standalone GeoModel) SQLite files, we cannot rely on the Sensitive Detector volumes having their copy number correctly set to a valid SiHitIdentifier, as we do when running the Geometry building fully within Athena.

Instead, we use the name of the GeoFullPhysVol to contain and later extract the necessary indices for building an Identifier when creating the HITS.

This relies on updates to GeoModel in GeoModelDev/GeoModel!181 (merged) in order for the names to be written properly, this is not used in any production workflow, so does not need to wait for those developments to be merged and picked up in Athena via an update of the GeoModel tag.

This approach can be extended to other systems using GeoModelXml, such as the HGTD (FYI @dshope @aleopold @sridouan)

Adding also @boudreau @dellacqu @tsulaia @rbianchi @jmasik @sroe @hhayward for info.

Edited Jan 30, 2023 by Nicholas Styles
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: SiHitIdsFromName