Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ITS] Anomalous clusters plots #2482

Merged
merged 6 commits into from
Nov 28, 2024

Conversation

nicolovalle
Copy link
Contributor

@nicolovalle nicolovalle commented Nov 21, 2024

This PR adds two sets of 2D plots published by the cluster task:

  • Number of long clusters vs chip ID (3 plots, for the IB layers)
  • Number of hits as sum of the clusters size for those events with at least one long cluster, vs chip ID (3 plots, for the IB layers)

Tested locally

@nicolovalle
Copy link
Contributor Author

cc @IsakovAD @iravasen

Copy link
Collaborator

@IsakovAD IsakovAD left a comment

Choose a reason for hiding this comment

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

Hello @nicolovalle,

Thank you very much for such prompt implementation. I think that it is better to keep old way how we calculated information about chip, or I am missing something?

@nicolovalle
Copy link
Contributor Author

Hello @nicolovalle,

Thank you very much for such prompt implementation. I think that it is better to keep old way how we calculated information about chip, or I am missing something?

Hi, I left a comment inline.. to my understanding that was un unused variable

@knopers8 knopers8 merged commit 2603c31 into AliceO2Group:master Nov 28, 2024
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