Skip to content

[PWGCF] Add event shape processing in pair track track task - #17814

Merged
ariedel-cern merged 2 commits into
AliceO2Group:masterfrom
ariedel-cern:master
Sep 8, 2026
Merged

[PWGCF] Add event shape processing in pair track track task#17814
ariedel-cern merged 2 commits into
AliceO2Group:masterfrom
ariedel-cern:master

Conversation

@ariedel-cern

@ariedel-cern ariedel-cern commented Sep 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions github-actions Bot added the pwgcf label Sep 7, 2026
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown

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

@ariedel-cern
ariedel-cern marked this pull request as ready for review September 8, 2026 07:00
@ariedel-cern
ariedel-cern enabled auto-merge (squash) September 8, 2026 07:00
alibuild
alibuild approved these changes Sep 8, 2026
View reviewed changes

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ariedel-cern.

@alibuild

alibuild commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 5b9b354 at 2026-09-08 11:22:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 0 warnings.'
Found 1 errors and 0 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/d0b77cedbbcd7e7e7e32f54ec68b23cfa8c27b71/O2Physics-code-check/errors.txt
PWGCF/Femto/Core/pairBuilder.h:221:32: error: no header providing "o2::analysis::femto::utils::HasEventShape" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@ariedel-cern
ariedel-cern merged commit a0ba873 into AliceO2Group:master Sep 8, 2026
13 of 15 checks passed
@ariedel-cern

ariedel-cern commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator Author

Code check error is fixed in this follow-up PR: #17829

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild alibuild approved these changes
@lauraser lauraser Awaiting requested review from lauraser lauraser is a code owner
@dimihayl dimihayl Awaiting requested review from dimihayl dimihayl is a code owner
@victor-gonzalez victor-gonzalez Awaiting requested review from victor-gonzalez victor-gonzalez is a code owner
@dsarkaralice dsarkaralice Awaiting requested review from dsarkaralice dsarkaralice is a code owner
@wrzesaCERN wrzesaCERN Awaiting requested review from wrzesaCERN wrzesaCERN is a code owner

Assignees

No one assigned

Labels

pwgcf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ariedel-cern @alibuild