Skip to content

[PWGJE] Adding centrality dependency for recoil jets#16611

Merged
nzardosh merged 9 commits into
AliceO2Group:masterfrom
KotliarovAr:analysis-update
Jun 10, 2026
Merged

[PWGJE] Adding centrality dependency for recoil jets#16611
nzardosh merged 9 commits into
AliceO2Group:masterfrom
KotliarovAr:analysis-update

Conversation

@KotliarovAr

Copy link
Copy Markdown
Contributor

Main update:

  • add dependency on centrality for recoil jet analysis
  • use 4 definitions of centrality: FT0A, FT0C, FT0M, and CentFT0CVar1
  • centrality FT0C and FT0M are main, the rest will be used only for QA studies
  • new process function to analyse the tracking efficiency as a function of EA and Centrality

Other updates:

  • fixed the issues when the same particle level jet could be filled in the "Miss" histogram several times

@github-actions

Copy link
Copy Markdown

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

@github-actions github-actions Bot changed the title Analysis update [PWGJE] Analysis update Jun 10, 2026
@KotliarovAr

Copy link
Copy Markdown
Contributor Author

Dear @vkucera,

The MegaLinter job failed because the Docker build attempted to pull the alpine:latest image from Docker Hub, but encountered I/O timeout errors while connecting to the registry.

Unfortunately, I am unable to rerun the job myself. Would it be possible to proceed with the PR as is, or does this issue need to be resolved first?

Thanks in advance for your advice.

@KotliarovAr

Copy link
Copy Markdown
Contributor Author

Dear @nzardosh

I have already written to Vit asking a question about the MegaLinter issue. I see that all other checks were passed. I don't know whether I should wait for the issue with MegaLinter to be fixed or not. If not, could you approve the PR?

@vkucera vkucera left a comment

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.

Your PR title is one of the most vague ones possible. Please follow the contribution guidelines.

@vkucera

vkucera commented Jun 10, 2026

Copy link
Copy Markdown
Collaborator

Dear @nzardosh

I have already written to Vit asking a question about the MegaLinter issue. I see that all other checks were passed. I don't know whether I should wait for the issue with MegaLinter to be fixed or not. If not, could you approve the PR?

Fixed

@nzardosh nzardosh changed the title [PWGJE] Analysis update [PWGJE] Adding centrality dependency for recoil jets Jun 10, 2026
@nzardosh nzardosh requested a review from vkucera June 10, 2026 15:40
@nzardosh nzardosh enabled auto-merge (squash) June 10, 2026 15:40
@nzardosh nzardosh dismissed vkucera’s stale review June 10, 2026 15:41

changed the title

@nzardosh nzardosh merged commit fe9194c into AliceO2Group:master Jun 10, 2026
17 of 18 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.

3 participants