Expand/add the type provided from MeasurementBase to avoid dynamic casts
Expand/add the type provided from MeasuementBase to also cover the particular RIO on track cases, combinded they should allow to get rid of a few more dyn casts.
Start from TRT_ElectronPID
tool to check it
Merge request reports
Activity
This merge request affects 5 packages:
- InnerDetector/InDetRecEvent/InDetRIO_OnTrack
- InnerDetector/InDetRecTools/TRT_ElectronPidTools
- MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonRIO_OnTrack
- Simulation/ISF/ISF_Fatras/ISF_FatrasEvent
- Tracking/TrkEvent/TrkRIO_OnTrack
Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@sroe ,@rosati ,@jchapman ,@calfayan ,@vpascuzz ,@nkoehler as watchers
CI Result FAILURE (hash 5dd865ce)Athena AthSimulation AnalysisBase AthGeneration 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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 14660]This merge request affects 5 packages:
- InnerDetector/InDetRecEvent/InDetRIO_OnTrack
- InnerDetector/InDetRecTools/TRT_ElectronPidTools
- MuonSpectrometer/MuonReconstruction/MuonRecEvent/MuonRIO_OnTrack
- Simulation/ISF/ISF_Fatras/ISF_FatrasEvent
- Tracking/TrkEvent/TrkRIO_OnTrack
Adding @goetz ,@cgrefe ,@amorley ,@wleight ,@sroe ,@rosati ,@jchapman ,@calfayan ,@vpascuzz ,@nkoehler as watchers
CI Result SUCCESS (hash 6100b9c1)Athena AthSimulation AnalysisBase AthGeneration 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
AnalysisBase: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 14685]added review-approved label and removed review-pending-level-1 label
mentioned in commit 8e3a408e
added sweep:ignore label