Skip to content

Conversation

@joachimckh
Copy link
Contributor

@joachimckh joachimckh commented Nov 3, 2025

Added tables for calculating mean-pT shape selection in light ions. Table contains value between 0-99 reflecting the quantile of the global mean-pT ensemble.

@github-actions
Copy link

github-actions bot commented Nov 3, 2025

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

@joachimckh
Copy link
Contributor Author

@ddobrigk Hi David, can you have a look here when you have time?

I tag also @victor-gonzalez for a review. I put a comment in the top of what is added. I will present in the correlations PAG next week. Hopefully it can be merged, so I can use the tables in the ese analysis and I can run it on some more data before then!

Cheers,
Joachim

@joachimckh joachimckh marked this pull request as ready for review November 3, 2025 19:25
Comment on lines 106 to 120
if (first) {
auto tp = reinterpret_cast<TProfile*>(tar->FindObject("pMeanPt"));
if (!tp) {
tar->Add(new TProfile("pMeanPt", "", centBin, 0, centBin));
tp = reinterpret_cast<TProfile*>(tar->At(tar->GetEntries() - 1));
}
tp->Fill(centrality, pt);
} else {
auto th2 = reinterpret_cast<TH2D*>(tar->FindObject("hPtWeight"));
if (!th2) {
tar->Add(new TH2D("hPtWeight", "", centBin, 0, centBin, ptBin, ptAxis->GetXmin(), ptAxis->GetXmax()));
th2 = reinterpret_cast<TH2D*>(tar->At(tar->GetEntries() - 1));
}
th2->Fill(centrality, pt);
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it actually needed to have a FindObject which involves string comparison (char by char) on a per track level?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @victor-gonzalez,
I have created pointers for this, and only assign them once for the pT objects as they remain static.

Comment on lines 211 to 212
// printf("qh not found!\n");
// LOGF(info, "FFitWeights qh not found!");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Don't leave code commented

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have removed the printf and added LOGF and generally removed outcommented lines in the code.

@joachimckh
Copy link
Contributor Author

@victor-gonzalez Do you have more comments? Otherwise I will ping David :)
Cheers

@ddobrigk ddobrigk merged commit 4412080 into AliceO2Group:master Nov 14, 2025
14 of 15 checks passed
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants