Skip to content

Conversation

@abmodak
Copy link
Collaborator

@abmodak abmodak commented Nov 13, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 13, 2025

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

@github-actions github-actions bot changed the title Add updated track efficiency estimation [PWGLF] Add updated track efficiency estimation Nov 13, 2025
@abmodak abmodak enabled auto-merge (squash) November 13, 2025 20:36
Copy link
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @abmodak.

@abmodak abmodak merged commit 1592596 into AliceO2Group:master Nov 13, 2025
11 of 12 checks passed
@abmodak abmodak deleted the update-heavy-ion branch November 13, 2025 20:37
minjungkim12 added a commit to minjungkim12/O2Physics that referenced this pull request Nov 16, 2025
auto mcpart = Rectrack.mcParticle();
histos.fill(HIST("etaResolution"), Rectrack.eta(), Rectrack.eta() - mcpart.eta());
if (mcpart.isPhysicalPrimary()) {
pid = kRecoPrimary;
Copy link
Collaborator

Choose a reason for hiding this comment

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

This value is never used.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hi @vkucera, sorry, I do not understand your comment.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Hi @abmodak , the value kRecoPrimary assigned in the variable pid is never used, therefore the assignment is useless.

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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants