Skip to content

[PWGHF] added new file for Xic correlation#16510

Open
singhra1994 wants to merge 16 commits into
AliceO2Group:masterfrom
singhra1994:master
Open

[PWGHF] added new file for Xic correlation#16510
singhra1994 wants to merge 16 commits into
AliceO2Group:masterfrom
singhra1994:master

Conversation

@singhra1994
Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 2, 2026

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

Please consider the following formatting changes to AliceO2Group#16510
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx
Comment thread PWGHF/HFC/TableProducer/CMakeLists.txt Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
@vkucera vkucera marked this pull request as draft June 2, 2026 16:54
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/Utils/utilsCorrelations.h Outdated
auto nSigmaTPC = o2::aod::pidutils::tpcNSigma(pid, track);

if (tofForced && !track.hasTOF()) {
if ((track.pt() > ptThreshold) && !track.hasTOF()) {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is this needed to be part of this PR?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I just noticed it and also I want maintaining this function, therefore I changed as it was a minor correction in the function, function is doing exactly what is need to be done but in efficient

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

If it is not a part of the changes described by the PR title, it should be in a different PR.

Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx Outdated
Comment thread PWGHF/HFC/TableProducer/correlatorXicHadrons.cxx
@singhra1994 singhra1994 marked this pull request as ready for review June 4, 2026 16:42
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.

3 participants