Skip to content

Conversation

@singhra1994
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 14, 2025

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@github-actions github-actions bot changed the title Added PID histograms for V0 daughter and fixed bug for selection of V0 mass window [PWGHF] Added PID histograms for V0 daughter and fixed bug for selection of V0 mass window Nov 14, 2025
Copy link
Collaborator

@zhangbiao-phy zhangbiao-phy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me!

@zhangbiao-phy zhangbiao-phy enabled auto-merge (squash) November 14, 2025 15:58
@zhangbiao-phy zhangbiao-phy enabled auto-merge (squash) November 14, 2025 16:06
@ddobrigk
Copy link
Collaborator

ddobrigk commented Nov 14, 2025

Hi @singhra1994, sorry to jump in, but I wanted to warn you that using primary particle TOF (the standard TOF) for strange decay daughters may incur in a significant signal loss (up to 30-50% in proton-proton collisions). Further, this signal loss is not well reproduced by MC, which could lead to problems in case MC corrections are needed. The key problem is that primary TOF assumes the track-to-collision association stored in AO2Ds to be correct, while secondary decay daughters are often assigned to a collision that isn't the same as the V0 or Cascade they make up. To avoid this kind of issue, one should either not apply TOF at all or take these effects into account using the strangeness TOF: see this talk in the strangeness PAG, with the strangeness TOF also being mentioned in the tutorial here. A central core service wagon that provides strangeness TOF PID is also provided in hyperloop. Hope this helps!

@vkucera vkucera disabled auto-merge November 14, 2025 20:10
@singhra1994
Copy link
Contributor Author

Hi @singhra1994, sorry to jump in, but I wanted to warn you that using primary particle TOF (the standard TOF) for strange decay daughters may incur in a significant signal loss (up to 30-50% in proton-proton collisions). Further, this signal loss is not well reproduced by MC, which could lead to problems in case MC corrections are needed. The key problem is that primary TOF assumes the track-to-collision association stored in AO2Ds to be correct, while secondary decay daughters are often assigned to a collision that isn't the same as the V0 or Cascade they make up. To avoid this kind of issue, one should either not apply TOF at all or take these effects into account using the strangeness TOF: see this talk in the strangeness PAG, with the strangeness TOF also being mentioned in the tutorial here. A central core service wagon that provides strangeness TOF PID is also provided in hyperloop. Hope this helps!

Thanks a lot @ddobrigk for mentioning this, indeed we were having this issue. I wrote this code to calculate data driven efficiency of proton for PID cuts using V0_lambda. I will certainly look to the material you have provided and will come back to you in case of doubt.

@ddobrigk
Copy link
Collaborator

Hi @singhra1994, sorry to jump in, but I wanted to warn you that using primary particle TOF (the standard TOF) for strange decay daughters may incur in a significant signal loss (up to 30-50% in proton-proton collisions). Further, this signal loss is not well reproduced by MC, which could lead to problems in case MC corrections are needed. The key problem is that primary TOF assumes the track-to-collision association stored in AO2Ds to be correct, while secondary decay daughters are often assigned to a collision that isn't the same as the V0 or Cascade they make up. To avoid this kind of issue, one should either not apply TOF at all or take these effects into account using the strangeness TOF: see this talk in the strangeness PAG, with the strangeness TOF also being mentioned in the tutorial here. A central core service wagon that provides strangeness TOF PID is also provided in hyperloop. Hope this helps!

Thanks a lot @ddobrigk for mentioning this, indeed we were having this issue. I wrote this code to calculate data driven efficiency of proton for PID cuts using V0_lambda. I will certainly look to the material you have provided and will come back to you in case of doubt.

Just in case: From what we've seen in the strangeness TOF PID testing, in primary TOF QA plots, you might see two effects:

  • there will be a very slight offset from N-sigma = 0 (due to deep secondary tracks effectively traveling part of the distance to the TOF as a heavier particle). This should not be enough to explain a large signal loss if you apply a loose Nsigma cut.
  • there will be a significant population that won't be visible in an Nsigma plot, because they're displaced in time by multiples of 25000 picoseconds (BC start time), equivalent to many hundreds of sigma. You will only see that population if you check explicitly the under- and overflow bins of the QA plots.

In summary: the actual histogram of Nsigma might look like a gaussian, but the problem is in wildly large Nsigma values (still well-defined) and those will end up in under/overflow...

@singhra1994
Copy link
Contributor Author

Hi @singhra1994, sorry to jump in, but I wanted to warn you that using primary particle TOF (the standard TOF) for strange decay daughters may incur in a significant signal loss (up to 30-50% in proton-proton collisions). Further, this signal loss is not well reproduced by MC, which could lead to problems in case MC corrections are needed. The key problem is that primary TOF assumes the track-to-collision association stored in AO2Ds to be correct, while secondary decay daughters are often assigned to a collision that isn't the same as the V0 or Cascade they make up. To avoid this kind of issue, one should either not apply TOF at all or take these effects into account using the strangeness TOF: see this talk in the strangeness PAG, with the strangeness TOF also being mentioned in the tutorial here. A central core service wagon that provides strangeness TOF PID is also provided in hyperloop. Hope this helps!

Thanks a lot @ddobrigk for mentioning this, indeed we were having this issue. I wrote this code to calculate data driven efficiency of proton for PID cuts using V0_lambda. I will certainly look to the material you have provided and will come back to you in case of doubt.

Just in case: From what we've seen in the strangeness TOF PID testing, in primary TOF QA plots, you might see two effects:

* there will be a very slight offset from N-sigma = 0 (due to deep secondary tracks effectively traveling part of the distance to the TOF as a heavier particle). This should not be enough to explain a large signal loss if you apply a loose Nsigma cut.

* there will be a significant population that won't be visible in an Nsigma plot, because they're displaced in time by multiples of 25000 picoseconds (BC start time), equivalent to many hundreds of sigma. You will only see that population if you check explicitly the under- and overflow bins of the QA plots.

In summary: the actual histogram of Nsigma might look like a gaussian, but the problem is in wildly large Nsigma values (still well-defined) and those will end up in under/overflow...

Thanks again @ddobrigk, We (me and @arossi81 ) were exactly discussing this issue, it is useful for us

@vkucera
Copy link
Collaborator

vkucera commented Nov 18, 2025

@zhangbiao-phy I disabled the auto-merge just to leave room for potential modifications resulting from the discussion. If the points have been resolved, feel free to proceed with the merging.

@singhra1994
Copy link
Contributor Author

@zhangbiao-phy I disabled the auto-merge just to leave room for potential modifications resulting from the discussion. If the points have been resolved, feel free to proceed with the merging.

@vkucera @zhangbiao-phy, current version of strangeness TOF pid is not useful for our purpose. There is no harm to merge this PR, later if need we can add strangeness PID

@zhangbiao-phy
Copy link
Collaborator

@zhangbiao-phy I disabled the auto-merge just to leave room for potential modifications resulting from the discussion. If the points have been resolved, feel free to proceed with the merging.

@vkucera @zhangbiao-phy, current version of strangeness TOF pid is not useful for our purpose. There is no harm to merge this PR, later if need we can add strangeness PID

make sense! Thanks! Looks good to me!

@zhangbiao-phy zhangbiao-phy merged commit 50409a3 into AliceO2Group:master Nov 19, 2025
16 checks passed
yakparo pushed a commit to yakparo/O2Physics that referenced this pull request Nov 21, 2025
…ion of V0 mass window (AliceO2Group#13835)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
…ion of V0 mass window (AliceO2Group#13835)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pwghf PWG-HF

Development

Successfully merging this pull request may close these issues.

5 participants