FT3 Tiling: Stave Geometry - #15339

Merged

njacazio merged 31 commits into

AliceO2Group:devfrom

JustusRudolph:ft3-tiling

May 4, 2026

Commits on Apr 28, 2026

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
Please consider the following formatting changes
alibuild
committed

Commits on Apr 29, 2026

Merge pull request #1 from alibuild/alibot-cleanup-15339
JustusRudolph
authored

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

FT3 Tiling: Stave Geometry - #15339

FT3 Tiling: Stave Geometry#15339
njacazio merged 31 commits into
AliceO2Group: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

Please consider the following formatting changes

Commits on Apr 29, 2026

Merge pull request #1 from alibuild/alibot-cleanup-15339