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

Update MFT json files: 'onlineQC' flag in Cluster task + grpGeomRequest #1541

Merged
merged 3 commits into from
Mar 21, 2024

Conversation

grundd
Copy link
Contributor

@grundd grundd commented Mar 21, 2024

Update the MFT json files in:

  • DATA/production/qc-async/ ... onlineQC = 0
  • MC/config/QC/json/ ... onlineQC = 0
  • DATA/production/qc-sync/ ... onlineQC = 1

grpGeomRequest settings modified everywhere to normalize the QC objects to the # orbits
Relates to AliceO2Group/QualityControl#2198

Copy link

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-2022-pp-apass4
async-2023-pbpb-apass
async-2023-pp-apass1
async-data
async-mc
async-2022-pp-apass6

@benedikt-voelkel
Copy link
Contributor

@JianLIUhep from my side it looks good! But I cannot judge the changes in QC. Could you take a look?
Thanks a lot!

Note, that the CI failure is not related to this PR. There might be a genuine problem in one of the analyses that were tested. I will check.

@JianLIUhep
Copy link
Contributor

Hi @benedikt-voelkel, yes, it looks good to me as well.

Thanks @grundd for opening this PR.

@benedikt-voelkel benedikt-voelkel merged commit 2ce7d32 into AliceO2Group:master Mar 21, 2024
6 of 7 checks passed
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.

3 participants