Skip to content

Comments

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

Merged
dsekihat merged 10 commits intoAliceO2Group:masterfrom
dsekihat:separate
Feb 23, 2026
Merged

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

Conversation

@dsekihat
Copy link
Collaborator

@dsekihat dsekihat commented Feb 22, 2026

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

@github-actions 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
@dsekihat 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
@alibuild
Copy link
Collaborator

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.

@alibuild
Copy link
Collaborator

alibuild commented Feb 23, 2026

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
@dsekihat dsekihat enabled auto-merge (squash) February 23, 2026 08:58
Copy link
Collaborator

@alibuild alibuild left a comment

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 @dsekihat.

@mhemmer-cern
Copy link
Collaborator

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
Copy link
Collaborator Author

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
Copy link
Collaborator

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 dsekihat merged commit eebd803 into AliceO2Group:master Feb 23, 2026
11 of 12 checks passed
@dsekihat dsekihat deleted the separate branch February 23, 2026 11:02
@dsekihat
Copy link
Collaborator Author

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

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