Fix MuonCaloTagAlg tool config

Trying to fix ATLASRECTS-7384. Not sure why it did not work before, but it was anyways a bit messy. Maybe the public tool needs to be added before adding to the fragment?

Note that ToolHandleArray still behaves strangely. It looks private but it requires a public tool. @wlampl, @fwinkl, @leggett, @jchapman any idea?

Do we ever even need an array here?

Tagging also @jojungge, @emoyse, @tsulaia

added NewConfig bugfix urgent labels

This merge request affects 1 package:

• Reconstruction/MuonIdentification/MuonCombinedConfig

This merge request affects 1 file:

• Reconstruction/MuonIdentification/MuonCombinedConfig/python/MuonCombinedReconstructionConfig.py

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

added Reconstruction master review-pending-level-1 labels

CI Result SUCCESS (hash 222bbf24)

	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 62546]

Please note that ATLASRECTS-7384 applies also to old-style config so this cannot fix it (unless it’s related to wrapped muon config pulled in by something else, but I don’t think so because I tested a similar fix locally and I still saw the error occasionally). Anyway, no objections to this MR of course - it removes an ‘Ugh’ which is always good. ;-)

removed review-pending-level-1 label

added review-approved label

removed review-approved label

added review-user-action-required label

removed review-user-action-required label

added review-approved label

Lets put this in

But I would imagine there might be something else lurking behind the array/public tool etc here ...

mentioned in commit c4eca5cc

merged

added sweep:ignore label