Skip to content

[PWGCF] Lambda cascade correlation#16869

Draft
Oveissheibani wants to merge 6 commits into
AliceO2Group:masterfrom
Oveissheibani:lambda-cascade-correlation-phase16c
Draft

[PWGCF] Lambda cascade correlation#16869
Oveissheibani wants to merge 6 commits into
AliceO2Group:masterfrom
Oveissheibani:lambda-cascade-correlation-phase16c

Conversation

@Oveissheibani

Copy link
Copy Markdown
Contributor

Lambda-Cascade correlation refactors:

Old: LambdaTableProducer + CascadeSelector + LambdaR2Correlation + CascadeCorrelations
New: LambdaCascadeProducer + LambdaXiCorrelation (LambdaTracksExtProducer kept)

Added:

  • Combined Lambda + Cascade producer -> single event loop
  • IsTrueCascade column -> MC truth filter
  • processMCGenXi / processMCGenOmega -> gen-level R2 closure
  • Eff/Gen + Eff/Reco maps -> 3D (cent x pt x y) efficiency
  • OutputObj -> per-candidate topology + truth tags
  • EventCuts struct -> centralized event selection
  • ITS strangeness tracking switches
  • processAllPairs, processYields

Removed:

  • CascadeSelector, CascadeCorrelations

Main Advantage:
One event loop -> Lambda and cascade share identical event sample

@github-actions github-actions Bot added the pwgcf label Jun 29, 2026
@github-actions github-actions Bot changed the title Lambda cascade correlation [PWGCF] Lambda cascade correlation Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

O2 linter results: ❌ 11 errors, ⚠️ 49 warnings, 🔕 11 disabled

@vkucera

vkucera commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator

You are supposed to check your formatting before you open a PR for review.

@Oveissheibani Oveissheibani force-pushed the lambda-cascade-correlation-phase16c branch from 21c8f1d to 48c435f Compare June 30, 2026 17:25
@Oveissheibani

Copy link
Copy Markdown
Contributor Author

You are supposed to check your formatting before you open a PR for review.

I apologize. I rechecked the standards agains but it failed again on O2 linter again. I need a day or two to fix it.
shall i close and reopen again or keep this one ?

@vkucera

vkucera commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

You are supposed to check your formatting before you open a PR for review.

I apologize. I rechecked the standards agains but it failed again on O2 linter again. I need a day or two to fix it. shall i close and reopen again or keep this one ?

No need to close the PR. You can keep it as draft until you are done with the fixes. Please note that you can enable automatic local formatting and run O2 linter locally as well. See the documentation.

@@ -12,6 +12,8 @@
/// \file Lambdacascadecorrelation.cxx
/// \brief Correlation-balance functions of multistrange baryons
/// \author Oveis Sheibani <oveis.sheibani@cern.ch>
//
// o2-linter: disable=name/workflow-file

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.

Why?

@vkucera

vkucera commented Jun 30, 2026

Copy link
Copy Markdown
Collaborator

@Oveissheibani Why are you trying to disable the O2 linter errors?

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