ATLASRECTS-5071: Remove flavor tagging edm schema
This is a lot of edits but they are entirely to remove dead code.
While retraining the flavor tagging algorithms for pflow jets, we realized that some of the inputs to our neural networks were saved as doubles in derivations but would be better saved as floats. In addition the ad-hoc names we'd used in derivations didn't conform very well to the style we've used elsewhere in the EDM.
Unfortunately we were already working with derivations that used the less-desirable "old" convention. To accommodate a transition to the new convetion, we'd used an EDMSchema
enum that allowed the algorithms to to work in either case.
Now that the major retraining effort is over we should switch everything over to the new naming convention. We can also remove the EDMSchema
enum. This simplifies the code quite a bit.
Merge request reports
Activity
added 11 commits
-
3a98cbba...ec47c180 - 10 commits from branch
atlas:21.2
- 9771b3b5 - Merge remote-tracking branch 'atlas/21.2' into remove-edm-schema
-
3a98cbba...ec47c180 - 10 commits from branch
added 21.2 Analysis BTagging Derivation JetEtmiss review-pending-level-1 labels
added 1 commit
- 7c2595aa - Remove track augmenter tool (was a duplicate anyway)
CI Result SUCCESS AnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-26722-2019-09-22-19-19
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 40742] CI Result SUCCESS AnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-26722-2019-09-22-20-38
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 40744] CI Result SUCCESS AnalysisBase AnalysisTop AthAnalysis AthDerivation externals cmake make required tests optional tests Full details available at NICOS MR-26722-2019-09-22-23-13
AnalysisBase: number of compilation errors 0, warnings 0
AnalysisTop: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 2
AthDerivation: number of compilation errors 0, warnings 11
For experts only: Jenkins output [CI-MERGE-REQUEST 40743] added review-approved label and removed review-pending-level-1 label
mentioned in commit e3dc310a