CPAlgorithms: update JES/JER to 2023 prerecs
Summer 2023 pre-recommendations following JetUncertaintiesRel22
Merge request reports
Activity
CI Result SUCCESS (hash 879c2cea)Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 2728]added review-approved label and removed review-pending-level-1 label
- Resolved by Baptiste Ravina
Hi @ravinab,
Overall this looks fine, so I'm going to set it to analysis-review-approved.
However two specific considerations:
- should we also update the legacy configuration, so that it doesn't break the unit test?
- should there be an option to get the old configuration, or is there essentially no value to that given that we have the new one?
Cheers, Nils
added Recommendation review-user-action-required labels and removed review-approved label
added analysis-review-approved label and removed analysis-review-required label
This merge request affects 1 package:
- PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms
This merge request affects 2 files:
- PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms/python/JetAnalysisConfig.py
- PhysicsAnalysis/Algorithms/JetAnalysisAlgorithms/python/JetAnalysisSequence.py
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash c9d4e732)Athena AnalysisBase AthAnalysis externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 2742]added review-approved label and removed review-pending-level-1 label
mentioned in commit 7b50e5ec