Skip to content

Conversation

@gianniliveraro
Copy link
Contributor

This PR includes the following updates:

  • New functions for processing generated photons and Lambdas, taking event selection into account. These are intended for use in efficiency calculations and follow the implementation used in derivedlambdakzeroanalysis. A similar approach for Sigma0s is planned for a future update;

  • Removal of IR information from the Sigma0 data model;.

  • Safeguard added to the analyzeV0CollAssoc function to prevent double-counting of V0s. A check to ensure that only primary particles are considered has also been introduced.

@github-actions
Copy link

github-actions bot commented May 15, 2025

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

romainschotter
romainschotter previously approved these changes May 15, 2025
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for fb94bdb at 2025-05-16 00:55:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11216-slc9_x86-64/0/PWGLF/TableProducer/Strangeness/sigma0builder.cxx:1392:315: error: unused parameter 'mcmothers' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for da7db79 at 2025-05-16 06:24:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/11216-slc9_x86-64/0/PWGLF/TableProducer/Strangeness/sigma0builder.cxx:1392:315: error: unused parameter 'mcmothers' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@vkucera
Copy link
Collaborator

vkucera commented May 16, 2025

@gianniliveraro @romainschotter How about fixing some of those 375+ errors? :-)

@gianniliveraro gianniliveraro marked this pull request as draft May 16, 2025 12:05
@gianniliveraro gianniliveraro marked this pull request as ready for review May 16, 2025 12:16
@romainschotter romainschotter enabled auto-merge (squash) May 19, 2025 14:06
@romainschotter romainschotter merged commit 1fc194d into AliceO2Group:master May 22, 2025
11 of 13 checks passed
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
jinhyunni pushed a commit to jinhyunni/O2Physics that referenced this pull request Jun 11, 2025
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request Jun 12, 2025
…es (AliceO2Group#11216)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
ddobrigk pushed a commit to ddobrigk/O2Physics that referenced this pull request Jun 14, 2025
…es (AliceO2Group#11216)

Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
alibuild added 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.

5 participants