Skip to content

Add active state info to link info

cc @jodafons

This MR is to help end users who request features with TrigDefs::includeFailedDecisions. As this call returns objects which both passed and failed the HypoTool of a given chain / set of chains, the user is not immediately aware which of these cases was true.

Here an ENUM is added to the returned LinkInfo which can be used to communicate this info on a feature-by-feature basis.

This replaces the Run2 use of myTriggerElement->getActiveState()

@tbold might have opinions on this.

NOTE: In the updated reference, we are still affected by the TDT not knowing the collection which produces the final accept on a chain-by-chain basis. While it only produces incorrect results in very niche cases, we should still be thinking about solving this.

Edited by Tim Martin

Merge request reports