-
Notifications
You must be signed in to change notification settings - Fork 614
[Common] mean-pt shape selection tables #13664
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
O2 linter results: ❌ 0 errors, |
|
@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, |
| 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); | ||
| } |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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.
Common/Core/FFitWeights.cxx
Outdated
| // printf("qh not found!\n"); | ||
| // LOGF(info, "FFitWeights qh not found!"); |
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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.
|
@victor-gonzalez Do you have more comments? Otherwise I will ping David :) |
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.