Skip to content

Introduce Phase 2 FastSim (tracker) workflows#50437

Merged
cmsbuild merged 34 commits into
cms-sw:masterfrom
kpedro88:phase2fastsim_161X_trackerdev
Mar 19, 2026
Merged

Introduce Phase 2 FastSim (tracker) workflows#50437
cmsbuild merged 34 commits into
cms-sw:masterfrom
kpedro88:phase2fastsim_161X_trackerdev

Conversation

@kpedro88
Copy link
Copy Markdown
Contributor

PR description:

The culmination of substantial work primarily by the IISER Pune and Johns Hopkins University groups, with assistance from SIM and the tracker DPG.

Major developments:

  • New definition of FastSim simplified tracker geometry and material introduced to correspond to Phase 2 upgrade
  • New pixel resolution histograms from JHU at phase 2 pixel histograms (from JHU) cms-data/FastSimulation-TrackingRecHitProducer#5
  • Phase 2 digitization sequences updated to work with FastSim
  • Updated FastSim tracking iterations to correspond to Phase 2 sequences
  • Workflows added for D110 (development baseline for past ~year) and D121 (current default)

Technical developments:

  • Change handling of FastSim geometry sequences to adapt more easily to multiple Phase 2 geometries
  • Add an Era "Util" to make temporary FastSim-specific workflow changes to Phase 2 Eras
  • Simplify some old code in DIGI aliases

Still to come later:

  • Realistic outer tracker resolutions
  • Overall physics validation
  • Move beyond tracking-only reconstruction in workflows

PR validation:

Ran new workflows 36434.0 and 36834.0 successfully.

Also ran Run 3 workflow 16434.0 to ensure previous sequences were not disrupted by these changes.

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

N/A

alaha999 and others added 30 commits March 13, 2026 18:28
… separate modifier for phase 2 strip resolutions (values to be revisited)
@civanch
Copy link
Copy Markdown
Contributor

civanch commented Mar 19, 2026

+simulation

@gabrielmscampos
Copy link
Copy Markdown
Member

+dqm

@jfernan2
Copy link
Copy Markdown
Contributor

+1

@AdrianoDee
Copy link
Copy Markdown
Contributor

@kpedro88 is it worth having one FastSIM (such as 36434.0) in the limited matrix?

@kpedro88
Copy link
Copy Markdown
Contributor Author

I was thinking about it, but I think we are not quite ready yet. Let's start with IB tests only and see how it goes.

@AdrianoDee
Copy link
Copy Markdown
Contributor

+pdmv

@kpedro88
Copy link
Copy Markdown
Contributor Author

+fastsim

@cmsbuild
Copy link
Copy Markdown
Contributor

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @mandrenguyen, @ftenchini (and backports should be raised in the release meeting by the corresponding L2)
Notice This PR was tested with additional Pull Request(s), please also merge them if necessary: cms-data/FastSimulation-TrackingRecHitProducer#5, cms-sw/cmsdist#10426

@ftenchini
Copy link
Copy Markdown

+1

@cmsbuild cmsbuild merged commit 39f44e8 into cms-sw:master Mar 19, 2026
11 checks passed
@kpedro88
Copy link
Copy Markdown
Contributor Author

@cms-sw/orp-l2 please consult the following message (from comment above):

Notice This PR was tested with additional Pull Request(s), please also merge them if necessary: cms-data/FastSimulation-TrackingRecHitProducer#5, cms-sw/cmsdist#10426

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.