Skip to content

Fix ITS seeding for B=0#15157

Merged
shahor02 merged 1 commit intoAliceO2Group:devfrom
shahor02:pr_itsb0
Mar 13, 2026
Merged

Fix ITS seeding for B=0#15157
shahor02 merged 1 commit intoAliceO2Group:devfrom
shahor02:pr_itsb0

Conversation

@shahor02
Copy link
Collaborator

The convention is to assign to q/pT = 1/mostProbablePt(=0.6) in case of B=0

Due to the sign inversion the ITS/TPC matching for B=0 was failing.

image

The convention is to assign to q/pT = 1/mostProbablePt(=0.6) in case of B=0
@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

@shahor02 shahor02 requested a review from f3sch March 13, 2026 14:30
@f3sch
Copy link
Collaborator

f3sch commented Mar 13, 2026

@shahor02 thanks, this was broken then for whole while already since #14370 for B=0

@shahor02
Copy link
Collaborator Author

@shahor02 thanks, this was broken then for whole while already since #14370 for B=0

Why? https://github.com/AliceO2Group/AliceO2/pull/14370/changes#diff-eae8fda90330056fb3951aed0952fec5a6370dbec65f6164ebbb94636e3a36ccR1249 looks correct.

@f3sch
Copy link
Collaborator

f3sch commented Mar 13, 2026

@shahor02 thanks, this was broken then for whole while already since #14370 for B=0

Why? https://github.com/AliceO2Group/AliceO2/pull/14370/changes#diff-eae8fda90330056fb3951aed0952fec5a6370dbec65f6164ebbb94636e3a36ccR1249 looks correct.

Yeah, sorry, meant in the subsequent change it broke (the linked one was the last correct one)

@shahor02 shahor02 merged commit ddce6a5 into AliceO2Group:dev Mar 13, 2026
12 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