Skip to content

[PWGLF] Add K0S-K0S flow analysis task - #18316

Open
syano0822 wants to merge 3 commits into
AliceO2Group:masterfrom
syano0822:k0sk0s_raa_v2
Open

syano0822 wants to merge 3 commits into
AliceO2Group:masterfrom
syano0822:k0sk0s_raa_v2

Conversation

@syano0822

Copy link
Copy Markdown
Contributor

Add a configurable K0S-K0S analysis task for pp, pO, and OO collisions. The task provides single-track, K0S, and pair-level QA, same-event and mixed-event distributions, like-sign and sideband background estimators, and observables needed for scalar-product and event-plane flow measurements.

Add angular distributions and P2- and P4-weighted histograms to support the separation of scalar and tensor contributions.

Add a configurable K0S-K0S analysis task for pp, pO, and OO collisions. The task provides single-track, K0S, and pair-level QA, same-event and mixed-event distributions, like-sign and sideband background estimators, and observables needed for scalar-product and event-plane flow measurements.

Add angular distributions and P2- and P4-weighted histograms to support the separation of scalar and tensor contributions.
@github-actions

Copy link
Copy Markdown

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

@dmallick2

Copy link
Copy Markdown
Collaborator

@syano0822 Just for your information, for flow analysis on same K0s-K0s pair already there is a task available and others (junlee and sawan) are doing same ( it is not presented in PAG). https://github.com/AliceO2Group/O2Physics/blob/master/PWGLF/Tasks/Resonances/higherMassResonances.cxx. Is this one your requested different than this task or different motivation otherwise you can use this task.

@syano0822

Copy link
Copy Markdown
Contributor Author

Does the task contain scalar product function?

@dmallick2

Copy link
Copy Markdown
Collaborator

@syano0822. At a quick look, it seems that the scalar product is not included. I just wanted to inform you that there was a task for similar studies. We can either approve a new task where the scalar product is the only method, if that is more convenient, or modify the existing task by adding the scalar product. I have no problem proceeding with another task using only one method. I just wanted to flag this now to avoid any potential conflict between the analyzers in the future.

@syano0822

Copy link
Copy Markdown
Contributor Author

OK. I understand your point. Is it better to close this PR and the modify the existing f0 analysis code?

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 6a58dce at 2026-10-15 20:27:

## 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 25 errors and 90 warnings.'
Found 25 errors and 90 warnings.
++ [[ 25 -gt 0 ]]
++ cat /sw/BUILD/e92197611c536bce078e5f5d513d06ecadf5c066/O2Physics-code-check/errors.txt
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:32:1: error: included header GroupedCombinations.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:37:1: error: included header Vector3D.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:38:1: error: included header Vector4D.h is not used directly [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:127:3: error: no header providing "o2::framework::SliceCache" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:205:42: error: no header providing "o2::framework::VARIABLE_WIDTH" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:215:27: error: no header providing "o2::soa::Join" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:215:37: error: no header providing "o2::aod::Collisions" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:216:41: error: no header providing "o2::aod::TracksIU" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:216:56: error: no header providing "o2::aod::TracksExtra" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:216:74: error: no header providing "o2::aod::TracksCovIU" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:217:3: error: no header providing "o2::framework::Preslice" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:218:60: error: no header providing "o2::aod::track::collisionId" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:219:25: error: no header providing "o2::framework::ColumnBinningPolicy" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:219:61: error: no header providing "o2::aod::collision::PosZ" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:221:13: error: no header providing "o2::framework::InitContext" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:231:5: error: no header providing "o2::framework::AxisSpec" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:238:59: error: no header providing "o2::framework::HistType" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:315:38: error: no header providing "TH1" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:319:43: error: no header providing "TH2" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:320:10: error: no header providing "size_t" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:341:37: error: no header providing "ROOT::Math::PxPyPzMVector" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:467:91: error: no header providing "o2::constants::physics::MassK0Short" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:506:80: error: no header providing "o2::constants::physics::MassLambda0" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:538:17: error: no header providing "ROOT::Math::XYZVector" is directly included [misc-include-cleaner,-warnings-as-errors]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:595:82: error: no header providing "o2::constants::physics::MassPionCharged" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 90 -gt 0 ]]
++ cat /sw/BUILD/e92197611c536bce078e5f5d513d06ecadf5c066/O2Physics-code-check/warnings.txt
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:320:73: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:322:73: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:324:71: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:334:93: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:336:93: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:374:51: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:376:14: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:378:44: warning: statement should be inside braces [readability-braces-around-statements]
PWGLF/Tasks/Resonances/k0sK0sFlow.cxx:380:14: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 6a58dce at 2026-10-15 20:32:

No log files found

Full log here.

@vkucera

vkucera commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator

@syano0822 Since you are adding a new file, please fix all reported errors and warnings so that the CI reports only new issues in the future PRs.

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.

4 participants