Bug fixes for BJT in R22
This MR
- implements the Ntrk variable needed for the ANN tagger (centrally calculated now in JSSTaggerBase rather than calculating it in various places) [feature which was not migrated from R21 to R22 beforehand]
- fixes some of the substructure variables used in the top DNN tagger (values stored in the DAOD were overwritten with default values)
- updates the testing scripts to run in R22 with the latest config files
Merge request reports
Activity
This merge request affects 1 package:
- Reconstruction/Jet/BoostedJetTaggers
This merge request affects 8 files:
- Reconstruction/Jet/BoostedJetTaggers/BoostedJetTaggers/JSSTaggerBase.h
- Reconstruction/Jet/BoostedJetTaggers/Root/JSSTaggerBase.cxx
- Reconstruction/Jet/BoostedJetTaggers/Root/JSSWTopTaggerANN.cxx
- Reconstruction/Jet/BoostedJetTaggers/Root/JSSWTopTaggerDNN.cxx
- Reconstruction/Jet/BoostedJetTaggers/Root/SmoothedWZTagger.cxx
- Reconstruction/Jet/BoostedJetTaggers/util/test_JSSWTopTaggerANN.cxx
- Reconstruction/Jet/BoostedJetTaggers/util/test_JSSWTopTaggerDNN.cxx
- Reconstruction/Jet/BoostedJetTaggers/util/test_SmoothedWZTagger.cxx
Adding @goetz ,@cdelitzs ,@jveatch ,@meehan ,@mleblanc ,@rnewhous ,@sschramm as watchers
CI Result SUCCESS (hash 739d4d78)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-CC7 63154]added review-approved label and removed review-pending-level-1 label
added review-approved-point1 label
added analysis-review-approved label and removed analysis-review-required label
- Resolved by Tadej Novak
added review-user-action-required label
removed review-user-action-required label
mentioned in commit 777259f1
added sweep:ignore label
mentioned in merge request !59115 (merged)