Skip to content

[ATR-22328][ATR-22641] Make HLT runtime validation optional. Tidy up StreamerHypoAlg

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

HLT runtime validation catches malformated output from a slice's HypoAlg at the source.

However, as it fully back-validates the nav graph from every Decision Object at the end of every Step, its CPU draw has grown somewhat in the larger menus we are producing.

This MR restricts the validation to run only in the core CI-ART tests.

As the runtime validation now cannot be avoided (any flag the user sets to try and turn it off won't stick), this showed up a minor violation in the streaming HypoAlg instance. This generic HypoAlg has been modernised a bit to allow multiple Decision Objects. Also bringing in here some new static string IDs which will be picked up on more in following MRs.

As part of this modernisation, the SetInitialRoIAsFeature flag is removed - minor edits are made to JET and MET. This streamer HypoAlg now always sets a dummy "feature", a link-to-self (as practices elsewhere where a "feature" link is required by framework design, but no suitable "feature" exists to be linked).

Merge request reports