MuonR4- MuonGeoModelR4 - pick up the Rpc gasGaps from their Identifiers

Pick-up the rpc gasGap order from the volume Identifiers and not from the local x position. Allows to reshuffle the positions of the gas volumes to bring BOG eta 0 in agreement with the legacy description.

added 1 commit

• 3d038e7f - Revert to bigger 0

Compare with previous version

added 930 commits

• 3d038e7f...6fb949e4 - 929 commits from branch atlas:main
• 06fff9f6 - Merge branch athena:main into FixRpcGaps

Compare with previous version

added 589 commits

• 06fff9f6...d9e03514 - 588 commits from branch atlas:main
• b5c62591 - Merge branch athena:main into FixRpcGaps

Compare with previous version

added full-unit-tests label

Jenkins please retry a build

Build area was cleaned as per request posted in the DB. The full software build will be performed

This merge request affects 2 packages:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelR4
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4

Affected files list will not be printed in this case

Adding @kluit ,@apsallid ,@jojungge ,@markowen ,@wleight ,@stavrop ,@pscholer ,@ddicroce ,@mvozak as watchers

added MuonSpectrometer Upgrade main labels

CI Result FAILURE (hash b5c62591)

	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 (remote access info)

marked this merge request as ready

changed title from Draft: MuonR4- GeoModel Fix BOG gasgap description to MuonR4- MuonGeoModelR4 - pick up the Rpc gasGaps from their Identifiers

changed the description

This merge request affects 2 packages:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelR4
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4

Affected files list will not be printed in this case

Adding @pscholer ,@ddicroce ,@wleight ,@markowen ,@apsallid ,@jojungge ,@stavrop ,@mvozak ,@kluit as watchers

added review-pending-level-1 label

CI Result FAILURE (hash b5c62591)

	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 (remote access info)

There is the follwowing error The volume RPC BIL eta -1 phi 7 dbR 1 dbZ 1 dbPhi 1 does not have any childern with Identifiers -L1

added review-user-action-required label and removed review-pending-level-1 label

added 87 commits

• b5c62591...09fc8e71 - 84 commits from branch atlas:main
• 79351b5f - Fix BOG gasgap description
• 7fe4da02 - Revert to bigger 0
• bba0b439 - Update geometry file

Compare with previous version

This merge request affects 2 packages:

• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelR4
• MuonSpectrometer/MuonPhaseII/MuonDetDescr/MuonGeoModelTestR4

Affected files list will not be printed in this case

Adding @ddicroce ,@jojungge ,@wleight ,@kluit ,@markowen ,@pscholer ,@stavrop ,@mvozak ,@apsallid as watchers

added review-pending-level-1 label and removed review-user-action-required label

CI Result SUCCESS (hash bba0b439)

	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 (remote access info)

Changes are fine and CI succeeds, approving for L1. -L1

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

merged

mentioned in commit be0315d2