Skip to content

[Common] Adding task to read the table created for light ions and ZDC analysis #13721

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
ddobrigk merged 13 commits into from
Nov 13, 2025
Merged

[Common] Adding task to read the table created for light ions and ZDC analysis #13721

ddobrigk merged 13 commits into from
Nov 13, 2025

Conversation

@coppedis
Copy link
Contributor

@coppedis coppedis commented Nov 6, 2025

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 6, 2025 •
edited
Loading

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

@github-actions github-actions bot changed the title Adding task to read the table created for light ions and ZDC analysis [Common] Adding task to read the table created for light ions and ZDC analysis Nov 6, 2025
@alibuild
Copy link
Collaborator

alibuild commented Nov 7, 2025

Error while checking build/O2Physics/o2 for 4e6a40e at 2025-11-07 10:49:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:105:12: error: unused variable 'zvtx' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:106:12: error: unused variable 'centrFT0C' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:107:12: error: unused variable 'centrFT0A' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:108:12: error: unused variable 'centrFT0M' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:109:12: error: unused variable 'timestamp' [-Werror=unused-variable]
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:110:12: error: unused variable 'selectionBits' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@alibuild
Copy link
Collaborator

alibuild commented Nov 7, 2025 •
edited
Loading

Error while checking build/O2Physics/o2 for 93124bc at 2025-11-07 14:55:

## sw/BUILD/O2Physics-latest/log
cc1plus: fatal error: /sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Core/OrbitRange.cxx: No such file or directory
ninja: build stopped: subcommand failed.

Full log here.

@alibuild alibuild mentioned this pull request Nov 7, 2025
Closed
@alibuild
Copy link
Collaborator

alibuild commented Nov 8, 2025

Error while checking build/O2Physics/o2 for 8929ca5 at 2025-11-08 09:21:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:132:12: error: unused variable 'selectionBits' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@coppedis
Change in ZDC reader task
3ca0fed 
Timestamp min value for histos as configurable param
@alibuild
Copy link
Collaborator

alibuild commented Nov 10, 2025

Error while checking build/O2Physics/o2 for 3ca0fed at 2025-11-10 17:51:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/13721-slc9_x86-64/0/Common/Tasks/zdcTableReader.cxx:133:12: error: unused variable 'selectionBits' [-Werror=unused-variable]
ninja: build stopped: subcommand failed.

Full log here.

@coppedis
Comment out selectionBits
0a3abdd 
Comment out the selectionBits variable for the time being.
@alibuild alibuild mentioned this pull request Nov 10, 2025
Closed
@ddobrigk ddobrigk merged commit bfc91e5 into AliceO2Group:master Nov 13, 2025
12 of 13 checks passed
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
@coppedis
[Common] Adding task to read the table created for light ions and ZDC…
471729c 
… analysis (AliceO2Group#13721)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ddobrigk ddobrigk ddobrigk approved these changes

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@jgrosseo jgrosseo Awaiting requested review from jgrosseo jgrosseo is a code owner

@iarsene iarsene Awaiting requested review from iarsene iarsene is a code owner

@ktf ktf Awaiting requested review from ktf ktf is a code owner

Assignees

No one assigned

Labels

common datamodel

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@coppedis @alibuild @ddobrigk