Skip to content

Conversation

@YazhenLin
Copy link
Contributor

Add event selections, V0 selections, cascade selections for Pb-Pb collisions in PWGLF/Tasks/Strangeness/hStrangeCorrelation.cxx.

@github-actions
Copy link

github-actions bot commented May 7, 2025

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

if (posdau.tpcNClsCrossedRows() < minTPCNCrossedRows)
continue;

if (std::abs(posdau.tpcNSigmaPi()) < strangedEdxNSigmaLoose && std::abs(negdau.tpcNSigmaPi()) < strangedEdxNSigmaLoose)
Copy link
Contributor

Choose a reason for hiding this comment

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

Please use { } for the code inside of the condition. Also in all if cases below

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I've fixed it now. 😊

}
}
}

Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove this empty line

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, I've fixed it now. 😊


Configurable<int> triggerBinToSelect{"triggerBinToSelect", 0, "trigger bin to select on if processSelectEventWithTrigger enabled"};
Configurable<int> triggerParticleCharge{"triggerParticleCharge", 0, "For checks, if 0 all charged tracks, if -1 only neg., if 1 only positive"};

Copy link
Contributor

Choose a reason for hiding this comment

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

Please remove empty line

@romainschotter romainschotter enabled auto-merge (squash) May 8, 2025 09:30
@romainschotter romainschotter merged commit 6466b09 into AliceO2Group:master May 8, 2025
11 of 13 checks passed
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request May 11, 2025
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request May 11, 2025
louisemillot pushed a commit to louisemillot/O2Physics that referenced this pull request May 14, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request May 17, 2025
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 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