Skip to content

[PWGCF] New task for FT0 di-hadron correlation with the main focus on Nch dependence#15852

Merged
wenyaCern merged 2 commits intoAliceO2Group:masterfrom
Thorkj:master
Apr 23, 2026
Merged

[PWGCF] New task for FT0 di-hadron correlation with the main focus on Nch dependence#15852
wenyaCern merged 2 commits intoAliceO2Group:masterfrom
Thorkj:master

Conversation

@Thorkj
Copy link
Copy Markdown
Contributor

@Thorkj Thorkj commented Apr 17, 2026

No description provided.

@github-actions github-actions Bot added the pwgcf label Apr 17, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 17, 2026

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

@github-actions github-actions Bot changed the title New task for FT0 di-hadron correlation with the main focus on Nch dependence [PWGCF] New task for FT0 di-hadron correlation with the main focus on Nch dependence Apr 17, 2026
@Thorkj
Copy link
Copy Markdown
Contributor Author

Thorkj commented Apr 17, 2026

From what I can see of the errors from the O2 linter is from old code names in the cmake list.

@Thorkj Thorkj marked this pull request as ready for review April 17, 2026 23:48
@wenyaCern
Copy link
Copy Markdown
Collaborator

Dear @Thorkj
Since the code is a new framework, I need time to review it, so approval might be a bit slow.

@Thorkj
Copy link
Copy Markdown
Contributor Author

Thorkj commented Apr 22, 2026

Dear @wenyaCern,
The O2Linter error above is not related to the new task, it is from old file names in the cMakeList.

PUBLIC_LINK_LIBRARIES O2Physics::AnalysisCore
COMPONENT_NAME Analysis)

o2physics_add_dpl_workflow(lambdacorr
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.

Dear @Thorkj
Could you please modify the name as the introduction from O2-linter? It would be better if you use the camelCase for the name

@wenyaCern wenyaCern merged commit 28a8d2b into AliceO2Group:master Apr 23, 2026
11 of 13 checks passed
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.

2 participants