Skip to content

[PWGCF] Fix q-vector and event plane calculation in femto frame work - #17931

Merged
ariedel-cern merged 4 commits into
AliceO2Group:masterfrom
ariedel-cern:master
Sep 17, 2026
Merged

ariedel-cern merged 4 commits into
AliceO2Group:masterfrom
ariedel-cern:master

Conversation

@ariedel-cern

@ariedel-cern ariedel-cern commented Sep 16, 2026

Copy link
Copy Markdown
Collaborator
  • Use the configured harmonic for |q_n| (previously always used n=2)
  • Allow the event plane angle and the q-vector to come from different detectors
  • Guard against out-of-range harmonic indices and non-positive FT0 amplitude sums
  • Fix histogram axes for event shape variables

@github-actions github-actions Bot added the pwgcf label Sep 16, 2026
@github-actions

Copy link
Copy Markdown

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

@ariedel-cern
ariedel-cern marked this pull request as ready for review September 16, 2026 11:09
@ariedel-cern
ariedel-cern enabled auto-merge (squash) September 16, 2026 11:09
alibuild
alibuild previously approved these changes Sep 16, 2026

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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 @ariedel-cern.

@ariedel-cern
ariedel-cern marked this pull request as draft September 16, 2026 11:29
auto-merge was automatically disabled September 16, 2026 11:29

Pull request was converted to draft

@ariedel-cern ariedel-cern changed the title [PWGCF] Fix histogram axis in femto framework [PWGCF] Fix q-vector and event plane calculation in femto frame work Sep 16, 2026
[PWGCF] Please consider the following formatting changes to AliceO2Group#17931
@ariedel-cern
ariedel-cern marked this pull request as ready for review September 16, 2026 12:16
@ariedel-cern
ariedel-cern enabled auto-merge (squash) September 17, 2026 07:17

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

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 @ariedel-cern.

@ariedel-cern
ariedel-cern merged commit 7ec2d8c into AliceO2Group:master Sep 17, 2026
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.

3 participants