Skip to content

[ATR-19236] Trigger navigation naming convention & EDM definition

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

Naming convention updates to the collections produced by the L1Decoder. Improvements to the navigation output.

  • The initial navigation collections produced by the L1Decoder are prefixed "HLTNav_" (like all other navigation collections)
  • The initial ROI and initial Rec ROI collections produced by the L1Decoder are prefixed "HLT_" (like all other persistifiable collections produced during HLT execution)
  • The white listing of ROI containers in the recordable() checking function is removed.
  • The white listing of "HLTNav_" containers in the recordable() checking function is changed to an ERROR.
  • All "HLTNav_" containers are added to the EDM (online) via TriggerConfig.py so they should never be specified by hand in TriggerEDMRun3.py
  • Added some missing navigation containers to the collection script (L1Decoder fullscan output and HLT summary maker output)
  • When exporting "HLTNav_", dynamic variables were removed. The output of HypoAlgs requires dynamic variables to re-map element links.
  • Both decObj and decObjFilterOut are now passed to triggerOutputCfg such that the sub-set in decObjFilterOut get the correct aux dyn added.

cc @tbold, @fpastore, @rbielski for changes to L1Decoder, TriggerConfig, HLTCFConfig

cc @okuprash for changes to TriggerConfig, TriggerEDMRun3

cc @damazio @joheinri for changes to TrigT2CaloCommon and CaloDef (changing EMRoIs to HLT_EMRoIs, FSRoI to HLT_FSRoI)

cc @sshaw @nakahama for changes to Muon*DecodeConfig, MuFastSteering, TrigL2MuonSAConfig_newJO, HLTMenuConfig/Muon/MuonSetup.py (changing MURoIs to HLT_MURoIs, RecMURoIs to HLT_RecMURoIs)

Edited by Tim Martin

Merge request reports