Skip to content

[PWGCF] Improvements on efficiency correction - #18120

Merged
abilandz merged 2 commits into
AliceO2Group:masterfrom
meiALICE:master
Oct 1, 2026
Merged

abilandz merged 2 commits into
AliceO2Group:masterfrom
meiALICE:master

Conversation

@meiALICE

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the pwgcf label Sep 30, 2026
@github-actions github-actions Bot changed the title Improvements on efficiency correction [PWGCF] Improvements on efficiency correction Sep 30, 2026
@github-actions

Copy link
Copy Markdown

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

@abilandz
abilandz enabled auto-merge (squash) September 30, 2026 13:57
@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/staging for 33c32ba at 2026-09-30 17:41:

No log files found

Full log here.

@alibuild

Copy link
Copy Markdown
Collaborator

Error while checking build/O2Physics/o2 for 33c32ba at 2026-09-30 17:43:

No log files found

Full log here.

@abilandz

Copy link
Copy Markdown
Collaborator

@meiALICE : This is another failure unrelated to your code, as always, we alert in cases like this @vkucera

@abilandz
abilandz requested a review from vkucera September 30, 2026 15:54
@vkucera

vkucera commented Sep 30, 2026

Copy link
Copy Markdown
Collaborator

@meiALICE : This is another failure unrelated to your code, as always, we alert in cases like this @vkucera

Hi @abilandz , there is no need to alert me in these cases. The "Git fetch failed" failures are caused by problems with a third-party repository which is not under our control and the tests recover automatically when the repository becomes available again, as explained in the error message.

@abilandz

Copy link
Copy Markdown
Collaborator

@meiALICE : This is another failure unrelated to your code, as always, we alert in cases like this @vkucera

Hi @abilandz , there is no need to alert me in these cases. The "Git fetch failed" failures are caused by problems with a third-party repository which is not under our control and the tests recover automatically when the repository becomes available again, as explained in the error message.

@vkucera : Thanks for your feedback - I wanted to explicitly get your confirmation that the tests will recover automatically. But please nevertheless keep an eye on this PR, as still nothing has happened. Thanks!

@abilandz
abilandz merged commit e86c4d7 into AliceO2Group:master Oct 1, 2026
14 of 15 checks passed
VDiBella pushed a commit to VDiBella/O2Physics that referenced this pull request Oct 2, 2026
Co-authored-by: Yuanjun Mei <yuanjun.mei@cern.ch>
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.

4 participants