Skip to content

[PWGJE] Adding new Angularity Task to PWGJE - #17299

Draft
Rajdeep-nandi wants to merge 5 commits into
AliceO2Group:masterfrom
Rajdeep-nandi:master
Draft

[PWGJE] Adding new Angularity Task to PWGJE#17299
Rajdeep-nandi wants to merge 5 commits into
AliceO2Group:masterfrom
Rajdeep-nandi:master

Conversation

@Rajdeep-nandi

Copy link
Copy Markdown

This Task calculates the Angularity for D0 particles.

@github-actions github-actions Bot added the pwgje label Aug 3, 2026
@github-actions github-actions Bot changed the title Adding new Angularity Task to PWGJE [PWGJE] Adding new Angularity Task to PWGJE Aug 3, 2026
@vkucera

vkucera commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

@Rajdeep-nandi Please follow the contribution guidelines and make sure your PR passes the tests before you open it for review.

#include "PWGJE/DataModel/Jet.h"
#include "PWGJE/DataModel/JetReducedData.h"
#include "PWGJE/DataModel/JetReducedDataHF.h"
#include "PWGJE/DataModel/JetSubstructure.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.

you seem to have alot of includes which i am not sure are needed?

@vkucera vkucera Aug 3, 2026

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.

When you look at the output of the code-check, you will be sure. ;-)

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.

Will the code check run if I dont approve? It seems to be waiting for approval

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.

No it won't, as neither build check would. It requires your approval but also a valid PR that can be merged, which is currently not the case.

@github-actions

github-actions Bot commented Aug 4, 2026

Copy link
Copy Markdown

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

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