Skip to content

PWGHF: Add histogram on vertex covariance matrix to QA task#4883

Merged
jgrosseo merged 1 commit into
AliceO2Group:devfrom
njacazio:dev
Nov 22, 2020
Merged

PWGHF: Add histogram on vertex covariance matrix to QA task#4883
jgrosseo merged 1 commit into
AliceO2Group:devfrom
njacazio:dev

Conversation

@njacazio

Copy link
Copy Markdown
Collaborator

No description provided.

@njacazio njacazio requested a review from ginnocen as a code owner November 20, 2020 16:27
@ginnocen ginnocen changed the title Add histogram on vertex covariance matrix to QA task PWGHF: DO NOT MERGE YET Add histogram on vertex covariance matrix to QA task Nov 20, 2020
@ginnocen ginnocen changed the title PWGHF: DO NOT MERGE YET Add histogram on vertex covariance matrix to QA task PWGHF: Add histogram on vertex covariance matrix to QA task Nov 20, 2020
Comment on lines +189 to +194
histograms.add("Covariance/xx", "xx;Cov_{xx} [cm]", kTH1D, {{200, -0.1, 0.1}});
histograms.add("Covariance/xy", "xy;Cov_{xy} [cm]", kTH1D, {{200, -0.1, 0.1}});
histograms.add("Covariance/xz", "xz;Cov_{xz} [cm]", kTH1D, {{200, -0.1, 0.1}});
histograms.add("Covariance/yy", "yy;Cov_{yy} [cm]", kTH1D, {{200, -0.1, 0.1}});
histograms.add("Covariance/yz", "yz;Cov_{yz} [cm]", kTH1D, {{200, -0.1, 0.1}});
histograms.add("Covariance/zz", "zz;Cov_{zz} [cm]", kTH1D, {{200, -0.1, 0.1}});

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The units are cm^{2} not cm.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True, I missed it when copying the lines, I will fix this in the next PR

@ginnocen ginnocen self-requested a review November 21, 2020 11:29
@jgrosseo jgrosseo merged commit f47c6ea into AliceO2Group:dev Nov 22, 2020
EmilGorm pushed a commit to EmilGorm/AliceO2 that referenced this pull request Nov 22, 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.

4 participants