Skip to content

[PWGDQ] Implementation of Transverse Spherocity in PWGDQ - #18299

Draft
akavumpa wants to merge 4 commits into
AliceO2Group:masterfrom
akavumpa:Sphero1
Draft

akavumpa wants to merge 4 commits into
AliceO2Group:masterfrom
akavumpa:Sphero1

Conversation

@akavumpa

Copy link
Copy Markdown

We have implemented Transverse Spherocity in PWGDQ to analyse the J/Psi production cross-section as a function of Spherocity in the forward rapidity.

@github-actions github-actions Bot changed the title Implementation of Transverse Spherocity in PWGDQ [PWGDQ] Implementation of Transverse Spherocity in PWGDQ Oct 14, 2026
@github-actions

Copy link
Copy Markdown

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

@lucamicheletti93

Copy link
Copy Markdown
Contributor

Dear @akavumpa , this PR looks very interesting but I was thinking that instread of adding this function for the spherocity calculation in the varManger, you could also think to create a service wagon and so that it is somewhat centralized and easier to manage by the analyzers. I think we can discuss this in the next days also with @axti98

@vkucera vkucera 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.

@akavumpa Please pay attention to what you commit. Your PR is adding a zip file.

@akavumpa

Copy link
Copy Markdown
Author

@akavumpa Please pay attention to what you commit. Your PR is adding a zip file.

Thank you for pointing this out. The zip file that was accidentally placed has now been removed.

@lucamicheletti93

Copy link
Copy Markdown
Contributor

Dear @akavumpa , before moving this draft to PR we should check a bit if there is a better way to impelment this part (DQ service wagon for instance). Did you already give a try?

@akavumpa

Copy link
Copy Markdown
Author

Dear @akavumpa , before moving this draft to PR we should check a bit if there is a better way to impelment this part (DQ service wagon for instance). Did you already give a try?

Dear @lucamicheletti93 ,
I am currently trying to implement your suggestion of introducing a service wagon for the same. I shall update you as soon as it starts working locally.

@axti98

axti98 commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Dear @akavumpa, thanks for this PR. Before you start developing in the blue I think we should briefly discuss on what is actually needed and how we should implement this. Since there are several event shape related observables (Spherocity, Sphericity, Flattenicity, ...) we could do in one central task. Based on this we can then deduce a DQ implementation.

Also, the way you propose it here, by adding the tables to the existing ones, previously skimmed data would become uselesse. Also just adding a new task might introduce problems with memory consumption.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants