Skip to content

Conversation

@ddobrigk
Copy link
Collaborator

@ddobrigk ddobrigk commented Apr 10, 2025

  • [strangeness builder] adds a configurable useV0BufferForCascades to determine if the builder should save V0s used in cascades (uses more memory, less CPU) or if it should re-generate from scratch for each cascade (saves memory, uses more CPU). Ideal also when pipelining the strangeness builder (smaller memory footprint per device).
  • [strangeness builder] adds a de-duplication procedure for dealing with v0Type>1 V0s (i.e. photon conversion candidates). The algorithm for de-duplication can be set via deduplicationAlgorithm. Three options are added for testing: best CosPA, best DCA daughters and best CosPA + DCA daughters (consistent with EM techniques).
  • [strangeness builder helper] adds a function to recognize and group duplicated V0s at the aod::V0s level for ease of handling. Adds the possibility to skip all selections in the V0 building (necessary for de-duplication procedure). Adds an 'empty out content' command whenever a candidate is not successfully built.
  • [v0assoqa] adds a QC task to study the de-duplication procedure. Uses the full modularity of the strangeness builder helper to test. Not meant to be used as analysis but rather as development QC for the strangeness builder.

Tagging @gianniliveraro

@github-actions github-actions bot removed the common label Apr 10, 2025
@ddobrigk ddobrigk marked this pull request as ready for review April 13, 2025 08:25
@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 2da02ca at 2025-04-13 11:09:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/10811-slc9_x86-64/0/PWGLF/Tasks/QC/v0assoqa.cxx:469:49: error: comparison of integer expressions of different signedness: 'int' and 'std::vector<int>::size_type' {aka 'long unsigned int'} [-Werror=sign-compare]
/sw/SOURCES/O2Physics/10811-slc9_x86-64/0/PWGLF/Tasks/QC/v0assoqa.cxx:228:32: error: unused variable 'grpo' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@njacazio njacazio merged commit 69b8e67 into AliceO2Group:master Apr 13, 2025
10 of 13 checks passed
prottayCMT pushed a commit to prottayCMT/O2Physics2024 that referenced this pull request May 17, 2025
…etc (AliceO2Group#10811)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
ariedel-cern pushed a commit to ariedel-cern/O2Physics that referenced this pull request May 23, 2025
…etc (AliceO2Group#10811)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
smaff92 pushed a commit to smaff92/O2Physics that referenced this pull request Jun 17, 2025
…etc (AliceO2Group#10811)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
alibuild added a commit to alibuild/O2Physics that referenced this pull request Aug 11, 2025
…etc (AliceO2Group#10811)

Co-authored-by: David Dobrigkeit Chinellato <david.dobrigkeit.chinellato.cern.ch>
Co-authored-by: ALICE Builder <alibuild@users.noreply.github.com>
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.

3 participants