update muonLRT RoI size

Increase the size of the RoI used for FastTrackFinder_MuonLRT tracking to phi x eta=0.4 x 0.2 in order to be efficient for large d0 tracks. The default muon RoI is not wide enough in phi. The change only affects muon LRT chains. This MR has been tested with runTrigArt.py

This merge request affects 1 package:

• Trigger/TriggerCommon/TriggerMenuMT

This merge request affects 1 file:

• Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Muon/MuonSequenceSetup.py

Adding @sutt ,@hrussell ,@dzanzi ,@bernius as watchers

added Trigger TriggerMenu master review-pending-level-1 labels

Hi, might it make sense to instead wait a while, and then change the width in the ConfigSettings, after this have been merged ...

!42085 (merged)

Then we fix the Roi sizes in the Configuration, and extract the size from the configuration and set the width. This way we avoid having Roi Width information spread about all over the place.

If you need this urgently, then indeed do not wait for 42085, but go ahead here, and we can make the configuration based change in a separate merge request once !42085 (merged) has been merged.

Cheers Mark

Hi @sutt , This MR is updating the RoI position in the same way as done for the muon Iso RoI i.e. using a method that I see is unchanged in your MR, see https://gitlab.cern.ch/sutt/athena/-/blob/bull/Trigger/TriggerCommon/TriggerMenuMT/python/HLTMenuConfig/Muon/MuonSequenceSetup.py#L664

so I think it must be OK and will not conflict with your MR.

Cheers John

Hi John, that's fine, this won't conflict with my changes, by all means go ahead. We can just make the change to use the new configuration later.

Cheers Mark

CI Result SUCCESS (hash 9cd04bff)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31052]

added review-approved label and removed review-pending-level-1 label

CI successful, Approving... Lino (L1)

mentioned in commit f7cfa813

merged

added sweep:ignore label