eic / epic

DD4hep Geometry Description of the ePIC Experiment
https://eic.github.io/epic
GNU Lesser General Public License v3.0
23 stars 41 forks source link

Geometry Matrix Changes #730

Open Chao1009 opened 1 month ago

Chao1009 commented 1 month ago

There are some changes in the latest geometry matrix, which affects many detector subsystems.

Detector subsystem liaisons (#64) need to confirm the changes are within expectation and ensure they are implemented in the simulation.

A summary of the changes can be found at: Diff between 04/26/2024 and [01/17/2024] https://docs.google.com/spreadsheets/d/1WEGXm7pbVmUkJ_gGimLgE9as6nY1O2QEwOB8Z6GEU1E/edit#gid=1829269715 Affected detector subsystems:

        [('CENTRAL DETECTOR', 'Barrel EMCal', '')
         ('CENTRAL DETECTOR', 'Barrel EMCal', 'HD Readout Electronics')
         ('CENTRAL DETECTOR', 'Barrel EMCal', 'Imaging Part')
         ('CENTRAL DETECTOR', 'Barrel EMCal', 'LD Readout Electronics')
         ('CENTRAL DETECTOR', 'Barrel Time of Flight/Tracker', '')
         ('CENTRAL DETECTOR', 'Dual RICH', '')
         ('CENTRAL DETECTOR', 'Dual RICH', 'Aerogel Section')
         ('CENTRAL DETECTOR', 'Dual RICH', 'Detector Section')
         ('CENTRAL DETECTOR', 'HD MPGD 1', '')
         ('CENTRAL DETECTOR', 'HD MPGD 2', '')
         ('CENTRAL DETECTOR', 'HD Time of Flight/Tracker', '')
         ('CENTRAL DETECTOR', 'Inner Tracker (Micromegas)', '')
         ('CENTRAL DETECTOR', 'Inner Tracker (Micromegas)', 'Micromega 1')
         ('CENTRAL DETECTOR', 'Inner Tracker (Micromegas)', 'Micromega 2')
         ('CENTRAL DETECTOR', 'Inner Tracker (Micromegas)', 'Micromega 3')
         ('CENTRAL DETECTOR', 'Inner Tracker (Micromegas)', 'Micromega 4')
         ('CENTRAL DETECTOR', 'Inner Tracker (Si Disks)', '')
         ('CENTRAL DETECTOR', 'LD EMCal', '')
         ('CENTRAL DETECTOR', 'Proximity Focusing RICH', '')
         ('HADRON DIRECTION END CAP', 'Electromagnetic Calorimeter', '')
         ('HADRON DIRECTION END CAP', 'Hadron Calorimeter', '')]

Diff between 05/15/2024 and 04/26/2024 https://docs.google.com/spreadsheets/d/15ir32va9rCkXhu0a9mtf--FQuamc1B2LutBr8hxjYAc/edit#gid=1550196875 Affected detector subsystems:

        [('CENTRAL DETECTOR', 'HD Time of Flight/Tracker', '')
         ('CENTRAL DETECTOR', 'Inner Tracker (Si Disks)', '')
         ('CENTRAL DETECTOR', 'Proximity Focusing RICH', '')]
Chao1009 commented 1 month ago

Related PRs: Barrel EMCal: #729

Chao1009 commented 1 month ago

@mariakzurek @DrKKwd @ssedd1123 @chchatte92 @mposik1983 @nschmidtALICE @ShujieL @johnny8266 @kkauder @jizhongling @FriederikeBock @rymilton Could you please review the latest geometry table and either confirm that no changes are needed for the simulation or open an issue if any detector parameters need to be updated?

ruse-traveler commented 1 month ago

Confirming: BHCal does not need any change.

chchatte92 commented 1 month ago

For the dRICH in the table the aerogel cone is extended up to 33 cm. However, in the attached image is still pointing that the aerogel cone should be around 25 cm. This 25 cm was chosen in order not to shadow the photons in the wall of the aerogel cone for high rapidity particles. Why do we have to move this aerogel cone depth down to 33 cm? The dRICH start has already been moved to 198 cm.

DevilsZ commented 1 month ago

For the Barrel ToF, I don't see any changes in the recent matrix from the previous version.

ShujieL commented 1 month ago

The Si disk volume changes from 0.75 to 0.8, but I don't think this parameter is used explicitly anywhere in simulation.

mariakzurek commented 1 month ago

729 addresses the changes in the recent Barrel ECal geometry. So things are in order now. The Detector Envelopes Matrix requires an update (it looks like it is using a very old parameters for some of the values, e.g., depth of the BIC), and we will take care of that.