Skip to content

Fix TPG for v19 #169

Merged
pfs merged 1 commit intoCMS-HGCAL:dev/fix_v19_digistepfrom
jbsauvan:fix_v19_tpg
Jun 4, 2025
Merged

Fix TPG for v19 #169
pfs merged 1 commit intoCMS-HGCAL:dev/fix_v19_digistepfrom
jbsauvan:fix_v19_tpg

Conversation

@jbsauvan
Copy link
Copy Markdown

@jbsauvan jbsauvan commented Jun 4, 2025

Fixing TPG for additional silicon type in V19 geometry:

  • Thickness corrections
    • Extend the vector to 4 items, copying LD200 to make HD200 correction
    • Could possibly use a V19 modifier if used to modify the original reco thickness correction vector
  • Update / improve the way silicon module density is extracted
    • Add lowDensity() and highDensity() methods for HGCalTriggerDetId and HGCalTriggerModuleDetId (similarly to what is already done in HGCSiliconDetId)
    • Add higher-level methods in HGCalTriggerTools to check low/high density type for any type of detid
    • Used in floating point energy compression (HGCROC) and SuperTriggerCell construction (ECON-T)
  • SuperTriggerCell sizes
    • STC sizes are defined by layer and by thickness, so increased vector size to account for 4 silicon thicknesses

⚠️ code-format modified a line in the HGC digitizer, not related to TPG

@pfs
Copy link
Copy Markdown

pfs commented Jun 4, 2025

many thanks

@pfs pfs merged commit 3abb9a2 into CMS-HGCAL:dev/fix_v19_digistep Jun 4, 2025
pfs pushed a commit that referenced this pull request Jun 12, 2025
pfs pushed a commit that referenced this pull request Jul 1, 2025
pfs pushed a commit that referenced this pull request Jul 1, 2025
pfs pushed a commit that referenced this pull request Jul 1, 2025
pfs pushed a commit that referenced this pull request Aug 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants