Skip to content

ALICE3-TRK: adapt ordering key for digits to the large number of columns in the VD#15070

Open
atriolo wants to merge 1 commit intoAliceO2Group:devfrom
atriolo:prevIRIS
Open

ALICE3-TRK: adapt ordering key for digits to the large number of columns in the VD#15070
atriolo wants to merge 1 commit intoAliceO2Group:devfrom
atriolo:prevIRIS

Conversation

@atriolo
Copy link
Contributor

@atriolo atriolo commented Feb 16, 2026

In the VD case, the sensor covers the full z acceptance of the layer, from z=-25 to z=25 cm, for a total of 50000 columns for a single sensor.
When the number of columns exceeded 32768 = 2^15, the function getOrderingKey(...), which uniquely assigns an ID to each digit, was not working properly anymore.

Issue explained here.

The issue is solved with this commit by casting the column variable to 64-bit before the bit-to-bit shift operated in the function.
This modification is applied only to the TRK class.

@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

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.

1 participant