Skip to content

[PWGDQ] Adding new process function for electron-muon analysis#15490

Open
Shunsuke-Kurita wants to merge 3 commits intoAliceO2Group:masterfrom
Shunsuke-Kurita:emu-tableReader-direct
Open

[PWGDQ] Adding new process function for electron-muon analysis#15490
Shunsuke-Kurita wants to merge 3 commits intoAliceO2Group:masterfrom
Shunsuke-Kurita:emu-tableReader-direct

Conversation

@Shunsuke-Kurita
Copy link
Contributor

This PR implements processElectronMuonDirect in AnalysisSameEventPairing for direct electron-muon pairing on AO2D tracks, completing the tableReader_withAssoc_direct workflow for EMu analyses. To support this, it also adds processDirectWithFilter to AnalysisEventSelection — enabling filterPPwithAssociation as an upstream pre-filter — and introduces a new AnalysisMuonSelection struct for direct muon selection on AO2D FwdTracks.

@github-actions github-actions bot added the pwgdq label Mar 23, 2026
@github-actions github-actions bot changed the title Adding new process function for electron-muon analysis [PWGDQ] Adding new process function for electron-muon analysis Mar 23, 2026
@vkucera
Copy link
Collaborator

vkucera commented Mar 23, 2026

@Shunsuke-Kurita Your PR fails the compilation and the formatting. Please test your changes before making a PR.

@alibuild
Copy link
Collaborator

Error while checking build/O2Physics/o2 for 189bde2 at 2026-03-23 12:36:

## sw/BUILD/O2Physics-latest/log
/sw/SOURCES/O2Physics/15490-slc9_x86-64/0/PWGDQ/Tasks/tableReader_withAssoc_direct.cxx:1315:40: error: unused parameter 'events' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/15490-slc9_x86-64/0/PWGDQ/Tasks/tableReader_withAssoc_direct.cxx:1997:110: error: unused parameter 'tracks1' [-Werror=unused-parameter]
/sw/SOURCES/O2Physics/15490-slc9_x86-64/0/PWGDQ/Tasks/tableReader_withAssoc_direct.cxx:1998:107: error: unused parameter 'tracks2' [-Werror=unused-parameter]
ninja: build stopped: subcommand failed.

Full log here.

@vkucera vkucera marked this pull request as draft March 23, 2026 13:32
@Shunsuke-Kurita
Copy link
Contributor Author

Dear @vkucera ,
Thank you for your feedback.
The code compiles successfully in my local environment, and I have also tested that the newly added process function works as expected.
However, I see that there are compilation errors in the CI. I will check the issues and update the code accordingly.
Thank you for pointing this out.

@vkucera vkucera marked this pull request as ready for review March 25, 2026 15:48
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