Skip to content

[PWGCF] feat: integrate angular histograms into PairHistManager#15973

Merged
ariedel-cern merged 3 commits intoAliceO2Group:masterfrom
Ptak07:angular-container
Apr 28, 2026
Merged

[PWGCF] feat: integrate angular histograms into PairHistManager#15973
ariedel-cern merged 3 commits intoAliceO2Group:masterfrom
Ptak07:angular-container

Conversation

@Ptak07
Copy link
Copy Markdown
Contributor

@Ptak07 Ptak07 commented Apr 27, 2026

No description provided.

@github-actions github-actions Bot added the pwgcf label Apr 27, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

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

Please consider the following formatting changes to AliceO2Group#15973
@ariedel-cern
Copy link
Copy Markdown
Collaborator

Hi @Ptak07,
integrating the histograms for angluar correlation like you did here seems good from my side.

@Ptak07 Ptak07 marked this pull request as ready for review April 28, 2026 13:16
@ariedel-cern ariedel-cern enabled auto-merge (squash) April 28, 2026 21:17
@ariedel-cern ariedel-cern merged commit c04e323 into AliceO2Group:master Apr 28, 2026
13 of 14 checks passed
#include "PWGCF/Femto/Core/histManager.h"
#include "PWGCF/Femto/Core/modes.h"

#include <Common/Core/RecoDecay.h>
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.

This is wrong. This is an O2Physics header.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for catching this. What is the correct include path, and what's the best way to submit the fix since the PR is merged?

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.

The path is correct. The format is wrong. I will include it in my fixes.

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.

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.

4 participants