Skip to content

[Common] Add zdc-extra-table-reader task#16028

Open
udmitrie wants to merge 7 commits intoAliceO2Group:masterfrom
udmitrie:zdcextrareader
Open

[Common] Add zdc-extra-table-reader task#16028
udmitrie wants to merge 7 commits intoAliceO2Group:masterfrom
udmitrie:zdcextrareader

Conversation

@udmitrie
Copy link
Copy Markdown
Contributor

zdc-extra-table-reader is needed to read AOD/ZDCEXTRA derived data and proceed with multi-step calibration (Q-vectors recentering) of ZDC data

zdc-extra-table-reader is needed to read AOD/ZDCEXTRA derived data and proceed with multistep calibration (Q-vectors recentering) of ZDC data
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 29, 2026

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

Refactor logging and constants in zdcExtraTableReader.
Replaced std::cerr logging with LOGF for error handling and updated constant names for consistency.
@udmitrie udmitrie marked this pull request as ready for review April 29, 2026 17:05
@udmitrie udmitrie marked this pull request as draft April 29, 2026 18:21
Add check number of fired towers with isZN*SpDeterminable (to reconstruct Q-vector at least 2 fired towers needed);
Replace FindBin with FindFixBin; 
update variables.
@udmitrie udmitrie marked this pull request as ready for review April 29, 2026 19:49
@udmitrie
Copy link
Copy Markdown
Contributor Author

Dear code owners,
Please take a look at this pull request.

There are no O2 linter errors connected to my changes.
The job is failing due to workflow naming convention violations in old tasks in Common/Tasks/CMakeLists.txt.

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