Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PWGCF - Calibration study of ZDC SP for charge dependent flow studies #8136

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

cnkoster
Copy link
Contributor

For the analysis of v1 these calibrations are needed to get the correct spectator plane.
The task creates derived data to enable offline analysis and creation of the calibration histograms.

@github-actions github-actions bot added the pwgcf label Oct 24, 2024
@victor-gonzalez
Copy link
Collaborator

The code as is now, produces a table, but so far there are not subscribers to that table
What is the aim, to use the produced table as derived data to do some post-processing? If that is the case, I would suggest to put the data model within the own task source file
If the aim is to produce a table whose data will be used by other task/s then I would suggest moving the task source file to the TableProducer directory

Regarding the table as such, it is really needed to have the run number for each collision?

@cnkoster
Copy link
Contributor Author

For now table is indeed needed for post-processing. At a later stage the output will be used by another task to calculate v1. So I could move inside the task for now and extend it to the table producer at a later stage, or would it be better to move it there directly? For the post processing, al q-vectors need to be analysed on a run-by-run basis and it seemed convenient to add the run numbers to the table.

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