-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Add Phase 2 Trackers T21 + T22 + T23 (Mechanical Update in Outer Tracker + Sensors studies in Inner Tracker) #30976
Conversation
…mmusich with cms-merge-topic
…acilitate IT insertion) + In all TEDD, update dee sensors inter-spacing in Z. Add T22 Inner Tracker topology file.
…dd D64 (Tracker T23).
…hen I added 3D sensors support, so update README now ;p.
…s in the upgradeWorkflowComponents file
add Global Tags for geometries T21, T22, T23 and adjust customizations in the upgradeWorkflowComponents file
… Outer Trackers, as evth should be identical from OT cabling map point of view. Given how the cabling map is computed, these geo changes should not impact the cablinmg map. Commit may not be necessary, but should not harm.
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-30976/17421
|
A new Pull Request was created by @ghugo83 for master. It involves the following packages: Configuration/AlCa @chayanit, @wajidalikhan, @ianna, @kpedro88, @rekovic, @tlampen, @pohsun, @civanch, @makortel, @cmsbuild, @davidlange6, @Dr15Jones, @cvuosalo, @mdhildreth, @tocheng, @qliphy, @fabiocos, @benkrikler, @christopheralanwest, @pgunnell, @silviodonato, @franzoni can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
@cvuosalo yes, that's the fix I'm talking about |
+upgrade |
@bsunanda that would be great, thanks! |
Yes exactly. Also that would just save up a few lines, for getting the same results, with risks of regressions, and that file will for sure still exist in tkLayout anyway. |
+1 |
+operations |
do you have any comments @cms-sw/alca-l2 @cms-sw/l1-l2 ? |
+1 |
@silviodonato Can this PR be merged - I am waiting for this to update some of the Ox parts of Phase2 scenarios |
merge |
fix upgrade workflow steps for wf introduced in #30976
PR description:
This PR:
T21:
tkLayout description: http://ghugo.web.cern.ch/ghugo/layouts/T21/OT800_IT615/layoutpixel.html
Geometry scenario: 2026D63.
Workflows: 294xx (no PU), 296xx (PU).
T22:
tkLayout description: http://ghugo.web.cern.ch/ghugo/layouts/T22/OT800_IT621/layoutpixel.html
Geometry scenario: 2026D64.
Workflows: 298xx (no PU), 300xx (PU).
T23:
tkLayout description: http://ghugo.web.cern.ch/ghugo/layouts/T23/OT800_IT700/layoutpixel.html
Geometry scenario: 2026D65.
Workflows: 302xx (no PU), 304xx (PU).
Following comparisons should be made:
Validate new Outer Tracker:
Compare T21 versus T20 (2026D63 versus 2026D56).
Should not expect any meaningful change in the tracking performance (maybe an extremely tiny degradation).
Inner Tracker sensors studies:
Now have a common base to compare different sensors options.
Only bricked pixels sensors option is missing (I included that geometry in another private branch).
25x100 -> 50x50:
Compare T22 (50x50, planar) versus T21 (25x100, planar) (2026D64 versus 2026D63).
An interesting fast sim estimate is available at:
25x100: http://ghugo.web.cern.ch/ghugo/layouts/T21/OT800_IT615/errorstracker.html
50x50: http://ghugo.web.cern.ch/ghugo/layouts/T22/OT800_IT621/errorstracker.html
With obviously, especially visible here at high pT, a significant degradation of pT and transverse impact parameter resolution, and an improvement of longitudinal impact parameter resolution. Of course less visible at low pT (multiple scattering).
planar -> 3D:
Compare T23 (3D) versus T21 (planar) (2026D65 versus 2026D63).
tkLayout track parameters code fully debugged and operational in the case of 3D sensors.
Though, would need to fix 3D local resolution parametrization used as input in tkLayout, to get meaningful fast sim results on that front as well.
PR validation:
Following was done for CMSSW validation:
FYI: @dpiparo @VinInn @cvuosalo @civanch @ianna @emiglior @skinnari @mmusich @jalimena @fabiocos @kpedro88