Skip to content

Tidy up of TrigMuonHypoMT

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

Hello @markowen, @nakahama, @sshaw

Please could you review and comment / OK these changes.

  • Propagate event context to linkToParent (though I see Scott just beat me to it here)
  • Changed string literals in the muon hypos to use functions - but the underlying strings remain the same.
  • Added common debug printing
  • Removed the "view" link as it was not needed
  • Tried to remove the "RoI" link as it was just re-attaching the existing "initialRoI" link with a new name - however this caused a problem in EventViewCreatorAlgorithmWithMuons.cxx as it had been configured to search for a ROI link with the name "RoI"

For this last one - is it expected in the future that the TrigROIDescriptor will be updated in these affected places? If so, we can leave as-is, if not - then we don't need to duplicate this link and the EventViewCreator config should be updated to look for the "initialRoI" link.

Thanks - Tim.

Merge request reports