Muon Detector Manager speed up

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

added MuonSpectrometer master review-pending-level-1 labels

CI Result FAILURE (hash 859e1037)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

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

added 1 commit

• acef4277 - Fix the typo

Compare with previous version

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

Hi @amete,

since I am on holidays this week I asked my old friend and favorite coder @jojungge to make the rpcStationTypeIdx method faster. You can find his changes here. It will most likely fix your problem since no string comparisons are done anymore during execute.

Best, Nico

CI Result FAILURE (hash acef4277)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 13, warnings 10
AthSimulation: number of compilation errors 6, warnings 0
AthGeneration: number of compilation errors 17, warnings 1
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20172]

marked as a Work In Progress

removed review-pending-level-1 label

added 1 commit

• 72b57cd1 - Fix the typos - part II

Compare with previous version

mentioned in merge request !36312 (merged)

unmarked as a Work In Progress

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

added review-pending-level-1 label

CI Result FAILURE (hash 72b57cd1)

	Athena	AthSimulation	AthGeneration	AnalysisBase
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 13, warnings 3
AthSimulation: number of compilation errors 6, warnings 0
AthGeneration: number of compilation errors 17, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20183]

added 1 commit

• 09549871 - No inline for the rpc helper

Compare with previous version

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

CI Result SUCCESS (hash 09549871)

	Athena	AthSimulation	AthGeneration	AnalysisBase
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
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 20192]

• Large number of whitespace changes
• All tests passing
• Actual changes are to replace string calcs with simple integer-indexd maps.

Approving - L1 Shifter.

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