FT3 Tiling: Stave Geometry by JustusRudolph · Pull Request #15339 · AliceO2Group/AliceO2

Add separate file for constants in FT3 module creation

JustusRudolph

committed

Create modular design in similar fashion as before, but with slightly different module placement. Rewrite structure to be more granular and extendable. The old code remains for backward compatibili…

JustusRudolph

committed

Give new geometry creation an FT3Layout enum and change Layer&Module accordingly. Also simplify create_layout_scopingV3 function since the layout_type isn't needed.

JustusRudolph

committed

Add functionality for the user to enforce a strict cut on module placement to remain within nominal radii. Also remove initial placement of sensor stack before calling fill_stave, moving to using y…

JustusRudolph

committed

Make forward and backward directions mirrored, front and back faces swap based on direction

JustusRudolph

committed

Add loop over all sensors in stack so all sensors are added to volume, not just the first in the stack

JustusRudolph

committed

Add constants for hollow stave based geometry

JustusRudolph

committed

Add carbon fiber material, functionality to toggle between stave and old geometry. Also rename material counts to 'volume' instead of 'sensor'. EDIT: Change TGeo layer thickness back to original. E…

JustusRudolph

committed

Add full hollow stave geometry functionality

JustusRudolph

committed

Make Stave layout the standard and remove old kSegmentedMarch26: EDIT 20/04/26: Revert to newly added middle disc layer segmentation.

JustusRudolph

committed

Remove all instances of old slab geometry and work only with staves.

JustusRudolph

committed

Remove all log(info) statements, except one which is changed to debug.

JustusRudolph

committed

Don't create the separation layer for stave layout, since we already get the structural support (in carbon) from the staves

JustusRudolph

committed

add bools for cutting staves and sensors on staves on nominal radii

JustusRudolph

committed

merge two if statements with the same if

JustusRudolph

committed

Add implementations for cutting staves and sensors on nominal radii. Also change kSensorsPerStack to a vector in which order of sensor stack height we will pad the staves.

JustusRudolph

committed

Fix bug in default y range for staves & sensor placements

JustusRudolph

committed

Bugfix: Stack correctly by using previous stack height in fill_stave, and move in correct direction when placing sensors later

JustusRudolph

committed

Remove now stale info statements. TODO: let staves be cut as well on nominal radii

JustusRudolph

committed

Add splitting of stave in case of strict inner cutoff

JustusRudolph

committed

Add support to place either a stack gap or single sensor around y=0 in staves that have full +Rout to -Rout coverage

JustusRudolph

committed

Bugfix: When starting sensor placement around the x-axis you can have a different number of sensor stacks on the positive/negative side of the stave leading to garbage memory access. Fixed now.

JustusRudolph

committed

Add option to draw reference circles onto the layer -- strictly for visualisation purposes

JustusRudolph

committed

remove stale info statements

JustusRudolph

committed

Remove stale overlap argument, and add local offset in z. In contrast to previous layout, to encapsulate the staves with face at local z=0 in air, we need to shift the staves and sensors locally si…

JustusRudolph

committed

Fix placement issue to get sensor materials inside the volume. This now means that the stave faces are not at local z=0 but instead at z=+-totalSensorMaterialThickness+0.1. Also fix global position…

JustusRudolph

committed

Add OT only segmentation and change defaults

JustusRudolph

committed

Make stave geometry available with middle layer disks as well. Currently use simple calculated values for stave placements, these are subject to change. Hence the existence of kSegmentedOTOnly

JustusRudolph

committed

Change default to stave segmentation for outer disks only

JustusRudolph

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

FT3 Tiling: Stave Geometry#15339

FT3 Tiling: Stave Geometry#15339
JustusRudolph wants to merge 29 commits intoAliceO2Group:devfrom
JustusRudolph:ft3-tiling

Commits on Apr 28, 2026

Add separate file for constants in FT3 module creation

Create modular design in similar fashion as before, but with slightly different module placement. Rewrite structure to be more granular and extendable. The old code remains for backward compatibili…

Give new geometry creation an FT3Layout enum and change Layer&Module accordingly. Also simplify create_layout_scopingV3 function since the layout_type isn't needed.

Add functionality for the user to enforce a strict cut on module placement to remain within nominal radii. Also remove initial placement of sensor stack before calling fill_stave, moving to using y…

Make forward and backward directions mirrored, front and back faces swap based on direction

Add loop over all sensors in stack so all sensors are added to volume, not just the first in the stack

Add constants for hollow stave based geometry

Add carbon fiber material, functionality to toggle between stave and old geometry. Also rename material counts to 'volume' instead of 'sensor'. EDIT: Change TGeo layer thickness back to original. E…

Add full hollow stave geometry functionality

Make Stave layout the standard and remove old kSegmentedMarch26: EDIT 20/04/26: Revert to newly added middle disc layer segmentation.

Remove all instances of old slab geometry and work only with staves.

Remove all log(info) statements, except one which is changed to debug.

Don't create the separation layer for stave layout, since we already get the structural support (in carbon) from the staves

add bools for cutting staves and sensors on staves on nominal radii

merge two if statements with the same if

Add implementations for cutting staves and sensors on nominal radii. Also change kSensorsPerStack to a vector in which order of sensor stack height we will pad the staves.

Fix bug in default y range for staves & sensor placements

Bugfix: Stack correctly by using previous stack height in fill_stave, and move in correct direction when placing sensors later

Remove now stale info statements. TODO: let staves be cut as well on nominal radii

Add splitting of stave in case of strict inner cutoff

Add support to place either a stack gap or single sensor around y=0 in staves that have full +Rout to -Rout coverage

Bugfix: When starting sensor placement around the x-axis you can have a different number of sensor stacks on the positive/negative side of the stave leading to garbage memory access. Fixed now.

Add option to draw reference circles onto the layer -- strictly for visualisation purposes

remove stale info statements

Remove stale overlap argument, and add local offset in z. In contrast to previous layout, to encapsulate the staves with face at local z=0 in air, we need to shift the staves and sensors locally si…

Fix placement issue to get sensor materials inside the volume. This now means that the stave faces are not at local z=0 but instead at z=+-totalSensorMaterialThickness+0.1. Also fix global position…

Add OT only segmentation and change defaults

Make stave geometry available with middle layer disks as well. Currently use simple calculated values for stave placements, these are subject to change. Hence the existence of kSegmentedOTOnly

Change default to stave segmentation for outer disks only