WIP: Cleaning up PseudoJetGetter
Goals of this MR are to:
- Put explicit
EventContext
in theIPseudoJetGetter
interface to avoid needing to do the thread-local lookup inPseudoJetAlgorithm
. - Optionally, phase out
PseudoJetGetter
as a class entirely so we only have to configure and schedulePseudoJetAlgorithm
.
Currently this is blocked by TriggerPseudoJetGetter(2)
, because this is in use by the legacy master trigger, and this is not possible to migrate to DataHandles + EventContext
. It might actually be easier to do this if we follow step 2, because at that point we won't have any other classes that inherit from IPseudoJetGetter.
@delsart, @wbalunas, @loch, @peter any opinions?
Also I had to make some changes to JetIsolationTool
, which AFAIK is not used, but I think the changes (just take PseudoJetContainer from SG instead of working via a getter) probably maintain the functionality.