Skip to content

[Tutorial] Updating tutorial task - #18755

Merged
dsekihat merged 1 commit into
AliceO2Group:masterfrom
ChSonnabend:o2at6
Nov 15, 2026
Merged

dsekihat merged 1 commit into
AliceO2Group:masterfrom
ChSonnabend:o2at6

Conversation

@ChSonnabend

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Updating tutorial task [Tutorial] Updating tutorial task Nov 15, 2026
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 180c176 at 2026-11-15 13: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 3 errors and 11 warnings.'
Found 3 errors and 11 warnings.
++ [[ 3 -gt 0 ]]
++ cat /sw/BUILD/95c1d91b66631f8cdd850531259220a7b8f5a4d8/O2Physics-code-check/errors.txt
Tutorials/ML/benchmarkOnnxModel.cxx:62:22: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
Tutorials/ML/benchmarkOnnxModel.cxx:64:17: error: do not use reinterpret_cast [cppcoreguidelines-pro-type-reinterpret-cast,-warnings-as-errors]
Tutorials/ML/benchmarkOnnxModel.cxx:236:24: error: no header providing "std::ws" is directly included [misc-include-cleaner,-warnings-as-errors]
++ [[ 11 -gt 0 ]]
++ cat /sw/BUILD/95c1d91b66631f8cdd850531259220a7b8f5a4d8/O2Physics-code-check/warnings.txt
Tutorials/ML/benchmarkOnnxModel.cxx:80:3: warning: function 'used' should be marked [[nodiscard]] [modernize-use-nodiscard]
Tutorials/ML/benchmarkOnnxModel.cxx:165:15: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
Tutorials/ML/benchmarkOnnxModel.cxx:166:15: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
Tutorials/ML/benchmarkOnnxModel.cxx:167:81: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
Tutorials/ML/benchmarkOnnxModel.cxx:167:87: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
Tutorials/ML/benchmarkOnnxModel.cxx:252:11: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
Tutorials/ML/benchmarkOnnxModel.cxx:267:46: warning: do not declare C-style arrays, use 'std::array' instead [modernize-avoid-c-arrays]
Tutorials/ML/benchmarkOnnxModel.cxx:267:46: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
Tutorials/ML/benchmarkOnnxModel.cxx:285:46: warning: do not implicitly decay an array into a pointer; consider using gsl::array_view or an explicit cast instead [cppcoreguidelines-pro-bounds-array-to-pointer-decay]
Tutorials/ML/benchmarkOnnxModel.cxx:299:13: warning: use auto when initializing with a cast to avoid duplicating the type name [modernize-use-auto]
Tutorials/ML/benchmarkOnnxModel.cxx:315:15: warning: implicit conversion 'size_t' (aka 'unsigned long') -> 'bool' [readability-implicit-bool-conversion]
++ [[ 3 -gt 0 ]]
++ exit 1
--

Full log here.

@dsekihat
dsekihat merged commit 7e8b4a9 into AliceO2Group:master Nov 15, 2026
13 of 14 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