MuonGeoModelR4 - Add Wire & RadialStripDesign

These auxillary classes are needed to proceed with the readout geometry implementation in !66043 (merged) & !65801 (merged)

Tagging: @pscholer, @ivyas

added 1 commit

• c0f7d02f - Add strip direction

Compare with previous version

This merge request affects 1 package:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4

This merge request affects 6 files:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/RadialStripDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/StripDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/StripDesign.icc
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/WireGroupDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/src/RadialStripDesign.cxx
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/src/WireGroupDesign.cxx

Adding @rosati ,@stavrop ,@jojungge ,@apsallid ,@pscholer as watchers

added MuonSpectrometer main review-pending-level-1 labels

This merge request affects 1 package:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4

This merge request affects 6 files:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/RadialStripDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/StripDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/StripDesign.icc
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/MuonReadoutGeometryR4/WireGroupDesign.h
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/src/RadialStripDesign.cxx
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonReadoutGeometryR4/src/WireGroupDesign.cxx

Adding @rosati ,@stavrop ,@jojungge ,@apsallid ,@pscholer as watchers

CI Result SUCCESS (hash 8eab3af0)

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

The system selected 51 tests to probe the Athena changeset (out of 67 available tests). Link to tests selection rules

CI Result SUCCESS (hash c0f7d02f)

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

Changes look fine to me and the CI is good. Sending on to L2 for a second look due to the size. Clara (L1)

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

I do not see any problems as well, approving....

Pavol [as L2 MR shifter]

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

mentioned in commit 99bdd9c9

merged

added sweep:ignore label