Skip to content

[PWGCF] Add MFT-MFT correlation task - #18485

Open
mintu1999 wants to merge 8 commits into
AliceO2Group:masterfrom
mintu1999:mftcorrelation
Open

mintu1999 wants to merge 8 commits into
AliceO2Group:masterfrom
mintu1999:mftcorrelation

Conversation

@mintu1999

Copy link
Copy Markdown

Summary

This PR adds a new task, mftcorrelations, to PWGCF/TwoParticleCorrelations/Tasks. It studies two-particle azimuthal correlations built from MFT tracks. It constructs the same-event and mixed-event distributions, using CorrelationContainer for the correlation histograms.

Changes

  • New task: PWGCF/TwoParticleCorrelations/Tasks/mftcorrelations.cxx, registered in CMakeLists.txt as mft-correlations.
  • Core change: PWGCF/Core/CorrelationContainer.cxx now has an extra efficiency axis, used to correct the φ distribution of MFT tracks.

Testing

  • Data: pp, LHC24ap_apass1
  • MC: LHC25b4a6
  • clang-format applied and the O2-linter passes locally.

@github-actions

Copy link
Copy Markdown

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

mintu1999 added a commit to mintu1999/O2Physics that referenced this pull request Oct 28, 2026
Please consider the following formatting changes to AliceO2Group#18485
Please consider the following formatting changes to AliceO2Group#18485
@mintu1999

Copy link
Copy Markdown
Author

All required tests are successful. Kindly approve for further checks.

@vkucera

vkucera commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

You need to resolve the conflicts first.

Please consider the following formatting changes to AliceO2Group#18485
@mintu1999

Copy link
Copy Markdown
Author

Dear @vkucera,

I have now resolved the conflicts. All the required tests are successful. kindly approve for further checks.

@vkucera

vkucera commented Oct 29, 2026

Copy link
Copy Markdown
Collaborator

Dear @vkucera,

I have now resolved the conflicts. All the required tests are successful. kindly approve for further checks.

Why did you not fix issues reported by the non-required tests as well?

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