-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Run3-gex10 Recreate Run3 scenarios with the new xml files #31286
Changes from 1 commit
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,8 +5,10 @@ The Run 3 geometry is automatically created using the script [generate2021Geomet | |
Different versions of various subdetectors can be combined. The available versions are: | ||
|
||
Tracker: | ||
* T1: 2021 baseline | ||
* T1: 2021 baseline (old) | ||
* T2: as T1, but with zero material | ||
* T3: 2021 baseline after separating tracker specific material | ||
* T4: as T3, but with zero material | ||
|
||
Calorimeters: | ||
* C1: 2021 baseline | ||
|
@@ -15,16 +17,17 @@ Muon system: | |
* M1: 2021 baseline | ||
|
||
PPS: | ||
* P1: 2021 baseline | ||
* P1: 2021 baseline (old) | ||
* P2: 2021 baseline (after using its own material files for pixel) | ||
|
||
The script also handles the common and forward elements of the geometry: | ||
* O1: 2021 baseline | ||
* O2: as O1, but with zero material | ||
* O3: 2021 baseline | ||
* O4: as O3, but with zero material | ||
* F1: 2021 baseline | ||
|
||
Several detector combinations have been generated: | ||
* 2021 = T1+C1+M1+P1+O1+F1 | ||
* 2021ZeroMaterial = T2+C1+M1+P1+O2+F1 | ||
* 2021 = T3+C1+M1+P2+O3+F1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @ianna is this sort of change acceptable for Run 3? (It wouldn't be allowed for phase 2) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @bsunanda Could you try to respond to Kevin's questions? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I hope it is allowed. We are not versioning 2021 geometries. The basic design is not changed. Only the materials are rearranged for future usage (studies of tracker material budget by changing densities of tracker specific materials) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @kpedro88 - 2021 is still work in progress. There is no corresponding DB payload and it has not been used in production. IMHO, it’s ok to modify it at this point. |
||
* 2021ZeroMaterial = T4+C1+M1+P2+O4+F1 | ||
|
||
# Phase 2 Geometries | ||
|
||
|
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add
deprecatedSubdets
for 2021 (as is done for 2026)?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed that
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bsunanda - should we just name them O1 and O2? The scenario has not been used in production yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer to keep O1, O2 just as a reference. Here we changed the choice of xml files. Future fixes (if any) will be fixing the xml files