Skip to content

[PWGLF] Autoset the tables produced based on the task topology#16922

Open
njacazio wants to merge 4 commits into
masterfrom
nj-mccent-autoset
Open

[PWGLF] Autoset the tables produced based on the task topology#16922
njacazio wants to merge 4 commits into
masterfrom
nj-mccent-autoset

Conversation

@njacazio

@njacazio njacazio commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@njacazio njacazio marked this pull request as ready for review July 3, 2026 20:09
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown

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

@github-actions github-actions Bot added the pwglf label Jul 3, 2026
@github-actions github-actions Bot changed the title Autoset the tables produced based on the task topology [PWGLF] Autoset the tables produced based on the task topology Jul 3, 2026
@alibuild

alibuild commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 81f5ace at 2026-07-03 22:10:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:96:5: error: 'enableFlagIfTableRequired' was not declared in this scope; did you mean 'o2::common::core::enableFlagIfTableRequired'?
ninja: build stopped: subcommand failed.

Full log here.

@njacazio njacazio enabled auto-merge (squash) July 3, 2026 20:12
@alibuild

alibuild commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for b7268f4 at 2026-07-03 22:13:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:97:5: error: 'enableFlagIfTableRequired' was not declared in this scope; did you mean 'o2::common::core::enableFlagIfTableRequired'?
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for b7268f4 at 2026-07-03 22:22:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:97:5: error: 'enableFlagIfTableRequired' was not declared in this scope; did you mean 'o2::common::core::enableFlagIfTableRequired'?
ninja: build stopped: subcommand failed.

Full log here.

@alibuild

alibuild commented Jul 3, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for b7268f4 at 2026-07-04 07:41:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/slc9_x86-64-slc9_x86-64/0/PWGLF/TableProducer/Common/mcCentrality.cxx:97:5: error: 'enableFlagIfTableRequired' was not declared in this scope; did you mean 'o2::common::core::enableFlagIfTableRequired'?
ninja: build stopped: subcommand failed.

Full log here.

@vkucera vkucera marked this pull request as draft July 4, 2026 13:08
auto-merge was automatically disabled July 4, 2026 13:08

Pull request was converted to draft

@njacazio njacazio marked this pull request as ready for review July 5, 2026 09:45
@njacazio njacazio enabled auto-merge (squash) July 5, 2026 09:46
@alibuild

alibuild commented Jul 5, 2026

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/code-check for 3626810 at 2026-07-05 11:57:

## sw/BUILD/O2Physics-code-check-latest/log
--
========== List of issues found ==========
++ echo 'Found 5 errors and 0 warnings.'
Found 5 errors and 0 warnings.
++ [[ 5 -gt 0 ]]
++ cat /sw/BUILD/593aadf4fe7ca1cd53e67c86dfcf9d09057431bf/O2Physics-code-check/errors.txt
PWGLF/TableProducer/Common/mcCentrality.cxx:51:8: error: constructor does not initialize these fields: ccdb, pdgDB [cppcoreguidelines-pro-type-member-init,-warnings-as-errors]
PWGLF/TableProducer/Common/mcCentrality.cxx:114:20: error: use starts_with instead of rfind [modernize-use-starts-ends-with,-warnings-as-errors]
PWGLF/TableProducer/Common/mcCentrality.cxx:133:18: 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]
PWGLF/TableProducer/Common/mcCentrality.cxx:143:19: 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]
PWGLF/Utils/inelGt.h:117:11: error: using integer absolute value function 'abs' when argument is of floating point type [clang-diagnostic-absolute-value,-warnings-as-errors]
++ [[ 0 -gt 0 ]]
++ [[ 5 -gt 0 ]]
++ exit 1
--

Full log here.

@vkucera vkucera disabled auto-merge July 8, 2026 12:00
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.

2 participants