Streamline cpalgs
Begins to address #56 (closed)
- Defines
ConfigFlags.Input.isPHYSLITE
so we don't need to pass this as an argument anywhere - Similarly, uses
ConfigFlags.Analysis.DataType
(could maybe make this anInput
arg instead) - Eliminates all useless local variable definitions in
container_names.py
- Passes
containers
as an argument into CP alg configs, so that extraction of relevant container names occurs inside the CA wrappers - Implements more sensible/explicit jet naming conventions:
-
reco10Jet
-->reco10TopoJet
-
truth10Jet
-->truth10TrimmedJet
-
truth10UFOJet
-->truth10SDJet
-
-
@viruelas Renamed
TruthParticles to
TruthDiHiggsParticlesto avoid in principle clash with AOD
TruthParticles` -
@dabattul @frenner I eliminated the UFO-specific CP alg configs in favour of passing
lr_jet_type = Topo|UFO
as an argument, but didn't clean this up for all the branch configuration yet - @dabattul @arelycg I don't think we ever need Tau4, so removed it from UFO jets... it was inside a weird check.
- I changed PHYSLITE options so that we no longer automatically disable various things. Instead there is an early check that incompatible containers are not requested.
RunConfig-PHYSLITE.yaml
now has those changes, and the CI uses this instead. If it is more sensible for SH4b to use a different set of options on those samples, then we can similarly define anSH4b-PHYSLITE
yaml.
Edited by Teng Jian Khoo