Use MuonGeometryFlags for CSC/STGC/MM everywhere
Hi,
this MR adds the HasSTGC
and HasMM
options to MuonGeometryFlags
to retrieve already during configuration whether a geometry contains STGCs or MicroMegas. Analogously to HasCSC
it is now used throughout the configuration. This MR is a manual sweep of !27761 (merged) to 21.3
.
Best, Nico
Merge request reports
Activity
added sweep:ignore label
This merge request affects 11 packages:
- DetectorDescription/DetDescrCnvSvc
- DetectorDescription/GeoModel/AtlasGeoModel
- DetectorDescription/RegionSelector
- Event/EventOverlay/EventOverlayJobTransforms
- MuonSpectrometer/MuonDetDescr/MuonRegionSelector
- MuonSpectrometer/MuonReconstruction/MuonRecExample
- Simulation/G4Atlas/G4AtlasApps
- Simulation/G4Atlas/G4AtlasTests
- Simulation/G4Atlas/G4AtlasTools
- Simulation/ISF/ISF_Example
- Tools/FullChainTransforms
Adding @sutt ,@jchapman ,@rosati ,@wleight ,@tkharlam ,@ahaas ,@vpascuzz ,@tadej ,@nkoehler ,@rbianchi as watchers
added 21.3 EDM Geometry MuonSpectrometer Overlay Simulation Tools Trigger review-pending-level-1 labels
CI Result SUCCESS (hash d35f8db9)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 41350]added review-user-action-required label and removed review-pending-level-1 label
Waiting until !27843 (merged) is merged, then the SimulationTier0Test will run fine
This merge request affects 11 packages:
- DetectorDescription/DetDescrCnvSvc
- DetectorDescription/GeoModel/AtlasGeoModel
- DetectorDescription/RegionSelector
- Event/EventOverlay/EventOverlayJobTransforms
- MuonSpectrometer/MuonDetDescr/MuonRegionSelector
- MuonSpectrometer/MuonReconstruction/MuonRecExample
- Simulation/G4Atlas/G4AtlasApps
- Simulation/G4Atlas/G4AtlasTests
- Simulation/G4Atlas/G4AtlasTools
- Simulation/ISF/ISF_Example
- Tools/FullChainTransforms
Adding @sutt ,@jchapman ,@rosati ,@wleight ,@tkharlam ,@goetz ,@ahaas ,@vpascuzz ,@tadej ,@nkoehler ,@rbianchi as watchers
added review-pending-level-1 label and removed review-user-action-required label
CI Result SUCCESS (hash d35f8db9)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 41416]Forgot that this is not for master, thus we have to wait until !27843 (merged) is successfully swept into 21.3 and restart CI again
Edited by Nicolas KoehlerNeed to wait until !27938 (merged) is merged to restart CI because of failing SimulationTier0Test
This merge request affects 11 packages:
- DetectorDescription/DetDescrCnvSvc
- DetectorDescription/GeoModel/AtlasGeoModel
- DetectorDescription/RegionSelector
- Event/EventOverlay/EventOverlayJobTransforms
- MuonSpectrometer/MuonDetDescr/MuonRegionSelector
- MuonSpectrometer/MuonReconstruction/MuonRecExample
- Simulation/G4Atlas/G4AtlasApps
- Simulation/G4Atlas/G4AtlasTests
- Simulation/G4Atlas/G4AtlasTools
- Simulation/ISF/ISF_Example
- Tools/FullChainTransforms
Adding @sutt ,@jchapman ,@rosati ,@wleight ,@tkharlam ,@goetz ,@ahaas ,@vpascuzz ,@tadej ,@nkoehler ,@rbianchi as watchers
CI Result SUCCESS (hash d35f8db9)Athena externals cmake make required tests optional tests Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST 41440]added review-approved label and removed review-pending-level-1 label
mentioned in commit 984d67c3
mentioned in merge request !28069 (merged)
mentioned in merge request !28706 (merged)
mentioned in commit aca03460