Skip to content

ALICE3-TRK: fix y-axis orientation in the sensor local coordinate system, keeping the geometry unchanged#15134

Merged
njacazio merged 1 commit intoAliceO2Group:devfrom
atriolo:fixDigi
Mar 11, 2026
Merged

ALICE3-TRK: fix y-axis orientation in the sensor local coordinate system, keeping the geometry unchanged#15134
njacazio merged 1 commit intoAliceO2Group:devfrom
atriolo:fixDigi

Conversation

@atriolo
Copy link
Contributor

@atriolo atriolo commented Mar 9, 2026

This change modifies the method with which sensor and metal subvolumes are positioned within the ML/OT chips.

  • Before this change, their relative orientation was set with a rotation of the stave after their creation and positioning inside the module and stave.
  • Now, the rotation introduced with PR#15043 has been reversed, and a shift has been applied to the elements to adjust their relative orientation in space.
    This change does not alter the geometry, but ensures that the orientation of the chip's local reference system has the y-axis oriented toward the interaction point. This is the convention used in the digitization and reconstruction processes.
    Everything else remains unchanged.

@atriolo atriolo requested review from mconcas and njacazio as code owners March 9, 2026 11:02
@github-actions
Copy link
Contributor

github-actions bot commented Mar 9, 2026

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

@atriolo
Copy link
Contributor Author

atriolo commented Mar 10, 2026

Ciao @njacazio, could you please review also this PR? Thanks a lot!

@njacazio
Copy link
Collaborator

Ciao @atriolo I thought I did already, apologies!

@njacazio njacazio enabled auto-merge (squash) March 10, 2026 15:00
@njacazio njacazio merged commit 79f51af into AliceO2Group:dev Mar 11, 2026
13 checks passed
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.

2 participants