-
Notifications
You must be signed in to change notification settings - Fork 61
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 dual-readout tubes barrel calorimeter in IDEA_o2 #413
Open
lopezzot
wants to merge
24
commits into
key4hep:main
Choose a base branch
from
lopezzot:lp-ideao2drbarrel
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains 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
Co-authored-by: Andreas Loeschcke Centeno <[email protected]>
Add dual-readout capillary-tubes barrel calorimeter inside the IDEA_o2 xml file. Change IDEA_o2 muon system dimensions to avoid overlaps with the barrel calorimeter (new params checked with Mahmoud Ali).
Move the detector dimensions parameters inside DectDimensions_IDEA_o2_v01.xml.
Move the vis attributes for DRBarrelTubes subdetector to DectDimensions_IDEA_o2_v01.xml.
Increase DRTubesGranularity to 72 rotations aroung the beam pipe (as the endcap).
Make explicit printout from DRTubesconstructor.
Rename DDDRCaloTubes subdetector for consistency with k4geo standards.
Displace the barrel inner radius by 0.5 cm to avoid overlaps between the calo barrel staves and the solenoid.
Materials names defined in DDDRCaloTubes_o1_v01.xml where already defined in the endcap xml file, therefore the barrel geometry was using the endcap materials. This is a problem in case one wants to activate/inactivate optical properties for such materials in the endcap or barrel separately. Fixed by changing names of DDDRCaloTubes materials. Also, set particle direction to (0,1,0) in the IDEA_o2_vo1.py file to shoot into the barrel instead of the endcap.
Set the subdetector ID in DectDimensions_IDEA_o2_v01.xml file and assign.
Add description of the barrel dual-readout-tubes caloriemter in detectors/calorimeter README.md file.
By adding inclusion of the dual-readout-tubes barrel calorimeter in Decemeber 2024.
First attempt to produce signals from the barrel dual-readout-tubes calorimeter.
The theta coordinate was taken from the track momentum direction, instead is should be taken from the G4Step position to discriminate if a G4Step is inside the barrel or the endcap geometry.
Change the name of hit collections to avoid duplication of hits while using ROName in the hit collection name. The barrel was creating also hit collection for cher_right/left and scin_right/left which are only for the endcap.
Reinclude optical properties for fiber materials. Create PMMA_Scin material to differentiate between material used in Scin and Cher fibers in order to not have optical photons in Scin fibers but only in Cher fibers.
Remove printout and apply clang-format. I checked that this version of the custom SD action is consistent with the speedup obtained originally with the endcap geometry only (about 1 s/evt for 10 GeV e- events).
Fix printout to check that volIDs of the DRBarrelTubes are correctly reproduced.
Commet out volID printout check for barrel calo in DRTubesSDAction. This was used to check that the 64-bits volIDs recreated from the barrel copynumbers are identical to the DD4hep ones. However marking barrel fibers as sensitive makes the memory footprint explode (34 Gb before it is killed on lxplus). So the test was done on a single stave.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
BEGINRELEASENOTES
ENDRELEASENOTES
Hello,
this PR includes a new subdetector, the IDEA dual-readout capillary-tube barrel calorimeter. Together with the endcap calorimeter, it completes the hadronic calorimeter of IDEA_o2. The muon system dimensions have been adjusted to avoid overlaps with this subdetector. The only missing part for IDEA_o2 full-sim is now the crystal em calorimeter (#406 ).
DRTubesSDAction
is also expanded to produce hits from the barrel calorimeter.Testing:
Authors:
Geometry by Andreas Loeschcke Centeno, custom sensitive detector action, integration in IDEA_o2 and testing by @lopezzot
FYI @mahmoudali2