Skip to content

[PWGCF] Implement updates for multi-system analysis #14469

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
somabhatta wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

[PWGCF] Implement updates for multi-system analysis #14469

somabhatta wants to merge 5 commits into from
+320 −245

Conversation

@somabhatta
Copy link
Contributor

@somabhatta somabhatta commented Jan 13, 2026 •
edited
Loading

[PWGCF] : This PR is to allow the same code to use different efficiency files according to system under study. Option set for Flattening weights, can be set to 1 if needed.

@github-actions github-actions bot added the pwgcf label Jan 13, 2026
@github-actions github-actions bot changed the title Implement updates for multi-system analysis [PWGCF] Implement updates for multi-system analysis Jan 13, 2026
@github-actions
Copy link

github-actions bot commented Jan 13, 2026 •
edited
Loading

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

@alibuild alibuild mentioned this pull request Jan 13, 2026
Closed
@alibuild
Copy link
Collaborator

alibuild commented Jan 13, 2026 •
edited
Loading

Error while checking build/O2Physics/o2 for 2d78c2f at 2026-01-13 15:35:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14469-slc9_x86-64/0/PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:409:35: error: unused parameter 'cent' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/14469-slc9_x86-64/0/PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:409:47: error: unused parameter 'eta' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/14469-slc9_x86-64/0/PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:409:58: error: unused parameter 'phi' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/14469-slc9_x86-64/0/PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:409:67: error: unused parameter 'pidType' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Jan 13, 2026

Error while checking build/O2Physics/o2 for d03bd2c at 2026-01-13 16:41:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/14469-slc9_x86-64/0/PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:419:9: error: passing 'const o2::framework::Configurable<bool>' as 'this' argument discards qualifiers [-fpermissive]
ninja: build stopped: subcommand failed.

Full log here.

@somabhatta somabhatta marked this pull request as draft January 13, 2026 19:33
@somabhatta somabhatta marked this pull request as ready for review January 13, 2026 19:34
@somabhatta somabhatta marked this pull request as draft January 13, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@victor-gonzalez victor-gonzalez Awaiting requested review from victor-gonzalez victor-gonzalez is a code owner

@zchochul zchochul Awaiting requested review from zchochul zchochul is a code owner

@lgraczykCern lgraczykCern Awaiting requested review from lgraczykCern lgraczykCern is a code owner

@prchakra prchakra Awaiting requested review from prchakra prchakra is a code owner

@lauraser lauraser Awaiting requested review from lauraser lauraser is a code owner

@ariedel-cern ariedel-cern Awaiting requested review from ariedel-cern ariedel-cern is a code owner

@otonvd otonvd Awaiting requested review from otonvd otonvd is a code owner

@shouqiye shouqiye Awaiting requested review from shouqiye shouqiye is a code owner

@glromane glromane Awaiting requested review from glromane glromane is a code owner

@njacazio njacazio Awaiting requested review from njacazio njacazio is a code owner

@hscheid hscheid Awaiting requested review from hscheid hscheid is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

alice3 datamodel pwgcf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@somabhatta @alibuild @lhusova @njacazio