JetCalibTool scripts to produce paper figures
News:
- New script to plot eta correction as a function absolute detector eta
- Small modification to script to plot jet energy response as a function of detector eta. It is possible now to only fill the distribution when the jet is above 20 GeV (if requested).
Those scripts can be used to produce the inputs to prepare the paper figures.
Merge request reports
Activity
added Reconstruction label
added JetEtmiss label
added 21.2 label
added review-pending-level-1 label
CI Result SUCCESSAnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-20284-2019-01-10-14-59
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 32109] (for remote access see the FAQ for MR reviewers) CI Result SUCCESSAnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-20284-2019-01-10-15-30
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 32108] (for remote access see the FAQ for MR reviewers)- Resolved by Nils Erik Krumnack
- Resolved by Nils Erik Krumnack
added review-pending-level-2 label and removed review-pending-level-1 label
@jbossios, can you please implement the suggested changes for memory (using
std::cout
is OK), and address Nils' comment?Karolos (L2)
added review-user-action-required label
Hi @jbossios, could you please add ATLAS Robot as developer to your new fork (https://atlassoftwaredocs.web.cern.ch/gittutorial/gitlab-fork/#add-your-friendly-build-bot).
Thanks, Tadej (L1)
Hi @jbossios,
Once this is updated to use smart pointers this can be merged in, but you should check with the relevant expert if he can update
AccessorWrapper
when he has a slow afternoon. There is really no point in copy-pasting this utility over a number of scripts. However, I don't want to hold up this MR for that, particularly since we already have a number of other scripts using it.Cheers, Nils