Skip to content

Streamline cpalgs

Teng Jian Khoo requested to merge streamline-cpalgs into master

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 an Input 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 AODTruthParticles`
  • @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 an SH4b-PHYSLITE yaml.
Edited by Teng Jian Khoo

Merge request reports