Skip to content

Fix building issues with NucleiTask#1034

Merged
njacazio merged 9 commits into
AliceO2Group:masterfrom
giovannimalfattore:master
Jul 21, 2022
Merged

Fix building issues with NucleiTask#1034
njacazio merged 9 commits into
AliceO2Group:masterfrom
giovannimalfattore:master

Conversation

@giovannimalfattore

Copy link
Copy Markdown
Contributor

No description provided.

@njacazio

Copy link
Copy Markdown
Collaborator

Now the formatting seems to be wrong

@giovannimalfattore

Copy link
Copy Markdown
Contributor Author

Now the formatting seems to be wrong

Fixed, hopefully

histos.add<TH1>("tracks/h1Eta", "pseudoRapidity; #eta; counts", HistType::kTH1F, {{200, -1.0, 1.0}});
histos.add<TH1>("tracks/h1VarPhi", "#phi; #phi; counts", HistType::kTH1F, {{80, -0.5, 7.5}});
histos.add<TH2>("tracks/h2EtaVsPhi", "#eta (TOF) vs #phi; #eta; #phi", HistType::kTH2F, {{200, -1.0, 1.0}, {80, -0.5, 7.5}});
histos.add<TH1>("tracks/h1VarPhi", "#phi; #phi; counts", HistType::kTH1F, {{63, 0.0, 6.3}});

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.

So this is the only change right?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, I just changed that line, but missing one bracket make everything "explode".

@njacazio njacazio enabled auto-merge (squash) July 21, 2022 09:21
@njacazio njacazio merged commit 78a1657 into AliceO2Group:master Jul 21, 2022
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