Skip to content

[PWGCF] radial flow decorr task update to add C3 skewness as an observable and other binning changes - #17856

Merged
isputows merged 4 commits into
AliceO2Group:masterfrom
somabhatta:radial-flow-decorr-refactor2
Sep 10, 2026
Merged

[PWGCF] radial flow decorr task update to add C3 skewness as an observable and other binning changes#17856
isputows merged 4 commits into
AliceO2Group:masterfrom
somabhatta:radial-flow-decorr-refactor2

Conversation

@somabhatta

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

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

@somabhatta
somabhatta force-pushed the radial-flow-decorr-refactor2 branch from f908811 to 1edad99 Compare September 9, 2026 23:35
@somabhatta
somabhatta marked this pull request as ready for review September 9, 2026 23:45
@somabhatta
somabhatta marked this pull request as draft September 10, 2026 00:16
@somabhatta
somabhatta marked this pull request as ready for review September 10, 2026 01:20
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 2227b8a at 2026-09-10 04:22:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 1 errors and 7 warnings.'
Found 1 errors and 7 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/3099949a1da4c06b455edd781827db24057aae2c/O2Physics-code-check/errors.txt
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:58:1: error: included header tuple is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 7 -gt 0 ]]
++ cat /sw/BUILD/3099949a1da4c06b455edd781827db24057aae2c/O2Physics-code-check/warnings.txt
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:82:56: warning: casting (double + 0.5) to integer leads to incorrect rounding; consider using lround (#include <cmath>) instead [bugprone-incorrect-roundings]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2230:19: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2232:38: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2236:19: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2238:38: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2242:19: warning: statement should be inside braces [readability-braces-around-statements]
PWGCF/EbyEFluctuations/Tasks/radialFlowDecorr.cxx:2244:38: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@somabhatta
somabhatta marked this pull request as draft September 10, 2026 09:19
@somabhatta
somabhatta marked this pull request as ready for review September 10, 2026 10:10
@isputows
isputows enabled auto-merge (squash) September 10, 2026 13:07
@isputows
isputows merged commit 6481925 into AliceO2Group:master Sep 10, 2026
15 checks passed
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