Skip to content
Snippets Groups Projects

Toggles for running specific object types

Merged Teng Jian Khoo requested to merge do-object-flags into master

Allows independently toggling the CP algs for the following object types:

  • small-R jets
  • large-R topo jets
  • large-R UFO jets (default off)
  • VR jets
  • muons
  • electrons (default off)
  • photons (default off) These are on by default except as specified above.

In PHYSLITE jobs, it is enforced that VR jets and large-R topo jets are off, because these collections are not in the DAOD format.

It is further enforced that only one large-R type is permitted, as overlap removal can only handle one. If it is useful, this check could be made conditional on OR.

Activating write_X flags with do_X=False is an error condition. I kept this as an assert and not an auto-disable, because all this is configured from the yaml, and therefore people should keep it consistent.

Overlap removal configuration will be adapted automatically to the active collections.

TODO:

  • Add asserts for the other operations (resolved analysis, boosted analysis, truth matching) that depend on certain object types being present.
  • Consider if we should require certain types e.g. small-R jets given how the framework is jet-centric
  • Consider if we need a toggle for b-tagging
Edited by Teng Jian Khoo

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Dan Guest
    • Resolved by Teng Jian Khoo

      At some point I wish we'd split our configuration into a part that is meant to be overridden on the command line and a part that is allowed to be a higherarchy. This list is a prime example of where we should just have

      ntuple_contents:
      - large_r_jets
      - etc
      command_line_options:
        option1: blah
        option2: blah
  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 700d4e97 - Disable collections automatically on PHYSLITE

    Compare with previous version

  • Teng Jian Khoo changed the description

    changed the description

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 6361c1ae - Disable collections automatically on PHYSLITE

    Compare with previous version

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 3f75a4af - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo resolved all threads

    resolved all threads

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 108f2b03 - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • b6a86c78 - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 63ac5c5a - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo added 6 commits

    added 6 commits

    • 63ac5c5a...14ed086c - 2 commits from branch master
    • 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

    Compare with previous version

  • Teng Jian Khoo mentioned in merge request !173 (merged)

    mentioned in merge request !173 (merged)

  • Teng Jian Khoo added 10 commits

    added 10 commits

    • 5b6d5d27...b4c3f604 - 6 commits from branch master
    • f61d92cb - Toggles for running specific object types
    • 99c1ed16 - Save some characters on container names
    • e03352f4 - Disable collections automatically on PHYSLITE
    • c9550b0c - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo added 1 commit

    added 1 commit

    • 1ed0a0d5 - Move flag validation to new module

    Compare with previous version

  • Teng Jian Khoo marked the checklist item Consider if we should require certain types e.g. small-R jets given how the framework is jet-centric as completed

    marked the checklist item Consider if we should require certain types e.g. small-R jets given how the framework is jet-centric as completed

  • Teng Jian Khoo marked the checklist item Consider if we need a toggle for b-tagging as completed

    marked the checklist item Consider if we need a toggle for b-tagging as completed

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading