Skip to content

[PWGDQ] Add globalBc to DQ MC data model for MC truth BC checks#16398

Open
sigurdnese wants to merge 7 commits into
AliceO2Group:masterfrom
sigurdnese:reducedMcEvents
Open

[PWGDQ] Add globalBc to DQ MC data model for MC truth BC checks#16398
sigurdnese wants to merge 7 commits into
AliceO2Group:masterfrom
sigurdnese:reducedMcEvents

Conversation

@sigurdnese
Copy link
Copy Markdown
Contributor

  • Add globalBc to reducedMcEvents, in order to calculate MC truth TF and ITSROF border checks
  • New column is filled in tableMakerMc(_withAssoc), passing the globalBc from central data model McCollisions
  • Add converter to fill 'globalBc' column with dummy 0
  • Implement calculation of MC IsTFBorder and IsITSROFBorder variables in the VarManager
  • New helper function to set TF border calculation configurables and metadata
  • The TF border calculation ('recomputed') is also implemented for the real data case, akin to the existing ITSROFBorderRecomputed variable. The initialization of metadata and configurables are added to tableReader_withAssoc
  • All TFBorder and ITSROFBorder variables given names and units in the VarManager
  • The asymmetric-pairing task in MC (used for the UPC D0 analysis) also gets access to the MC TFBorder and ITSROFBorder calculations, by fetching the configurables from analysis-event-selection

https://indico.cern.ch/event/1688270/#11-update-of-the-reducedmceven

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 26, 2026

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

@github-actions github-actions Bot changed the title Add globalBc to DQ MC data model for MC truth BC checks [PWGDQ] Add globalBc to DQ MC data model for MC truth BC checks May 26, 2026
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not a task. This is a table producer.

@alibuild
Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 84e1ee3 at 2026-05-26 17:18:

## sw/BUILD/O2Physics-latest/log
CMake Error at cmake/O2PhysicsAddExecutable.cmake:113 (add_executable):
CMake Error at cmake/O2PhysicsAddExecutable.cmake:113 (add_executable):
ninja: error: rebuilding 'build.ninja': subcommand failed

Full log here.

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