Keep the ZEM geometry instead of skipping the whole ZDC module - #2449
Open
sawenzel wants to merge 1 commit into
Open
Keep the ZEM geometry instead of skipping the whole ZDC module#2449sawenzel wants to merge 1 commit into
sawenzel wants to merge 1 commit into
Conversation
This changes the simulation default from skipping the ZDC module to dropping only
its +-113 m beam line, so the ZEM material stays in the transport geometry.
- The ZDC geometry is the +-113 m beam line, its magnets, ZN, ZP and ZEM; the
transport cost is the beam line, while ZEM sits at z = 7.6 m.
- ZEM carries 217 kg of material between z = 745 and 862 cm, spanning
eta 2.8 to 5.6, which is inside the FIT and FDD acceptance.
- --skipModules no longer defaults to ZDC; create_geant_config() sets
ZDCSimParam.buildBeamLine=false instead, unless --with-ZDC is given.
- An explicit --skipModules ZDC still removes the module altogether.
- ZDC stays out of digitisation and reconstruction as before, which the existing
deactivate_detector('ZDC') already handles.
- The MODULES variable removed from the two anchor scripts was never referenced;
it read as if anchored MC opted out of ZDC, which the workflow default decided.
- Requires ZDCSimParam.buildBeamLine in AliceO2.
https://its.cern.ch/jira/browse/O2-7158
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
Contributor
Author
|
Needs AliceO2Group/AliceO2#15752 be merged beforehand |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This changes the simulation default from skipping the ZDC module to dropping only its +-113 m beam line, so the ZEM material stays in the transport geometry.
https://its.cern.ch/jira/browse/O2-7158