Skip to content

Conversation

@IsakovAD
Copy link
Collaborator

  • Increased ranges for some cluster size plots
  • Changed Type from TH1D to TH1L to avoid saturation of cluster size plots

@IsakovAD IsakovAD requested a review from iravasen as a code owner January 20, 2025 14:57
}

hClusterSizeLayerSummary[iLayer] = new TH1D(Form("Layer%d/AverageClusterSizeSummary", iLayer), Form("Layer%dAverageClusterSizeSummary", iLayer), 100, 0, 100);
hClusterSizeLayerSummary[iLayer] = new TH1L(Form("Layer%d/AverageClusterSizeSummary", iLayer), Form("Layer%dAverageClusterSizeSummary", iLayer), 1000, 0, 1000);
Copy link
Collaborator

Choose a reason for hiding this comment

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

would it be possible to set the maximum to 128*128?

hClusterSizeLayerSummary[iLayer]->SetStats(0);

hGroupedClusterSizeLayerSummary[iLayer] = new TH1D(Form("Layer%d/AverageGroupedClusterSizeSummary", iLayer), Form("Layer%dAverageGroupedClusterSizeSummary", iLayer), 100, 0, 100);
hGroupedClusterSizeLayerSummary[iLayer] = new TH1L(Form("Layer%d/AverageGroupedClusterSizeSummary", iLayer), Form("Layer%dAverageGroupedClusterSizeSummary", iLayer), 1000, 0, 1000);
Copy link
Collaborator

Choose a reason for hiding this comment

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

same here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Hello Ivan,

Sure. Fixed in the next commit

@iravasen
Copy link
Collaborator

Thank you! Approved.

@iravasen iravasen enabled auto-merge (squash) January 21, 2025 08:01
@IsakovAD IsakovAD changed the title [WIP] ITS/Clusters: increased ranges of some plots ITS/Clusters: increased ranges of some plots Jan 28, 2025
@iravasen
Copy link
Collaborator

@Barthelemy also this one can be merged.

@Barthelemy Barthelemy disabled auto-merge February 11, 2025 11:17
@Barthelemy Barthelemy merged commit 699425c into AliceO2Group:master Feb 11, 2025
5 of 6 checks passed
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