Skip to content

Teach MuonAnalysisSequence all available isolation WPs

Martin Errenst requested to merge merrenst/athena:21.2_CP_Muon_Check_Iso into 21.2

This is a small fix to a bug that @pagessin raised yesterday. Apparently the current Iso option for muons does not configure a meaningful isolation WP.

This is solved by checking for all available MuonWPs in the IsolationSelectionTool and requiring the user to name it explicitly. (e.g. FCLoose) The MuonWP are defined here: https://gitlab.cern.ch/atlas/athena/-/blob/21.2/PhysicsAnalysis/AnalysisCommon/IsolationSelection/Root/IsolationSelectionTool.cxx#L175 NonIso is still an option to skip isolation all together.

I am torn if it isn't more elegant to omit the isoWP check on the python-side completely. The IsolationSelectionTool checks for the WP by name anyway, so we duplicate that mechanism here.

I've added the alsoTargeting:master label -- the MuonAnalysisSequence should stay the same in both branches.

A couple of references to the MuonAnalysisSequence in the tutorials at atlassoftwaredocs will become outdated with this. There is a listing of the corresponding python code and the former Iso option might be used in a couple examples.

Merge request reports