Skip to content

[PWGUD] add true gap service and TPC-FT0 correlations#16083

Open
Luzhiyongg wants to merge 11 commits intoAliceO2Group:masterfrom
Luzhiyongg:master
Open

[PWGUD] add true gap service and TPC-FT0 correlations#16083
Luzhiyongg wants to merge 11 commits intoAliceO2Group:masterfrom
Luzhiyongg:master

Conversation

@Luzhiyongg
Copy link
Copy Markdown
Contributor

@Luzhiyongg Luzhiyongg commented May 4, 2026

No description provided.

Luzhiyongg and others added 9 commits May 4, 2026 13:36
@Luzhiyongg
Add UpcService.h
bece71e 
This header file defines the Truegapside service outputs for the UPC analysis.
@Luzhiyongg
Add UpcTrueGapService task for true gap analysis
f9dec36
@Luzhiyongg
Add flowRangeCorrUpc.cxx
86febfb 
TPC-FT0 correlations
@Luzhiyongg
Add workflows
ebf4471
@alibuild
Please consider the following formatting changes
3db2bdd
@Luzhiyongg
Merge pull request #25 from alibuild/alibot-cleanup-16081
aaf8aa8 
[PWGUD] Please consider the following formatting changes to AliceO2Group#16081
@Luzhiyongg
add header guard in UpcService.h
1ac0f05
@Luzhiyongg
remove unused tracks
8004063 
Removed unused UdTracksFull type alias and updated process function signature.
@Luzhiyongg
remove unused parameter and fix channel bit check
12f0276
@github-actions github-actions Bot added the pwgud label May 4, 2026
@github-actions github-actions Bot changed the title add true gap service and TPC-FT0 correlations [PWGUD] add true gap service and TPC-FT0 correlations May 4, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026 •
edited
Loading

O2 linter results: ❌ 1 errors, ⚠️ 14 warnings, 🔕 0 disabled

@alibuild alibuild mentioned this pull request May 4, 2026
Merged
@Luzhiyongg
Merge pull request #26 from alibuild/alibot-cleanup-16083
b5c095c 
[PWGUD] Please consider the following formatting changes to AliceO2Group#16083
rolavick
rolavick reviewed May 4, 2026
View reviewed changes
Copy link
Copy Markdown
Collaborator

@rolavick rolavick left a comment

Choose a reason for hiding this comment

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

Why do you need the UpcService? What is the usage of it?

@Luzhiyongg
Copy link
Copy Markdown
Contributor Author

Luzhiyongg commented May 4, 2026 •
edited
Loading

Why do you need the UpcService? What is the usage of it?

Hi roman,
For event-mixing, it's required to put the major event selection in the filter, otherwise the event-mixing pairs will include many unused pairs and significantly reduce the statistics.
We found this issue in pp and it's expected that the case will be more significant in UPC.

As the major event selection here are gapside and truegapside, I create a service task to produce truegapside so that I can use truegapside in the filter.
UpcService.h is the header defining truegapside output, and it's included by the upcTrueGapService.cxx and flowLongRangeCorrUpc.cxx

@alibuild
Copy link
Copy Markdown
Collaborator

alibuild commented May 5, 2026

Error while checking build/O2Physics/o2 for b5c095c at 2026-05-05 03:06:

No log files found

Full log here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rolavick rolavick rolavick left review comments

@alibuild alibuild Awaiting requested review from alibuild alibuild is a code owner

@amatyja amatyja Awaiting requested review from amatyja amatyja is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

pwgud

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Luzhiyongg @alibuild @rolavick