[PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvector by dsekihat · Pull Request #15108 · AliceO2Group/O2Physics

dsekihat · 2026-02-22T21:36:50Z

PWGEM/Dilepton: reduce data size in Event,Mult,Cent,Qvector
Reduce photon's dependencies on dilepton

Please consider the following formatting changes to AliceO2Group#15108

alibuild · 2026-02-22T22:55:16Z

Error while checking build/O2Physics/o2 for 55cfda2 at 2026-02-22 23:55:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/associateMCinfoDilepton.cxx:143:231: error: unused parameter 'v0photons' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/associateMCinfoDilepton.cxx:143:231: error: unused parameter 'v0photons' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/Dilepton/TableProducer/associateMCinfoDilepton.cxx:143:231: error: unused parameter 'v0photons' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

Comment out the processChargedTrack function and its associated PROCESS_SWITCH.

alibuild · 2026-02-23T01:21:42Z

Error while checking build/O2Physics/o2 for 4e2f295 at 2026-02-23 05:37:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
/sw/SOURCES/O2Physics/15108-slc9_x86-64/0/PWGEM/PhotonMeson/Core/EMPhotonEventCut.h:56:8: error: infinite recursion detected [-Werror=infinite-recursion]
ninja: build stopped: subcommand failed.

Full log here.

Please consider the following formatting changes to AliceO2Group#15108

alibuild

Auto-approving on behalf of @dsekihat.

mhemmer-cern · 2026-02-23T10:10:16Z

Hello Daiki,
Do I understand correctly that with this commit merged, the photon meson would use the "old" Event, Mult, Cent and QVector tables everywhere, however, the table that maps photon candidates (and the e+e- for Dalitz) to the event these tables will be updated and require the usage of converter?
How does the difference in versions of the main event tables affect the event selection task, that currently is within the dilepton code but is used by both dilepton and photon meson code? Is it compatible with both versions of the event tables?
Can we still create new derived data with the "old" event tables, since they are the ones still used as dependencies in our tasks?

dsekihat · 2026-02-23T10:23:41Z

Hi Marvin,
Do I understand correctly that with this commit merged, the photon meson would use the "old" Event, Mult, Cent and QVector tables everywhere, however, the table that maps photon candidates (and the e+e- for Dalitz) to the event these tables will be updated and require the usage of converter?
--> The converters are necessary for the indices from photon candidates to the event table. They are prepared in this PR, too.
How does the difference in versions of the main event tables affect the event selection task, that currently is within the dilepton code but is used by both dilepton and photon meson code? Is it compatible with both versions of the event tables?
--> EMPhotonEventCut is also updated and independent from EMEventCut from now on.
Can we still create new derived data with the "old" event tables, since they are the ones still used as dependencies in our tasks?
--> Yes, sure. One should store EMEvents_004 in createEMPhotonEvent.cxx.

mhemmer-cern · 2026-02-23T10:48:28Z

Hi Marvin, Do I understand correctly that with this commit merged, the photon meson would use the "old" Event, Mult, Cent and QVector tables everywhere, however, the table that maps photon candidates (and the e+e- for Dalitz) to the event these tables will be updated and require the usage of converter? --> The converters are necessary for the indices from photon candidates to the event table. They are prepared in this PR, too. How does the difference in versions of the main event tables affect the event selection task, that currently is within the dilepton code but is used by both dilepton and photon meson code? Is it compatible with both versions of the event tables? --> EMPhotonEventCut is also updated and independent from EMEventCut from now on. Can we still create new derived data with the "old" event tables, since they are the ones still used as dependencies in our tasks? --> Yes, sure. One should store EMEvents_004 in createEMPhotonEvent.cxx.

Thanks a lot for the quick reply and clarifications.

dsekihat · 2026-02-23T11:04:44Z

To be precise, the necessary converters are o2-analysis-em-v0photonid-converter1, o2-analysis-em-emcid-converter1, o2-analysis-em-phosid-converter1.

vkucera · 2026-02-24T11:31:43Z

PWGEM/Dilepton/TableProducer/Converters/eventCentConverter1.cxx

+#include "Framework/ASoAHelpers.h"
+#include "Framework/AnalysisTask.h"
+#include "Framework/runDataProcessing.h"


Include external headers using <>.

vkucera · 2026-03-02T23:05:29Z

PWGEM/Dilepton/TableProducer/Converters/eventCentConverter1.cxx

+
+WorkflowSpec defineDataProcessing(ConfigContext const& cfgc)
+{
+  return WorkflowSpec{adaptAnalysisTask<eventCentConverter1>(cfgc, TaskName{"event-cent-converter1"})};


Please do not use TaskName if you don't need to.

PWGEM/Dilepton: reduce data size in Event,Mult,Cent,Qvecot

d8e39a5

dsekihat requested review from alibuild, feisenhu, hscheid, ivorobye, jokonig, m-c-danisch, mhemmer-cern, mikesas, novitzky and rbailhac as code owners February 22, 2026 21:36

github-actions bot added pwgem datamodel labels Feb 22, 2026

github-actions bot changed the title ~~PWGEM/Dilepton: reduce data size in Event,Mult,Cent,Qvecot~~ [PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvecot Feb 22, 2026

Please consider the following formatting changes

a329d98

alibuild mentioned this pull request Feb 22, 2026

Please consider the following formatting changes to #15108 dsekihat/O2Physics#79

Merged

dsekihat added 2 commits February 22, 2026 22:51

Merge branch 'master' into separate

42ec3f0

Merge pull request #79 from alibuild/alibot-cleanup-15108

55cfda2

Please consider the following formatting changes to AliceO2Group#15108

alibuild mentioned this pull request Feb 22, 2026

Please consider the following formatting changes to #15108 dsekihat/O2Physics#80

Closed

dsekihat changed the title ~~[PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvecot~~ [PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvector Feb 22, 2026

dsekihat added 3 commits February 23, 2026 00:06

Fix function signature in skimmingMC

66804a8

Disable processChargedTrack function and switch

d0b2e28

Comment out the processChargedTrack function and its associated PROCESS_SWITCH.

Fix selection bitmask references in Sel8 column

4e2f295

dsekihat and others added 2 commits February 23, 2026 08:43

Add multiple selection criteria to IsSelected function

4e20dd7

Please consider the following formatting changes

a319b01

alibuild mentioned this pull request Feb 23, 2026

Please consider the following formatting changes to #15108 dsekihat/O2Physics#81

Merged

Merge pull request #81 from alibuild/alibot-cleanup-15108

a6fa6b0

Please consider the following formatting changes to AliceO2Group#15108

dsekihat enabled auto-merge (squash) February 23, 2026 08:58

alibuild approved these changes Feb 23, 2026

View reviewed changes

dsekihat merged commit eebd803 into AliceO2Group:master Feb 23, 2026
11 of 12 checks passed

dsekihat deleted the separate branch February 23, 2026 11:02

vkucera reviewed Feb 24, 2026

View reviewed changes

mhemmer-cern mentioned this pull request Mar 2, 2026

[PWGEM] Fix Event subscription in process functions #15218

Merged

vkucera reviewed Mar 2, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvector#15108

[PWGEM/Dilepton] reduce data size in Event,Mult,Cent,Qvector#15108
dsekihat merged 10 commits intoAliceO2Group:masterfrom
dsekihat:separate

dsekihat commented Feb 22, 2026 •

edited

Loading

Uh oh!

alibuild commented Feb 22, 2026

Uh oh!

alibuild commented Feb 23, 2026 •

edited

Loading

Uh oh!

alibuild left a comment

Uh oh!

mhemmer-cern commented Feb 23, 2026

Uh oh!

dsekihat commented Feb 23, 2026

Uh oh!

mhemmer-cern commented Feb 23, 2026

Uh oh!

Uh oh!

dsekihat commented Feb 23, 2026

Uh oh!

vkucera Feb 24, 2026

Uh oh!

vkucera Mar 2, 2026

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

4 participants

Conversation

dsekihat commented Feb 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alibuild commented Feb 22, 2026

Uh oh!

alibuild commented Feb 23, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

alibuild left a comment

Choose a reason for hiding this comment

Uh oh!

mhemmer-cern commented Feb 23, 2026

Uh oh!

dsekihat commented Feb 23, 2026

Uh oh!

mhemmer-cern commented Feb 23, 2026

Uh oh!

Uh oh!

dsekihat commented Feb 23, 2026

Uh oh!

vkucera Feb 24, 2026

Choose a reason for hiding this comment

Uh oh!

vkucera Mar 2, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Milestone

Development

Uh oh!

4 participants

dsekihat commented Feb 22, 2026 •

edited

Loading

alibuild commented Feb 23, 2026 •

edited

Loading