Skip to content

Comments

ALICE 3 feat: add IRIS option with inclined walls material #15098

Open
plariono wants to merge 4 commits intoAliceO2Group:devfrom
plariono:feat-iris-incl-walls-mat
Open

ALICE 3 feat: add IRIS option with inclined walls material #15098
plariono wants to merge 4 commits intoAliceO2Group:devfrom
plariono:feat-iris-incl-walls-mat

Conversation

@plariono
Copy link
Contributor

@plariono plariono commented Feb 24, 2026

This PR introduces a new geometry-building option:
createIRISGeometry3InclinedWalls(TGeoVolume* motherVolume);

The new option builds the same full cylindrical IRIS geometry as createIRISGeometryFullCyl, and additionally inserts three inclined wall structures in the barrel region that correspond to the petal design with the inclined walls.

✨ Key Features

  • Adds 3 equally spaced inclined ribs (120° apart in φ).
  • Each rib is composed of two TGeoBBox segments:
    • Segment 1: from tangency near L0 to the inner surface of L1.
    • Segment 2: from the outer surface of L1 to the inner surface of L2.
  • Walls are:
    • Tangentially oriented,
    • Confined strictly to the gaps between cylindrical layers,
    • Constructed with finite thickness (0.4 mm) of Al 5083,
    • Implemented with thickness-aware geometric bounds to avoid overlaps with L0/L1/L2.
  • Uses the same cylindrical shell thickness and Z-extent as the existing full-cylinder geometry.
  • Fully integrated into the existing vacuum cutout / boolean solid logic.

Usage:
--configKeyValues "TRKBase.layoutVD=kIRISFullCyl3InclinedWalls"

Screenshot 2026-02-24 at 15 03 56

@github-actions
Copy link
Contributor

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1
async-2024-PbPb-apass2
async-2023-PbPb-apass5

@njacazio
Copy link
Collaborator

Ciao @plariono many thanks! Does the geometry introduce overlaps?

@plariono
Copy link
Contributor Author

Hi @njacazio, you mean overlaps with the layers? No, it's safe.

@njacazio njacazio enabled auto-merge (squash) February 25, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants