Skip to content

[PWGLF] added different centrality estimators #12584

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
Merged
romainschotter merged 4 commits into from
Aug 16, 2025
Merged

[PWGLF] added different centrality estimators #12584

romainschotter merged 4 commits into from
Aug 16, 2025

Conversation

@prottayCMT
Copy link
Contributor

@prottayCMT prottayCMT commented Aug 14, 2025

No description provided.

@github-actions github-actions bot added the pwglf label Aug 14, 2025
@github-actions
Copy link

github-actions bot commented Aug 14, 2025 •
edited
Loading

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

@github-actions github-actions bot changed the title removed the added cuts on findable clusters [PWGLF] removed the added cuts on findable clusters Aug 14, 2025
romainschotter
romainschotter previously approved these changes Aug 14, 2025
View reviewed changes
@romainschotter romainschotter enabled auto-merge (squash) August 14, 2025 18:32
@alibuild
Copy link
Collaborator

alibuild commented Aug 14, 2025

Error while checking build/O2Physics/o2 for 4543138 at 2025-08-14 21:04:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/12584-slc9_x86-64/0/PWGLF/Tasks/Strangeness/lambdapolsp.cxx:479:16: error: unused variable 'tpcNClsF' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

auto-merge was automatically disabled August 16, 2025 09:37

Head branch was pushed to by a user without write access

@prottayCMT prottayCMT changed the title [PWGLF] removed the added cuts on findable clusters [PWGLF] added different centrality estimators Aug 16, 2025
@alibuild
Copy link
Collaborator

alibuild commented Aug 16, 2025

Error while checking build/O2Physics/o2 for 6a7a792 at 2025-08-16 18:42:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/12584-slc9_x86-64/0/PWGLF/Tasks/Strangeness/lambdapolsp.cxx:1288:16: error: 'centrality' may be used uninitialized [-Werror=maybe-uninitialized]
/sw/SOURCES/O2Physics/12584-slc9_x86-64/0/PWGLF/Tasks/Strangeness/lambdapolsp.cxx:874:16: error: 'centrality' may be used uninitialized [-Werror=maybe-uninitialized]
ninja: build stopped: subcommand failed.

Full log here.

@romainschotter romainschotter merged commit 706f0f0 into AliceO2Group:master Aug 16, 2025
11 of 12 checks passed
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
@prottayCMT @ThePhDane
[PWGLF] added different centrality estimators (AliceO2Group#12584)
0b32ad2 
Co-authored-by: Prottay Das <prottay@alipap1.cern.ch>
jloemker pushed a commit to jloemker/O2Physics that referenced this pull request Nov 11, 2025
@prottayCMT @jloemker
[PWGLF] added different centrality estimators (AliceO2Group#12584)
463c741 
Co-authored-by: Prottay Das <prottay@alipap1.cern.ch>
alibuild pushed a commit to alibuild/O2Physics that referenced this pull request Dec 5, 2025
@prottayCMT
[PWGLF] added different centrality estimators (AliceO2Group#12584)
a088a61 
Co-authored-by: Prottay Das <prottay@alipap1.cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@romainschotter romainschotter romainschotter approved these changes

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

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

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

Assignees

No one assigned

Labels

pwglf

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@prottayCMT @alibuild @romainschotter