Skip to content

Conversation

@mpuccio
Copy link
Collaborator

@mpuccio mpuccio commented Jun 2, 2025

This commit adds a new table called NucleiPairTable to the data model. The new table contains information about pairs of light nuclei, including their kinematic properties, detector signals, and quality flags.

The changes include:

  • Defining a new namespace NucleiPairTableNS with columns for the first and second particles in the pair.
  • Declaring a new table NucleiPairTable in the o2::aod namespace, which includes the columns from NucleiPairTableNS.
  • Modifying the nucleiSpectra.cxx file to produce the new NucleiPairTable in addition to the existing NucleiTable.
  • Adding a new configurable cfgFillPairTree to control whether the pair table should be filled.

These changes will enable the analysis of light nuclei pairs.

@mpuccio
Copy link
Collaborator Author

mpuccio commented Jun 2, 2025

@ivorobye

@github-actions
Copy link

github-actions bot commented Jun 2, 2025

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

@github-actions github-actions bot changed the title feat: Add nuclei pair table to data model [PWGLF] feat: Add nuclei pair table to data model Jun 2, 2025
This commit adds a new table called `NucleiPairTable` to the data model. The new table contains information about pairs of light nuclei, including their kinematic properties, detector signals, and quality flags.

The changes include:

- Defining a new namespace `NucleiPairTableNS` with columns for the first and second particles in the pair.
- Declaring a new table `NucleiPairTable` in the `o2::aod` namespace, which includes the columns from `NucleiPairTableNS`.
- Modifying the `nucleiSpectra.cxx` file to produce the new `NucleiPairTable` in addition to the existing `NucleiTable`.
- Adding a new configurable `cfgFillPairTree` to control whether the pair table should be filled.

These changes will enable the analysis of light nuclei pairs.
@mpuccio mpuccio enabled auto-merge (squash) June 2, 2025 16:03
Copy link
Collaborator

@maciacco maciacco left a comment

Choose a reason for hiding this comment

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

@skundu692 could you have a look?

@mpuccio mpuccio merged commit 8e467a8 into master Jun 2, 2025
13 of 15 checks passed
@mpuccio mpuccio deleted the mp-branch-17 branch June 2, 2025 19:38
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
hernasab pushed a commit to hernasab/O2Physics that referenced this pull request Jun 11, 2025
EmilGorm pushed a commit to EmilGorm/O2Physics that referenced this pull request Jun 12, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 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
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jul 16, 2025
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants