Skip to content

QC offline tracks + clusters task#978

Merged
Barthelemy merged 4 commits into
AliceO2Group:masterfrom
KotliarovAr:QC_Offline_Task
Dec 13, 2021
Merged

QC offline tracks + clusters task#978
Barthelemy merged 4 commits into
AliceO2Group:masterfrom
KotliarovAr:QC_Offline_Task

Conversation

@KotliarovAr
Copy link
Copy Markdown
Contributor

Task includes two executable files which open and read ROOT files with the TTree objects. Extracted data are distributed to the workflow.

@iravasen
Copy link
Copy Markdown
Collaborator

iravasen commented Dec 3, 2021

I can revise the PR for what concerns ITS functionalities.

@KotliarovAr KotliarovAr changed the title QC offline tracks + clusters task [WIP] QC offline tracks + clusters task Dec 3, 2021
@iravasen
Copy link
Copy Markdown
Collaborator

iravasen commented Dec 8, 2021

Hi @KotliarovAr, thank you! Very good piece of work. I think this is fine. One question for @Barthelemy or @knopers8: this PR contains new files, any naming convention of the files or other standards that we should follow? Thanks

@Barthelemy
Copy link
Copy Markdown
Collaborator

Hi,
We follow these conventions: https://rawgit.com/AliceO2Group/CodingGuidelines/master/naming_formatting.html
We also use the prefix run as you noted and used.

The name of your files seem good.

@iravasen
Copy link
Copy Markdown
Collaborator

iravasen commented Dec 9, 2021

Thank you, then since the code structure + variable names are taken from ITSClusterTask.cxx and ITSTrackTask.cxx (already in QC since long time), I think you can merge this PR unless you have further comments/suggestions. Thanks!

@KotliarovAr KotliarovAr changed the title [WIP] QC offline tracks + clusters task QC offline tracks + clusters task Dec 9, 2021
@Barthelemy Barthelemy merged commit d610eac into AliceO2Group:master Dec 13, 2021
Barthelemy pushed a commit to Barthelemy/QualityControl that referenced this pull request Jan 13, 2022
* Changed_CMake

* Code to run offline task with Cluster Root file

* Modified CMake file: add new executable

* QC offline tracks + clusters task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants