Skip to content

[PWGCF] Add new cuts and QA - #18565

Open
meiALICE wants to merge 3 commits into
AliceO2Group:masterfrom
meiALICE:master
Open

meiALICE wants to merge 3 commits into
AliceO2Group:masterfrom
meiALICE:master

Conversation

@meiALICE

@meiALICE meiALICE commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Add new cuts.

Add hist of number of selected tracks as new QA.

@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Add new cuts and QA [PWGCF] Add new cuts and QA Nov 3, 2026
abilandz
abilandz previously approved these changes Nov 3, 2026
@abilandz
abilandz enabled auto-merge (squash) November 3, 2026 11:15
@alibuild

alibuild commented Nov 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 56c72e3 at 2026-11-03 12:44:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'To reproduce the results below locally, run: '\''aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'\''.'
To reproduce the results below locally, run: 'aliBuild build O2Physics-code-check -e ALIBUILD_BASE_HASH=upstream/master'.
++ echo 'To apply all automatic fixes, add the '\''-e O2PHYSICS_CHECKER_FIX=1'\'' option.'
To apply all automatic fixes, add the '-e O2PHYSICS_CHECKER_FIX=1' option.
++ echo 'See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.'
See the documentation for more details at: <https://aliceo2group.github.io/analysis-framework/docs/tools/#running-with-o2physics-code-check>.
++ echo 'Found 1 errors and 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/c2f9fe9ce238bab7d540f6b0507a575fe83e03f5/O2Physics-code-check/errors.txt
PWGCF/MultiparticleCorrelations/Tasks/multiparticleCorrelationsMei.cxx:205:65: error: unused variable 'OccupancyEstimatorNames' [clang-diagnostic-unused-const-variable,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

auto-merge was automatically disabled November 3, 2026 13:34

Head branch was pushed to by a user without write access

@abilandz
abilandz enabled auto-merge (squash) November 3, 2026 14:38
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