Skip to content

Conversation

@torkjellsdatter
Copy link
Contributor

This PR introduces a table converter to handle updates in the REDUCEDMCEVENT table structure, introduced by #13569

Table definitions

Added two explicit table definitions in ReducedInfoTables.h:

  • ReducedMCEvents_000 — the old version of the REDUCEDMCEVENT table
  • ReducedMCEvents_001 — the new version including the additional variables

Converter task

Implemented a new converter task: ModelConverterReducedMCEvents

  • Reads from ReducedMCEvents_000 (old table)
  • Converts to ReducedMCEvents_001 (new table)
  • Sets newly introduced variables to -1 by default

Build integration

  • Updated CMakeLists.txt to include the new converter in the build system.

Usage

The converter can be run as part of the analysis chain:

o2-analysis-dq-efficiency-with-assoc -b --configuration json://config.json \
  | o2-analysis-dq-model-converter-mc-reduced-event -b --configuration json://config.json

@github-actions github-actions bot changed the title Add converter for updated REDUCEDMCEVENT table (ReducedInfoTables.h, CMakeList.txt, new task) [PWGDQ] Add converter for updated REDUCEDMCEVENT table (ReducedInfoTables.h, CMakeList.txt, new task) Oct 31, 2025
@github-actions
Copy link

github-actions bot commented Oct 31, 2025

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

@iarsene iarsene enabled auto-merge (squash) October 31, 2025 12:24
@iarsene iarsene merged commit c9dfedd into AliceO2Group:master Oct 31, 2025
12 of 13 checks passed
ThePhDane pushed a commit to ThePhDane/O2Physics that referenced this pull request Nov 3, 2025
…bles.h, CMakeList.txt, new task) (AliceO2Group#13620)

Co-authored-by: Ida Torkjellsdatter Storehaug <idats@eduroam-193-157-190-10.wlan.uio.no>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
lmattei01 pushed a commit to lmattei01/O2Physics that referenced this pull request Dec 5, 2025
…bles.h, CMakeList.txt, new task) (AliceO2Group#13620)

Co-authored-by: Ida Torkjellsdatter Storehaug <idats@eduroam-193-157-190-10.wlan.uio.no>
Co-authored-by: ALICE Action Bot <alibuild@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

3 participants