-
Notifications
You must be signed in to change notification settings - Fork 180
Add o2-analysis-time-dependent-qa to the suite of AnalysisQC tasks #1959
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
Conversation
o2-analysis-time-dependent-qa: some histograms that would be better to suppress (they will be heavy for long runs)
Include o2-analysis-time-dependent-qa
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
+async-label async-2023-PbPb-apass5, async-2024-PbPb-apass2 |
|
Ciao @alcaliva, This is the time dependent AnalysisQC from Igor. It would be nice to include it for the upcoming pass. Yours, Catalin. |
|
Hi @sawenzel, @chiarazampolli, @jackal1-66, could you please check this PR? |
|
The CI was failing becuse the fake run 310000 used for the fullCI had no CTP/Config/Scalers object while the added task requires it. |
|
Made dummy commit to trigger new O2DPG/O2fst/o2 CI |
chiarazampolli
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, and tests passed (not sure if this is tested, though).
o2-analysis-time-dependent-qa to the suite of AnalysisQC tasks
Regards, Catalin.