Skip to content

[PWGDQ] Change some code for the energy correlator study#15937

Merged
mcoquet642 merged 46 commits intoAliceO2Group:masterfrom
YazhenLin:master
Apr 23, 2026
Merged

[PWGDQ] Change some code for the energy correlator study#15937
mcoquet642 merged 46 commits intoAliceO2Group:masterfrom
YazhenLin:master

Conversation

@YazhenLin
Copy link
Copy Markdown
Contributor

@YazhenLin YazhenLin commented Apr 23, 2026

By default, the TH2::Interpolate(x, y) method uses bilinear interpolation. This algorithm calculates the weighted average based on the four nearest bin centers.
When the input coordinates (x,y) fall outside the range of the first or last bin centers, the function attempts to interpolate between the edge bin and the Underflow/Overflow bins. Since these bins are often empty (value = 0), the resulting weight drops sharply toward zero, which is physically incorrect for efficiency/acceptance corrections.
To prevent these, I keep the input coordinates to the safe range defined by the centers of the first and last bins. It does not alter the interpolated values for points within the internal range, it simply ensures that points beyond the boundary bin centers returns to the edge values.

@github-actions github-actions Bot added the pwgdq label Apr 23, 2026
@YazhenLin YazhenLin marked this pull request as ready for review April 23, 2026 12:23
@github-actions github-actions Bot changed the title Change some code for the energy correlator study [PWGDQ] Change some code for the energy correlator study Apr 23, 2026
@github-actions
Copy link
Copy Markdown

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

@mcoquet642
Copy link
Copy Markdown
Collaborator

Please add some description to explain what you are changing in your PRs

@YazhenLin
Copy link
Copy Markdown
Contributor Author

Please add some description to explain what you are changing in your PRs

Hi @mcoquet642, I add some description now. Thank you!

@mcoquet642 mcoquet642 merged commit 21bac9e into AliceO2Group:master Apr 23, 2026
12 of 15 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