Skip to content

QC fixes#909

Merged
Barthelemy merged 3 commits into
AliceO2Group:masterfrom
chiarazampolli:QCfixes
Oct 19, 2021
Merged

QC fixes#909
Barthelemy merged 3 commits into
AliceO2Group:masterfrom
chiarazampolli:QCfixes

Conversation

@chiarazampolli
Copy link
Copy Markdown
Contributor

@chiarazampolli
Copy link
Copy Markdown
Contributor Author

Hi,
I just realized that TEfficiency is not visualized by the QCG, so this might not be the right way...
Chiara

@chiarazampolli
Copy link
Copy Markdown
Contributor Author

Hi, I just realized that TEfficiency is not visualized by the QCG, so this might not be the right way... Chiara

Fixed. The order for numerator and denominator matter.
One open point, though, is the errors, which i need to check. TEfficiency has a low and up error, and without setting anything special for the statistics, I got:

[8651:QC-TASK-RUNNER-Vertexing_MC]: 2021-10-15 17:27:44.814631     ibin = 1, mNPrimaryMCEvWithVtx->GetBinContent(ibin + 1) = 17, mNPrimaryMCGen->GetBinContent(ibin + 1) = 281, efficiency = 0.060498220640569395
[8651:QC-TASK-RUNNER-Vertexing_MC]: 2021-10-15 17:27:44.814714     ibin = 1, mNPrimaryMCEvWithVtx->GetBinError(ibin + 1) = 4.1231056256176606, mNPrimaryMCGen->GetBinError(ibin + 1) = 16.763054614240211, efficiency error low = 0.014269771453455919, efficiency error up = 0.017781592794800988
[8651:QC-TASK-RUNNER-Vertexing_MC]: 2021-10-15 17:27:44.814723     ibin = 2, mNPrimaryMCEvWithVtx->GetBinContent(ibin + 1) = 161, mNPrimaryMCGen->GetBinContent(ibin + 1) = 439, efficiency = 0.36674259681093396
[8651:QC-TASK-RUNNER-Vertexing_MC]: 2021-10-15 17:27:44.814735     ibin = 2, mNPrimaryMCEvWithVtx->GetBinError(ibin + 1) = 12.68857754044952, mNPrimaryMCGen->GetBinError(ibin + 1) = 20.952326839756964, efficiency error low = 0.023793900883057717, efficiency error up = 0.024430153978088953

@chiarazampolli
Copy link
Copy Markdown
Contributor Author

Errors on o2-cs8 don't seem related to this PR.

@Barthelemy
Copy link
Copy Markdown
Collaborator

indeed, merging.

@Barthelemy Barthelemy merged commit 6014712 into AliceO2Group:master Oct 19, 2021
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.

2 participants