SUSYTools: extra protections for compressed EWK searches
Consolidating various hacks and additions that we've made to SUSYTools for the compressed EWK analysis, for ease of eventual analysis preservation and combinations. Two extra protections are added to account for crashes that came up (justifications in line). We also have four updated cross section files that I've been adding by hand into our local SUSYTools, but unsure if these have been added to PMG and/or elsewhere and this is actually redundant.
Tagging @oabouzei, @ykeisuke, @jmiguens, @szambito, as well as @miholzbo and @jshahini who might know more about the cross section situation.
Thanks!
Merge request reports
Activity
- Resolved by Julia Lynne Gonski
- Resolved by Julia Lynne Gonski
thanks @jgonski !
We don't want to maintain xsec files anymore, feel free to open CENTRPAGE JIRA for xsecs. For other two lines, can you test them locally? If it works fine for you and no need to update the reference (which we can see in this MR), I would be happy to proceed with this MR.
Keisuke
Hi @ykeisuke, thanks for the feedback! We'll follow the xsecs up elsewhere in the JIRA then. As for the other changes, is there any particular output you're looking for in terms of local tests? We've been running our productions with these changes for several months, and they don't have any effect except to avoid the crashes mentioned. As long as the logic of checking for vertices/caloClusters makes sense to you, I think we can call these safe. I'm removing the xsec files and will resolve the WIP status once you confirm.
Thanks again! Julia
added 21.2 Analysis SUSYTools review-pending-level-1 labels
added 60 commits
-
328549e7...60dc8957 - 58 commits from branch
atlas:21.2
- 45fe2023 - Removing xsec files from MR
- 9ea15c25 - Merge remote-tracking branch 'upstream/21.2' into dev-compressed-ewk
-
328549e7...60dc8957 - 58 commits from branch
CI Result FAILUREAnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-23698-2019-05-28-03-12
AnalysisBase: number of compilation errors 0, warnings 1
AnalysisTop: number of compilation errors 0, warnings 1
AthAnalysis: number of compilation errors 0, warnings 3
AthDerivation: number of compilation errors 0, warnings 11
For experts only: Jenkins output [CI-MERGE-REQUEST 39051]added review-user-action-required label and removed review-pending-level-1 label
It seems like the other MR (!23590 (merged)) has to get going before this MR.
Hi,
While we wait for (!23590 (merged)), please, remember to close the discussions still open so that we can proceed with the merge faster.
Cheers, Miguel
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESSAnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-23698-2019-05-28-21-01
AnalysisBase: number of compilation errors 0, warnings 0
AnalysisTop: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
AthDerivation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 39113]mentioned in commit 15ca986d
added sweep:ignore label