feat: add front support to BarrelImaging Calorimeter

[wdconinc]

Briefly, what does this PR introduce?

Adds the 0.5 cm aluminum front support

What kind of change does this PR introduce?

• Bug fix (issue #__)
• New feature (issue #__)
• Documentation update
• Other: __

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?

No.

[AkshayaVijay]

[wdconinc]

This is interesting. The front support plate is correct, but I am now noticing that we probably have a problem with how the trays are coded up. It seems the side should be flat but it seems the trays have a rectangular cross section instead of a trd.

Marked up a phone screenshot with the straightest line I can draw in freehand...

[AkshayaVijay]

In the barrel_interlayers.xml file, EcalBarrel_AvailThickness is currently defined as EcalBarrelRegion_thickness - EcalBarrel_BackSupportThickness. Not sure if it should be EcalBarrel_BackSupportThickness.

[mariakzurek]

This is interesting. The front support plate is correct, but I am now noticing that we probably have a problem with how the trays are coded up. It seems the side should be flat but it seems the trays have a rectangular cross section instead of a trd.

Marked up a phone screenshot with the straightest line I can draw in freehand...

@wdconinc I think that it was implemented first correctly by Chao, but then the walls and trapezoidal shape were gone. See the issue: #607

[mariakzurek]

In the barrel_interlayers.xml file, EcalBarrel_AvailThickness is currently defined as EcalBarrelRegion_thickness - EcalBarrel_BackSupportThickness. Not sure if it should be EcalBarrel_BackSupportThickness.

It shouldn't. The available thickness calculation should account both for front and back support structure. See for example here:

epic/compact/ecal/barrel_interlayers.xml

Line 84 in d94bd95

value="min(EcalBarrelImagingLayers_nMax,

or here:

epic/compact/ecal/barrel_interlayers.xml

Line 95 in d94bd95

value="max(0, EcalBarrel_AvailThickness-EcalBarrel_ImagingPartThickness)"/>

Also, the overall EcalBarrelRegion_thickness should be increased to have safe margin. Otherwise, when we add extra layers (like for example front aluminum plate) we can artificially shrink the active SciFi/Pb area. You can see here: https://indico.bnl.gov/event/20473/contributions/84620/ that the Calo is now ~ 40 cm in CAD model. So we should make EcalBarrelRegion_thickness at least 40 cm. https://github.com/eic/epic/blob/main/compact/definitions.xml#L547

[mariakzurek]

Please adjust the EcalBarrel_AvailThickness and EcalBarrelRegion_thickness as suggested in the last comment.

[rahmans1]

Reminder: We will be tagging a release in preparation for February campaign soon. When is this expected to be completed @AkshayaVijay ?

[AkshayaVijay]

Please adjust the EcalBarrel_AvailThickness and EcalBarrelRegion_thickness as suggested in the last comment.

Done