Skip to content

[Tutorial,PWGLF] Add StrangenessV0s hands-on tutorial - #18740

Open
spucillo wants to merge 10 commits into
AliceO2Group:masterfrom
spucillo:master
Open

[Tutorial,PWGLF] Add StrangenessV0s hands-on tutorial#18740
spucillo wants to merge 10 commits into
AliceO2Group:masterfrom
spucillo:master

Conversation

@spucillo

@spucillo spucillo commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

github-actions Bot commented Nov 13, 2026

Copy link
Copy Markdown

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

@alibuild alibuild mentioned this pull request Nov 13, 2026
Merged
@spucillo spucillo changed the title [PWGLF] Add StrangenessV0s hands-on tutorial [TutorialStrangeness,PWGLF] Add StrangenessV0s hands-on tutorial] Add StrangenessV0s hands-on tutorial Nov 13, 2026
@alibuild

alibuild commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 4620b42 at 2026-11-13 19:19:

## 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 5 errors and 32 warnings.'
Found 5 errors and 32 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/a2338fbc2e1ecdb736b12caa67d3b20db7f61688/O2Physics-code-check/errors.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:26:3: error: no header providing "TIter" is directly included [misc-include-cleaner,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:27:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:75:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:88:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:24:1: error: included header PhysicsConstants.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 32 -gt 0 ]]
++ cat /sw/BUILD/a2338fbc2e1ecdb736b12caa67d3b20db7f61688/O2Physics-code-check/warnings.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:45:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:46:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:47:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:48:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:49:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:50:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:53:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:54:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:55:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:56:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:57:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:58:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:59:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:61:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:62:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:63:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:64:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:65:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:117:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:122:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:125:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:137:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:142:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:145:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:146:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:151:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:154:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:171:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:176:42: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@spucillo spucillo changed the title [TutorialStrangeness,PWGLF] Add StrangenessV0s hands-on tutorial] Add StrangenessV0s hands-on tutorial Add StrangenessV0s hands-on tutorial Nov 13, 2026
@github-actions github-actions Bot changed the title Add StrangenessV0s hands-on tutorial [Tutorial,PWGLF] Add StrangenessV0s hands-on tutorial Nov 13, 2026
spucillo added a commit to spucillo/O2Physics that referenced this pull request Nov 13, 2026
@spucillo
Merge pull request #19 from alibuild/alibot-cleanup-18740
669a8b9 
[Tutorial,PWGLF] Please consider the following formatting changes to AliceO2Group#18740
@alibuild

alibuild commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 669a8b9 at 2026-11-13 19: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 5 errors and 32 warnings.'
Found 5 errors and 32 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/d02d1974e5fd0d3a3c73ffe88e7e3a904515c758/O2Physics-code-check/errors.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:36:3: error: no header providing "TIter" is directly included [misc-include-cleaner,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:37:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:85:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:98:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:24:1: error: included header PhysicsConstants.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 32 -gt 0 ]]
++ cat /sw/BUILD/d02d1974e5fd0d3a3c73ffe88e7e3a904515c758/O2Physics-code-check/warnings.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:55:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:56:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:57:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:58:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:59:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:60:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:63:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:64:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:65:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:66:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:67:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:68:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:69:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:71:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:72:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:73:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:74:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:75:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:117:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:122:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:125:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:137:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:142:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:145:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:146:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:151:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:154:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:171:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:176:42: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

alibuild commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 23ac5cd at 2026-11-13 19:33:

## 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 5 errors and 32 warnings.'
Found 5 errors and 32 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/518b31e0b1d26e047090c1f3459d79cc75dfa890/O2Physics-code-check/errors.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:36:3: error: no header providing "TIter" is directly included [misc-include-cleaner,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:37:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:85:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:98:22: error: do not use static_cast to downcast from a base to a derived class; use dynamic_cast instead [cppcoreguidelines-pro-type-static-cast-downcast,-warnings-as-errors]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:24:1: error: included header PhysicsConstants.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 32 -gt 0 ]]
++ cat /sw/BUILD/518b31e0b1d26e047090c1f3459d79cc75dfa890/O2Physics-code-check/warnings.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:55:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:56:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:57:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:58:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:59:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:60:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:63:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:64:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:65:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:66:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:67:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:68:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:69:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:71:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:72:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:73:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:74:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/checkTables.C:75:5: warning: use designated initializer list to initialize 'RequiredTable' [modernize-use-designated-initializers]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:117:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:122:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:125:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:137:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:142:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:145:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:146:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:151:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:154:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:171:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:176:42: warning: statement should be inside braces [readability-braces-around-statements]
[0 more errors; see full log]

