[PWGHF] treeCreatorDstarSpinAlignMixing: Adding new task for local mix candidate analysis in Dstar spin alignment measurement.#16721
Conversation
|
O2 linter results: ❌ 0 errors, |
Please consider the following formatting changes to AliceO2Group#16721
| PUBLIC_LINK_LIBRARIES O2Physics::AnalysisCore O2Physics::EventFilteringUtils | ||
| COMPONENT_NAME Analysis) | ||
|
|
||
| o2physics_add_dpl_workflow(task-mixing-dstar-cand-tree-creator |
There was a problem hiding this comment.
Please respect the alphabetical ordering.
fgrosa
left a comment
There was a problem hiding this comment.
Hi @Mingze129 thanks a lot! The implementation looks good to me, I just have a few minor comments (see below)
|
@Mingze129 can you please clarify what this workflow is supposed to produce? From the name and path, it is not clear whether it's a task, a candidate creator, or a tree creator. |
@vkucera Thank you for you comments on this draft. I need this task to produce derived data for local mixed-event analysis. I marked it as Draft mainly for two reasons. First, I would like to develop it together with Fabrizio. Second, I am not yet sure how this script should be named, where it should be placed in the repository, or whether we will eventually want to add a complete online mixed-event analysis workflow in the future. My original plan was to discuss these details with Fabrizio after the script was completed and had passed the tests, and then change the status to Ready. Since you have raised the question now, I look forward to hearing your suggestions and would greatly appreciate any guidance you can offer. |
Hi @Mingze129, I think you can rename it |
Thanks @Mingze129 @fgrosa . That sounds reasonable. |
Add a new task to obtain the candidates information required for local mixed-event analysis.