search

eic / epic

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

Tof new geom #796

Closed ssedd1123 closed 1 month ago

ssedd1123 commented 1 month ago

Briefly, what does this PR introduce?

Endcap TOF geometry modified to match the newest technical drawings from https://indico.bnl.gov/event/24949/attachments/57520/98759/ePICTOF_WP2_FTOFLayout_09272024v2.pdf.

The current epic geometry has each ETOF module located separately. But the newest drawing shows that a single power board house multiple modules together. This PR groups modules together according to the aforementioned drawing.

Furthermore, the current epic geometry shows that sensors are placed facing front and backing alternatively across rows. However, the latest drawing shows that ETOF are double sided on every row, but with sensors placed below the service hybrid on the front side, and above the service hybrid on the other side. Each module now contains 4 small sensors instead of one big sensor.

Here's the updated ETOF power board configuration,

image

Here's the design drawing,

image

Barrel TOF is also modified. Staves are broken into two so an engagement ring can fit in the middle. LGAD is also located on both sides alternatively,

image

Cooling tubes are also incorporated into the staves,

image

Latest technical drawing shows that sensor width is half of what was previously proposed. Sensors right next to the central engagement ring are only half sized,

image

What kind of change does this PR introduce?

Please check if this PR fulfills the following:

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

No

Does this PR change default behavior?

BTOF ETOF geometries are changed.

ssedd1123 commented 1 month ago

@veprbl bench:backwards_ecal exceeded the 2 hours limit and thus eicweb/eic_container failes. I check the logs and the snake jobs are progressing slowly until the very end so it's not like it's getting stuck in a loop. It seems strange to me that my modifications to TOF would have any significant effects on backward ecal. Do you have any idea what's going on?

ssedd1123 commented 1 month ago

@veprbl Unfortunately I don't think this issue is solved. I looked at other jobs in the merge queue and snakemake is still spawning hundreds of jobs on bench:backwards_ecal.

veprbl commented 1 month ago

@veprbl Unfortunately I don't think this issue is solved. I looked at other jobs in the merge queue and snakemake is still spawning hundreds of jobs on bench:backwards_ecal.

Right, back to a drawing board.