New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Phase-2 Inner Tracker local reconstruction configuration updates #32789
Conversation
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32789/20957
|
@cmsbuild, please test |
A new Pull Request was created by @mmusich (Marco Musich) for master. It involves the following packages: Configuration/AlCa @perrotta, @malbouis, @yuanchao, @jordan-martins, @chayanit, @wajidalikhan, @slava77, @christopheralanwest, @kpedro88, @srimanob, @tlampen, @jpata, @pohsun, @francescobrivio can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+1 |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c28da2/12645/summary.html Comparison SummarySummary:
|
RecoLocalTracker/SiPixelRecHits/python/CustomPixelCPEGeneric.py
Outdated
Show resolved
Hide resolved
RecoLocalTracker/SiPixelRecHits/python/CustomPixelCPEGeneric.py
Outdated
Show resolved
Hide resolved
Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py
Outdated
Show resolved
Hide resolved
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-32789/20982
|
+1 for #32789 b6b3a4c
|
+1 |
+1 |
+1 |
+Upgrade |
+alca |
+1 |
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 be automatically merged. |
PR description:
The purpose of this PR is to update the conditions and customizations used to deal with the Phase-2 Inner tracker local reconstruction.
With respect to last update (PR #32220) it has been found that:
The first point is addressed in commit 5e2562c by updating the
autoCondPhase2.py
file, while the second is addressed by introducing twoEra
s be used on top of thePixelCPEGeneric
process modifier (N.B. its name has been changed since now is generic enough not to entail only phase2):Phase2C11T23
(targeting geometry T23, and 3D pixels in general) removes any usage of template / genError payloads from the reconstruction, as those are not yet available for 3D pixelsPhase2C11T22
(targeting geometry T22 and square pixels in general) allows to use genErrors in the seeding step (when no track angles are available). This is needed because hardcoded errors in https://github.com/cms-sw/cmssw/blob/master/RecoLocalTracker/SiPixelRecHits/src/PixelCPEGeneric.cc#L113 have been optimized for rectangular 25x100 pixels, and in the current generic reco setup we use hardcoded errors for the first tracking pass (see PR Phase2 Updated Pixel Conditions and GenericCPE Seeding Errors Fix #29783)PR validation:
Run successfully
runTheMatrix.py --what upgrade -l 29806.0,30606.0,30206.0
to test technical soundness of the code changes.Private internal validation of the new setup is available here: https://dinardo.web.cern.ch/dinardo/TMPshare/
In particular the pulls for the track parameters for T22 have changed from:
In blue is T22 (with generic CPE) while in red is T21 (with template CPE, baseline).
if this PR is a backport please specify the original PR and why you need to backport that PR:
This PR is not a backport and no backport is needed.
cc:
@emiglior @tsusa @OzAmram @tvami @gennai @dinardo