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 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.

marked this merge request as draft

changed target branch from master to do-object-flags

added 5 commits

• 4df2b7dd - Toggles for running specific object types
• fdbd5b02 - Save some characters on container names
• 30abee36 - Disable collections automatically on PHYSLITE
• 5b6d5d27 - Move flag validation to new module
• 6343a643 - Refactor container names heavily, make more flags

Compare with previous version

changed the description

This is going to be a merge conflict bonanza if we don't get it in soon...

added 1 commit

• 49f5fd3a - Flaking

Compare with previous version

added 1 commit

• e150fd44 - Disable E241

Compare with previous version

resolved all threads

resolved all threads

resolved all threads

added 1 commit

• 75e33cef - Fix f-string

Compare with previous version

mentioned in merge request !175 (merged)

resolved all threads

added 14 commits

• 75e33cef...c9550b0c - 10 commits from branch do-object-flags
• a612c703 - Refactor container names heavily, make more flags
• bbe34c1e - Flaking
• 6038bc66 - Fix f-string
• 673be973 - E241 in linting CI yaml

Compare with previous version

added 5 commits

• 1ed0a0d5 - 1 commit from branch do-object-flags
• 69d17700 - Refactor container names heavily, make more flags
• cb2f6c83 - Flaking
• cbceedb0 - Fix f-string
• 01161d33 - E241 in linting CI yaml

Compare with previous version

added 6 commits

• 01161d33...f066466d - 2 commits from branch do-object-flags
• e2bc04b9 - Refactor container names heavily, make more flags
• f71972e9 - Flaking
• 5a6ef612 - Fix f-string
• e6921d0f - E241 in linting CI yaml

Compare with previous version

added 5 commits

• d3153d6b - Refactor container names heavily, make more flags
• ed3406fb - Flaking
• 3c6d7408 - Fix f-string
• 947ed76f - E241 in linting CI yaml
• 112875b2 - Missed conflict

Compare with previous version