Skip to content
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

Adjust Barrel ECal geometry according to the geometry database #606

Merged
merged 5 commits into from
Dec 9, 2023

Conversation

mariakzurek
Copy link
Contributor

@mariakzurek mariakzurek commented Dec 4, 2023

Briefly, what does this PR introduce?

This PR related to the issue #552, and implements small changes in the Barrel ECal geometry to make it more realistic. The following changes has been implemented:

  • SciFi/Pb z_min and z_max (and accordingly the full length) has been implemented according to https://eic.jlab.org/Geometry/Detector/Detector-20231031150001.html.
  • The total depth (envelope) of the calorimeter has been increased to 38 cm (from fixed 35 cm). 38 cm envelope is the correct one and fits both the SciFi and the 3 cm back Al plate.
  • In every AstroPix stave a carbon fiber slice has been added to mimic a bit better the material of the staves

What kind of change does this PR introduce?

Please check if this PR fulfills the following:

  • Tests for the changes have been added
  • Documentation has been added / updated
  • Changes have been communicated to collaborators

Does this PR introduce breaking changes? What changes might users need to make to their code?

No

Does this PR change default behavior?

Calorimeter (SciFi/Pb matrix) length, offset, and z_min z_max are in agreement with the geometry database:

Output of npdet_info dump $DETECTOR_PATH/epic_craterlake.xml | grep EcalBarrel

EcalBarrel_AvailThickness      =       35.000 = EcalBarrelRegion_thickness - EcalBarrel_Support_thickness
EcalBarrel_Calorimeter_length  =      440.000 = EcalBarrel_Calorimeter_zmax + EcalBarrel_Calorimeter_zmin
EcalBarrel_Calorimeter_offset  =      -38.750 = (EcalBarrel_Calorimeter_zmax - EcalBarrel_Calorimeter_zmin)/2.0
EcalBarrel_Calorimeter_zmax    =      181.250 = min(181.25*cm, EcalBarrelForward_zmax)
EcalBarrel_Calorimeter_zmin    =      258.750 = min(258.75*cm, EcalBarrelBackward_zmax)

Now calorimeter has proper depth (together with back plate)
SciFi part of the Calo. 12th layer has now proper width.
Zoom on AstroPix part of the Calo

Not included in this PR because of the problem with overlaps

Below you can see the Carbon Fiber slice added to each stave.
AstroPix part of the sector
Zoom on one layer with modules

@mariakzurek
Copy link
Contributor Author

mariakzurek commented Dec 4, 2023

There is an obvious problem with an overlap of modules of thickness EcalBarrel_AstroPix_thickness and the slice of thickness EcalBarrel_CarbonThickness placed inside the stave envelope of thickness EcalBarrel_AstroPix_thickness + EcalBarrel_CarbonThickness. This probably is related to wrong placement in BarrelCalorimeterImaging_geo.cpp. I decided to have the proper geometry included as a highest priority in this PR, I reversed the changes related to including the carbon fiber slice in the stave.

@wdconinc wdconinc added this pull request to the merge queue Dec 8, 2023
Merged via the queue into main with commit 0bd4549 Dec 9, 2023
85 checks passed
@wdconinc wdconinc deleted the becal-geometry-fix branch December 9, 2023 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants