Skip to content

[PWGJE] Fix leading-jet selection (Rec) and event normalization - #18050

Open
arvindkhuntia wants to merge 3 commits into
AliceO2Group:masterfrom
arvindkhuntia:master
Open

arvindkhuntia wants to merge 3 commits into
AliceO2Group:masterfrom
arvindkhuntia:master

Conversation

@arvindkhuntia

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

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

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 778a227 at 2026-09-24 21:55:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 778a227 at 2026-09-24 21:56:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 778a227 at 2026-09-24 21:57:

No log files found

Full log here.

Comment thread PWGJE/Tasks/nucleiInJets.cxx Outdated
float leadingDetJetPt = -1.f;
if (isWithLeadingJet) {
for (const auto& mcdjet : mcdjets) {
if (isConeAxisAccepted(mcdjet.eta()) && mcdjet.pt() > leadingDetJetPt) {

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.

instead of mcdjet.pt() shouldnt you be checking the jet pT after background subtraction here? you can use a conditional statement like this:

const float mcdJetPtForResponseSel = usebkgSubractionMC ? mcdjet.pt() - backgroundRho * jetArea : mcdjet.pt();
if (isConeAxisAccepted(mcdjet.eta()) && mcdJetPtForResponseSel > leadingDetJetPt) {
leadingDetJetPt = mcdJetPtForResponseSel;
leadingDetJetId = mcdjet.globalIndex();
}

@arvindkhuntia arvindkhuntia Sep 25, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Many thanks, @nzardosh, for the suggestion. The background-subtracted p_T cut is already applied later in the matching loop. I’ve now also used the corrected p_T to select the leading detector jet and added an early rejection of \rho < 0 when subtraction is enabled, avoiding unnecessary further processing for those events. Have a nice day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

4 participants