Skip to content

[Common] Add run 3 hyperfine percentiling#16275

Merged
ddobrigk merged 4 commits into
AliceO2Group:masterfrom
ddobrigk:multdev01
May 16, 2026
Merged

[Common] Add run 3 hyperfine percentiling#16275
ddobrigk merged 4 commits into
AliceO2Group:masterfrom
ddobrigk:multdev01

Conversation

@ddobrigk
Copy link
Copy Markdown
Collaborator

@ddobrigk ddobrigk commented May 16, 2026

  • adds an adaptive percentiling setting in which 0.00001% is viable, which should be fine in any run with over 10 million events. Based on inspecting the typical signals, a very high multiplicity selection is likely to be fine with midrapidity counters but caution should be exercised with FIT signals, as these have a tail that is likely beam-background or otherwise spurious: QA is always needed!
  • adds sanity checks to capture problematic calibration situations and issue an invalid (but mathematically functional) calibration in that case. Checks include excessively fine binning requested for the input information as well as situations in which the detector signal is zero (e.g. no MFT in datataking).
  • N.B.: the calibrator that is touched in this PR is legacy code (it's very old...) and thus spawns several linter warnings: will be fixed in a future PR.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 16, 2026

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

@ddobrigk ddobrigk marked this pull request as ready for review May 16, 2026 20:13
@ddobrigk ddobrigk enabled auto-merge (squash) May 16, 2026 20:14
Copy link
Copy Markdown
Collaborator

@alibuild alibuild left a comment

Choose a reason for hiding this comment

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

Auto-approving on behalf of @ddobrigk.

@ddobrigk ddobrigk merged commit d41332d into AliceO2Group:master May 16, 2026
20 of 23 checks passed
@ddobrigk ddobrigk deleted the multdev01 branch May 16, 2026 21:42
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