ActsGeometry - Introduce NoDeletePtr & DetectorAlignStore & misc
- Introduce
NoDeletPtr
to be used in theDetectorVolumeSvc
for the interfacing of theAtlasTools
with the Acts::IDetectorBuilders - Introduce
DetectorAlignStore
which will superseed theRawGeomStore
in !70106 (merged) - Explicitly schedule the ActsGeometrySvc for the
TruthParticleHitCountAlg
Merge request reports
Activity
added ACTS Tracking main review-pending-level-1 labels
This merge request affects 5 packages:
- MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4
- Tracking/Acts/ActsConfig
- Tracking/Acts/ActsGeoUtils
- Tracking/Acts/ActsGeometry
- Tracking/Acts/ActsGeometryInterfaces
Affected files list will not be printed in this case
Adding @stavrop ,@tbold ,@cvarni ,@pscholer ,@jojungge ,@rosati ,@pagessin ,@apsallid ,@toyamaza ,@kluit as watchers
added MuonSpectrometer Upgrade labels
CI Result FAILURE (hash 487c5cbc)Athena AthSimulation externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 2, warnings 6
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 7274] (remote access info) CI Result FAILURE (hash 6a337c70)Athena AthSimulation externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 2, warnings 2
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 7275] (remote access info) CI Result SUCCESS (hash 6f4c1375)Athena AthSimulation externals cmake make tests Full details available on this CI monitor view. Check the JIRA CI status board for known problems
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-EL9 7276] (remote access info)removed review-pending-level-1 label
added review-approved label
Hi @jojungge, is this urgent? Shall I merge it right away or better to wait for a day or two to give other ACTS developers a chance to have a look at these changes and comment? I assume most (all?) of them are away these days for the long weekend. Thanks.
Actually, it's breaking-out developments from !70106 (merged) and not used in production.
mentioned in commit 8076006c
mentioned in merge request !70106 (merged)