Sweeping !29858 from 21.3 to master. Inserting BLines for NSW

Inserting BLines for NSW

See merge request !29858 (merged)

added sweep:from 21.3 label

Adding original author @ckitsaki as watcher.

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

added MuonSpectrometer master review-pending-level-1 labels

CI Result FAILURE (hash b6f198a1)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 61
AthSimulation: number of compilation errors 1, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9089]

there are many compilation errors. Addressing the original author (@ckitsaki) to have a look and fix the errors or don't sweep into master.

Kira (L1)

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

added 1 commit

• d8392ca9 - fix print statements

Compare with previous version

added 1 commit

• 1a71c20b - a minor change

Compare with previous version

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

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

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

CI Result FAILURE (hash d8392ca9)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 61
AthSimulation: number of compilation errors 1, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9144]

CI Result FAILURE (hash 1a71c20b)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 1, warnings 60
AthSimulation: number of compilation errors 1, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 9145]

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

There are still compilation errors.

Tadej (L2)

Hi @ckitsaki,

you don't need the new MsgStream in master (that's the compilation error) since you have m_log as a member already available.

Best, Nico

added 1 commit

• cde2fcfc - remove msgstream

Compare with previous version

This merge request affects 1 package:

• MuonSpectrometer/MuonDetDescr/MuonReadoutGeometry

Adding @rosati ,@nkoehler ,@wleight as watchers

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

CI Result SUCCESS (hash cde2fcfc)

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

the orginal code checked and approved in MR29858, I checked the updates which are only tiny changes, CI passed, approving

Roman (L1)

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

merged

mentioned in commit 0eeb70bd

added sweep:ignore label