ITk config flags cleaning
This MR provides a cleaning of the ITk config flags. It is built on top of !49070 (closed) from @tadej .
In details, the following changes are implemented:
- removal of configs inherited from
InDetconfigs but never actually used (ITkGlobalChi2FitterLowPtCfg,ITkTrackFitterLowPtCfg,ITkGlobalChi2FitterBTCfg,ITkTrackFitterBTCfg,ITkNNScoringToolBaseCfg,ITkNNScoringToolCfg,ITkSiZvertexMaker_xkCfg,ITkSimpleAmbiguityProcessorToolCfg) - removal of flags not used or never validated in previous ITk studies beyond their default configurations (e.g.
doMinBias,doBeamHalo,useBroadClusterErrors,doRefitInvalidCov) - move of the tracking cuts flags into dedicated
ITk.Tracking.Cutssubcategory - split of the flags between detector-related flags with simple
ITkand ITk-tracking related flags underITk.Trackingsubcategory - minor updates in tracking geometry and primary vertex flags to make them more self-contained
This is a first step of a set of MRs, aiming at decoupling InDetConfigFlags and ITkConfigFlags from future TrackingConfigFlags and configs of common tracking tools used beyond InDet/ITk. Based on the review of this MR, a unified strategy is planned to be applied to the InDet configs.
FYI @nstyles @goetz @tadej @npetters @vcairo @gfacini @pagessin @lgagnon
Edited by Thomas Strebler