Full log here.

@alibuild

alibuild commented Nov 13, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 6907bed at 2026-11-13 19:33:

## 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 1 errors and 14 warnings.'
Found 1 errors and 14 warnings.
++ [[ 1 -gt 0 ]]
++ cat /sw/BUILD/331ee2ebe22e049f278338ab2842333b06b87f6f/O2Physics-code-check/errors.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:24:1: error: included header PhysicsConstants.h is not used directly [misc-include-cleaner,-warnings-as-errors]
++ [[ 14 -gt 0 ]]
++ cat /sw/BUILD/331ee2ebe22e049f278338ab2842333b06b87f6f/O2Physics-code-check/warnings.txt
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:117:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:122:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step1.cxx:125:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:137:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:142:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step2.cxx:145:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:146:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:151:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step3.cxx:154:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:171:28: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:176:42: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:179:44: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:305:31: warning: statement should be inside braces [readability-braces-around-statements]
Tutorials/PWGLF/Strangeness/StrangenessV0s_HandsOn/o2physics/AnalysisV0/strangeness_derived_v0_step4.cxx:310:37: warning: statement should be inside braces [readability-braces-around-statements]
++ [[ 1 -gt 0 ]]
++ exit 1
--

Full log here.

@alibuild alibuild mentioned this pull request Nov 13, 2026
Closed
@alibuild alibuild mentioned this pull request Nov 13, 2026
Merged
@spucillo
Merge pull request #21 from alibuild/alibot-cleanup-ml-18740
9546382 
[Tutorial,PWGLF] [MegaLinter] Apply linters automatic fixes to AliceO2Group#18740
romainschotter
romainschotter requested changes Nov 13, 2026
View reviewed changes

@romainschotter romainschotter left a comment •
edited
Loading

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.

Hi @spucillo ! Thank you very much for preparing the hands-on tasks!
It looks great!
I just have a few comments:

  • Since these are all tasks running over derived data, could you put everything within Tutorials/PWGLF/Strangeness/Derived
  • could you also rename the folder o2physics/AnalysisV0 into AnalysisV0? (since we are already working within O2Physics). Similarly, the existing folder Analysis within Tutorials/PWGLF/Strangeness/Derived could be renamed AnalysisCascade
  • could you please remove the input_data.txt and input_MC.txt? We usually don't put them in O2Physics and just provide them in slides for the hands-on session.

Thank you very much!

@alibuild alibuild mentioned this pull request Nov 14, 2026
Merged
@spucillo
Merge pull request #22 from alibuild/alibot-cleanup-ml-18740
055fdab 
[Tutorial,PWGLF] [MegaLinter] Apply linters automatic fixes to AliceO2Group#18740
@spucillo

spucillo commented Nov 14, 2026

Copy link
Copy Markdown
Contributor Author

Hi @romainschotter,

Thanks a lot for the review! I moved everything into Tutorials/PWGLF/Strangeness/Derived/AnalysisV0, organized in the same way as the existing cascade folder, and removed input_data.txt and input_MC.txt (the README now explains how to create them). I left the existing Analysis folder unchanged, since renaming it would also require updating the cascade tutorial material.

All the checks are now passing, so the PR should be ready for another look. Let me know if anything else needs to be changed!

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
@alcaliva alcaliva Awaiting requested review from alcaliva alcaliva is a code owner
@lbariogl lbariogl Awaiting requested review from lbariogl lbariogl is a code owner
@BongHwi BongHwi Awaiting requested review from BongHwi BongHwi is a code owner
@lbarnby lbarnby Awaiting requested review from lbarnby lbarnby is a code owner
@ercolessi ercolessi Awaiting requested review from ercolessi ercolessi is a code owner
@iravasen iravasen Awaiting requested review from iravasen iravasen is a code owner
@njacazio njacazio Awaiting requested review from njacazio njacazio is a code owner
@skundu692 skundu692 Awaiting requested review from skundu692 skundu692 is a code owner
@lhusova lhusova Awaiting requested review from lhusova lhusova is a code owner
@romainschotter romainschotter Awaiting requested review from romainschotter romainschotter is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

pwglf tutorial

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@spucillo @alibuild @romainschotter