Skip to content

[PWGCF] Femto: Allow multiple registries for SH histograms - #17296

Draft
IgorPtak wants to merge 1 commit into
AliceO2Group:masterfrom
IgorPtak:spherical-harmonics-registry
Draft

[PWGCF] Femto: Allow multiple registries for SH histograms#17296
IgorPtak wants to merge 1 commit into
AliceO2Group:masterfrom
IgorPtak:spherical-harmonics-registry

Conversation

@IgorPtak

@IgorPtak IgorPtak commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Problem

SH histograms scale as nCent * nKt * (2 * nJM + 2) and overflow the
512-histogram limit of a single HistogramRegistry.

Solution

PairHistManager::init() takes an optional pool of registries for SH
histograms and assigns each (cent, kT) cell to one with room. Builders
forward separate pools for same-event and mixed-event.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown

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

@vkucera

vkucera commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

If you need more than 512 histograms, I think you are doing something wrong.

@alibuild

alibuild commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for be6bd63 at 2026-08-05 10:23:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for be6bd63 at 2026-08-05 09:58:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@alibuild

alibuild commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for be6bd63 at 2026-08-05 12:53:

## sw/BUILD/O2-latest/log
CMake Error in Detectors/Base/CMakeLists.txt:

Full log here.

@IgorPtak
IgorPtak marked this pull request as draft August 5, 2026 13:24
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