Skip to content

Conversation

wenyaCern
Copy link

Overview

I've created a simple framework for calculating femtocorrelation functions with divided qₙ-vectors for Event-shape engineering of flow measurements, building on femtoUniverse (and incorporating small parts of femtodream). The analysis wants to estimate the relation between femtoRadii vs v2.

Why a New Framework

Key updates to the data model necessitated this:

  1. The collision table and track in FemtoDerived.h in Datamodel now includes q-vector information.
  2. The particle table has added PID checks using TOF-beta, improving particle species discrimination.
  3. More flow calculation functionalities will be added to the framework for the analysis.

Copy link

github-actions bot commented Aug 22, 2025

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

@github-actions github-actions bot changed the title framework for FemtoVsFlow [PWGCF] framework for FemtoVsFlow Aug 22, 2025
Please consider the following formatting changes to AliceO2Group#12690
@wenyaCern
Copy link
Author

fix formatting

@victor-gonzalez victor-gonzalez marked this pull request as draft August 22, 2025 17:57
@victor-gonzalez
Copy link
Collaborator

Please, keep it as draft until it gets reviewed/approved

Also, for the new files, follow linter and MegaLinter recommendations

The fact that linter and MegaLinter recommendations are not enforced is because that will require to stop any new change until the full O2Physcs code is adapted to those recommendations

But new files had to adhere to them. It is in the benefice of everyone

Thanks!!

@victor-gonzalez
Copy link
Collaborator

@wenyaCern, thanks for moving forward this functionality!
Are you aware that the femto data model is being redefined in order to have a single femto derived data format able to be used by all femto analyses?
Have you been in touch with the femto PAG coordinators for exposing your needs?
The aim is to have a single derived data production for femto analysis because storage space is a scarce resource and we cannot afford having multiple derived data productions unless it were really really necessary

@wenyaCern
Copy link
Author

Dear Victor,
Thanks a lot for your information and suggestion.

For this analysis, using the current femto-derived data format is impossible, as the analysis procedure in Task only reserves the filtered tracks by the table producer, while the calculation for flow requires inclusive hadrons. Events will be separated by the information of flow and that flow information will be transferred to analysis tasks. Considering the above reasons, I prefer to create a new data format by adding flow information in the data-driven, which will be isolated and not affect the current femto analysis.

I'm fixing the O2 linter and MegaLinter issues now. Hope you can consider my update and thanks a lot for that!

Best regards,
Wenya

@victor-gonzalez
Copy link
Collaborator

Dear Victor, Thanks a lot for your information and suggestion.

For this analysis, using the current femto-derived data format is impossible, as the analysis procedure in Task only reserves the filtered tracks by the table producer, while the calculation for flow requires inclusive hadrons. Events will be separated by the information of flow and that flow information will be transferred to analysis tasks. Considering the above reasons, I prefer to create a new data format by adding flow information in the data-driven, which will be isolated and not affect the current femto analysis.

I'm fixing the O2 linter and MegaLinter issues now. Hope you can consider my update and thanks a lot for that!

Best regards, Wenya

Dear Wenya,
I didn't ask you to use the current femto-derived data format. I know as today it is not feasible
What I asked you is if you have exposed your needs at the femto PAG and the people that is restructuring the femto derived data model for having such needs considered in the femto derived data data model (copying here @ariedel-cern and @lauraser)
Unfortunately each analysis cannot produce their own derived data because there is not enough storage space available for that

@wenyaCern
Copy link
Author

Dear Victor, Thanks a lot for your information and suggestion.
For this analysis, using the current femto-derived data format is impossible, as the analysis procedure in Task only reserves the filtered tracks by the table producer, while the calculation for flow requires inclusive hadrons. Events will be separated by the information of flow and that flow information will be transferred to analysis tasks. Considering the above reasons, I prefer to create a new data format by adding flow information in the data-driven, which will be isolated and not affect the current femto analysis.
I'm fixing the O2 linter and MegaLinter issues now. Hope you can consider my update and thanks a lot for that!
Best regards, Wenya

Dear Wenya, I didn't ask you to use the current femto-derived data format. I know as today it is not feasible What I asked you is if you have exposed your needs at the femto PAG and the people that is restructuring the femto derived data model for having such needs considered in the femto derived data data model (copying here @ariedel-cern and @lauraser) Unfortunately each analysis cannot produce their own derived data because there is not enough storage space available for that

Dear Victor,
Thanks for your suggestions! I'll discuss with Femto PAG for the function in the redefined data format of femto in the future.

[PWGCF] Please consider the following formatting changes to AliceO2Group#12690
@wenyaCern wenyaCern marked this pull request as ready for review August 27, 2025 20:13
@wenyaCern
Copy link
Author

wenyaCern commented Aug 27, 2025

The O2 linter errors have been fixed.
Rest O2 linter errors are in CMakeLists.txt, reviewing of workflow naming.

The derived data contains the qn bin information, which is the only thing changed from the femtoUniverse framework.

@shouqiye shouqiye enabled auto-merge (squash) August 28, 2025 07:35
@victor-gonzalez victor-gonzalez marked this pull request as draft August 28, 2025 09:04
auto-merge was automatically disabled August 28, 2025 09:04

Pull request was converted to draft

@wenyaCern wenyaCern marked this pull request as ready for review August 28, 2025 09:06
@victor-gonzalez
Copy link
Collaborator

@wenyaCern @shouqiye This is still not ready for being merged
I am in contact with @ariedel-cern on how to proceed with it
It changes the femto derived data model and require additional derived data which can only be used by this analysis

@victor-gonzalez victor-gonzalez marked this pull request as draft August 28, 2025 09:07
@victor-gonzalez
Copy link
Collaborator

Please, keep it as draft for the time being

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