Skip to content

WIP Input Maker Updates, V1 [ATR-17404]

Tim Martin requested to merge tamartin/athena:InputMakerV1 into master

This is a not-complete, but working, update to the InputMaker logic

  • IM now always merges, but can merge based on either ROI or Feature (latter to support muon SA case where all FS muons have the same FS-ROI)
  • IM modified to support more-than-one ROI per Decision object (== physics feature) for the BPhysics case. This was to be able to pass BPhysics Decision objects through the latter Muon sequences, but from the chat it looks better to do a combined hypo at Step 2 for Bphysics, and only form the Bphysics decision objects in a later step, after all muon is done.

I will now remove "this more-than-one ROI per Decision object", which will simplify the logic 👍

I leave this WIP MR as a benchmark, if we revise our plans again in the future

Ping @fpastore, @sshaw, @lyubushk

Merge request reports