[PWGEM,Photon] Add option to correct eta for vertex shift in event mi… - #17769
Merged
Conversation
…xing In event mixing two collisions with slightly different z vertex position can be used. Since clusters are corrected for the vertex position the possible eta values for single cell clusters depend on the z vertex position. In same event combinations these possible discrete values are the same resulting in discrete combined eta values. In mixed event without correcting for this the eta position of clusters from collion A and clusters from collision B can have different discrete values resuting in different discrete combined values. This new option aims to make eta values from mixed event pairs look more similar to same event pairs by shifting the eta value of all clusters from collision B according to the verte position of collision A.
mhemmer-cern
requested review from
alibuild,
amaringarcia,
dsekihat,
hscheid,
jokonig and
reygers
as code owners
September 4, 2026 12:21
|
O2 linter results: ❌ 0 errors, |
mhemmer-cern
enabled auto-merge (squash)
September 4, 2026 18:22
alibuild
approved these changes
Sep 4, 2026
alibuild
left a comment
alibuild
left a comment
Collaborator
There was a problem hiding this comment.
Auto-approving on behalf of @mhemmer-cern.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…xing
In event mixing two collisions with slightly different z vertex position can be used. Since clusters are corrected for the vertex position the possible eta values for single cell clusters depend on the z vertex position. In same event combinations these possible discrete values are the same resulting in discrete combined eta values. In mixed event without correcting for this the eta position of clusters from collion A and clusters from collision B can have different discrete values resuting in different discrete combined values. This new option aims to make eta values from mixed event pairs look more similar to same event pairs by shifting the eta value of all clusters from collision B according to the verte position of collision A.