Skip to content

[PWGDQ] Split the table-reader-with-assoc into several workflow specs#16863

Merged
iarsene merged 5 commits into
AliceO2Group:masterfrom
iarsene:master
Jun 29, 2026
Merged

[PWGDQ] Split the table-reader-with-assoc into several workflow specs#16863
iarsene merged 5 commits into
AliceO2Group:masterfrom
iarsene:master

Conversation

@iarsene

@iarsene iarsene commented Jun 29, 2026

Copy link
Copy Markdown
Collaborator
  1. Removed the WorkflowSpec from the tableReader_withAssoc.cxx
  2. Added several _workflowSpec.cxx files which define several workflows tailored for different analysis types
  3. Created a ProducesGroup in table-maker-with-assoc in order to save on the number of data members
  4. Re-enabled the processBConly() function in tableMaker_withAssoc

@github-actions github-actions Bot added the pwgdq label Jun 29, 2026
@github-actions github-actions Bot changed the title Split the table-reader-with-assoc into several workflow specs [PWGDQ] Split the table-reader-with-assoc into several workflow specs Jun 29, 2026
@github-actions

Copy link
Copy Markdown

O2 linter results: ❌ 275 errors, ⚠️ 126 warnings, 🔕 9 disabled

iarsene added a commit to iarsene/O2Physics that referenced this pull request Jun 29, 2026
[PWGDQ] Please consider the following formatting changes to AliceO2Group#16863
[PWGDQ] Please consider the following formatting changes to AliceO2Group#16863
@iarsene iarsene enabled auto-merge (squash) June 29, 2026 18:52

@alibuild alibuild left a comment

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.

Auto-approving on behalf of @iarsene.

@iarsene iarsene merged commit e8ed419 into AliceO2Group:master Jun 29, 2026
12 of 13 checks passed
// Contact: iarsene@cern.ch, i.c.arsene@fys.uio.no
// Configurable workflow for running several DQ or other PWG analyses

#include "PWGDQ/Tasks/tableReader_withAssoc.cxx"

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 acceptable. You are not supposed to include source files.

mcoquet642 pushed a commit to AliceMCH/O2Physics that referenced this pull request Jul 9, 2026
…AliceO2Group#16863)

Co-authored-by: Ionut Cristian Arsene <iarsene@cern.ch>
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

Labels

Development

Successfully merging this pull request may close these issues.

3 participants