Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[PWGEM,PWGEM-36] EMCal rotation background add SM edges #8800

Merged
merged 5 commits into from
Dec 4, 2024
Merged

[PWGEM,PWGEM-36] EMCal rotation background add SM edges #8800

merged 5 commits into from
Dec 4, 2024
+272 −125

Conversation

mhemmer-cern
Copy link
Collaborator

  • Add the option to exclude clusters that are rotated to the edge of a SM where we normally do not have clusters in EMCal. This should help describe the invariant mass shape espacially at low pT for Pi0 and at mid pT for eta.
  • Add resolution correction via ccdb object to flow task
  • Add sinus and cosine coefficient to check possible higher order contributions to event plane

@mhemmer-cern
[PWGEM],[PWGEM-36] EMCal rotation background add SM edges
e8acc8f 
- Add the option to exclude clusters that are rotated to the edge of a SM where we normally do not have clusters in EMCal. This should help describe the invariant mass shape espacially at low pT for Pi0 and at mid pT for eta.
@github-actions github-actions bot added the pwgem label Dec 3, 2024
@alibuild alibuild mentioned this pull request Dec 3, 2024
Merged
@mhemmer-cern
Merge pull request #9 from alibuild/alibot-cleanup-8800
fea39c8 
Please consider the following formatting changes to AliceO2Group#8800
@mhemmer-cern mhemmer-cern enabled auto-merge (squash) December 3, 2024 15:45
alibuild
alibuild approved these changes Dec 3, 2024
View reviewed changes
Copy link
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 @mhemmer-cern.

@mhemmer-cern mhemmer-cern merged commit e4ab16c into AliceO2Group:master Dec 4, 2024
11 of 14 checks passed
@mhemmer-cern mhemmer-cern deleted the EMCAL-670 branch December 4, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alibuild alibuild alibuild approved these changes

@mikesas mikesas Awaiting requested review from mikesas mikesas is a code owner

@rbailhac rbailhac Awaiting requested review from rbailhac rbailhac is a code owner

@m-c-danisch m-c-danisch Awaiting requested review from m-c-danisch m-c-danisch is a code owner

@novitzky novitzky Awaiting requested review from novitzky novitzky is a code owner

@dsekihat dsekihat Awaiting requested review from dsekihat dsekihat is a code owner

Assignees
No one assigned
Labels
pwgem
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mhemmer-cern @alibuild