[PWGHF] add THnSparse with the charm-bulk correlation for the v0 pt in taskPtFlucCharmHadrons#15941
Conversation
wuctlby
requested review from
NicoleBastid,
alibuild,
apalasciano,
deepathoms,
fcatalan92,
fcolamar,
fgrosa,
gluparel,
hahassan7,
jpxrk,
mfaggin,
mmazzilli,
singhra1994,
stefanopolitano,
vkucera and
zhangbiao-phy
as code owners
|
O2 linter results: ❌ 0 errors, |
github-actions
Bot
changed the title
Please consider the following formatting changes to AliceO2Group#15941
|
Hi @wuctlby ! Thanks for the development. This is very interesting, however it was already implemented in another task (https://github.com/AliceO2Group/O2Physics/blob/master/PWGHF/D2H/Tasks/taskPtFlucCharmHadrons.cxx). (Tagging @prottayCMT who is the main analyser) |
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Please consider the following formatting changes to AliceO2Group#15941
|
Hi @stefanopolitano and @prottayCMT , |
Please consider the following formatting changes to AliceO2Group#15941
|
Please do not mark my comments as resolved. It is explicitly mentioned in the guidelines. |
Co-authored-by: Copilot <copilot@github.com>
|
Co-authored-by: Copilot <copilot@github.com>
| std::array<float, 2> daugPts = {cand.ptProng0(), cand.ptProng1()}; | ||
| for (int iProng = 0; iProng < 2; ++iProng) { // o2-linter: disable=magic-number (for 2-prong) | ||
| if (std::binary_search(trkIDs.begin(), trkIDs.end(), daugIDs[iProng])) { | ||
| meanPt = (rawMeanPt * n - daugPts[iProng]) / (n - 1); |
There was a problem hiding this comment.
Hi @wuctlby ! Thanks for the devs!. I have a doubt on this point. It is good to remove the daughter tracks to avoid autocorrelations, but in this case you overwrite the meanPt at each iteration, this means that the final meanPt values account only for the last daughter in the loop. Am I missing something?
There was a problem hiding this comment.
Hi @stefanopolitano ! Thanks for the comment. No, you are right. I mixed up the algorithm of removing daughter tracks after several modifications. I was starting from the D0, so it's hard to have a D0 whose 2 daughter particles both come from the opposite subevent after eta gap is applied. I fixed this issue with the new commits. Please let me know if you have any other comments.
Please consider the following formatting changes to AliceO2Group#15941
|
@wuctlby The PR title does not say anything about what you did. |
wuctlby
changed the title
6 participants
Dear all,
This PR is about the analysis task of charm-bulk correlation, probing the radio-flow fluctuation, the underlying radial expansion, and the medium’s properties. And this task will be iterated in the near future.
Please let me know if you have any comments or suggestions!
Thanks a lot!
Also tagging @arossi81 and @mfaggin .