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

MID: check the filtered digits in the async QC #1894

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dstocco
Copy link
Contributor

@dstocco dstocco commented Feb 4, 2025

This PR allows for a better comparison between data and MC.
The masks are applied during reconstruction for both data and MC. But data are already masked since this is done at the electronics level, while MC is masked only during reconstruction.
So far the QC checked for digits before the masks are applied, which leads to a difference between data and MC.
This PR introduces a check on digits after the masks are applied, so that we can compare apples to apples.

Copy link

github-actions bot commented Feb 4, 2025

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2023-pbpb-apass4
async-2023-pp-apass4
async-2024-pp-apass1
async-2022-pp-apass7
async-2024-pp-cpass0
async-2024-PbPb-apass1
async-2024-ppRef-apass1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant