Skip to content

[PWGJE] Adding Matched MC function + dPhi fixes#15551

Merged
nzardosh merged 13 commits intoAliceO2Group:masterfrom
MattOckleton:master
Mar 31, 2026
Merged

[PWGJE] Adding Matched MC function + dPhi fixes#15551
nzardosh merged 13 commits intoAliceO2Group:masterfrom
MattOckleton:master

Conversation

@MattOckleton
Copy link
Copy Markdown
Contributor

Added processMCMatched function, and general fixes to the task.

@github-actions github-actions bot added the pwgje label Mar 27, 2026
@github-actions github-actions bot changed the title Adding Matched MC function + dPhi fixes [PWGJE] Adding Matched MC function + dPhi fixes Mar 27, 2026
@MattOckleton MattOckleton marked this pull request as ready for review March 27, 2026 22:22
if (McDJet.has_matchedJetGeo()) { // geometric matching
for (auto const& McPJet : McDJet.template matchedJetGeo_as<aod::ChargedMCParticleLevelJets>()) {
float dPhiP = RecoDecay::constrainAngle(McPJet.phi() - d0Particle.phi());
if (dPhiP > o2::constants::math::PI) {
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.

i think constrain angle should already take care of the cyclical nature?

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.

Fixed

@nzardosh nzardosh enabled auto-merge (squash) March 31, 2026 11:07
@nzardosh nzardosh merged commit 1ea9817 into AliceO2Group:master Mar 31, 2026
13 of 14 checks passed
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.

2 participants