Fix to RPC hit ordering

Apparently ordering by r only is insufficient to ensure that RPC hits with different doublet phi values are separated. The MuGirl hit ordering now explicity checks the RPC dbPhi before falling back to the r value. This should remove duplicate hits warnings such as the one reported in ATLASRECTS-6369.

Closes ATLASRECTS-6369

This merge request affects 1 package:

• Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools

This merge request affects 1 file:

• Reconstruction/MuonIdentification/MuonCombinedTrackFindingTools/src/MuonCandidateTrackBuilderTool.h

Adding @goetz ,@rosati ,@wleight ,@jojungge as watchers

added Reconstruction master review-pending-level-1 labels

CI Result SUCCESS (hash 4352667e)

	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 34959]

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

merged

mentioned in commit 7f32d74b

added sweep:ignore label