Fix ROOTService in Gaudi-Allen

The ROOTService is not used at present when calling Allen from Moore, using Gaudi-Allen algorithms. In fact, it crashes when enabling the monitoring in a line, likely since several instances of the ROOTService are created within the different instances of ProvideRuntimeOptions needed to provide different raw bank types.

This MR ensures there's only a single instance of the ROOTService and that it's filename can be configured to avoid clashes between tests.

Closes Moore#480 (closed)

changed the description

added 1 commit

• 36dbea1e - Fixed formatting

Compare with previous version

mentioned in issue Moore#480 (closed)

requested review from @dcampora and @raaij

added RTA label

I think extra checks may be needed in a few lines that use the ROOTService for monitoring purposes. It's switched off by default with a property, but should not crash when switched on. The RICH alignment lines use it.

mentioned in issue Moore#485 (closed)

marked this merge request as draft

mentioned in issue Moore#487 (closed)

added 71 commits

• 36dbea1e...83f7a80b - 67 commits from branch master
• ff6a09d2 - no usage of ROOT service when calling Allen from Moore
• a9904877 - adjust docs for ROOTService only being available in standalone Allen event loop
• d2e1b3c8 - Fixed formatting
• 78c48e69 - Use a service to hold and configure the Allen ROOTService

Compare with previous version

marked this merge request as ready

changed title from DRAFT: Disable ROOTService in Gaudi-Allen to Fix ROOTService in Gaudi-Allen

changed the description

mentioned in merge request Moore!1852 (merged)

mentioned in merge request Moore!1842 (merged)

• [2022-11-03 16:18] Validation started with lhcb-master-mr#6109
• [2022-11-04 09:47] Validation started with lhcb-master-mr#6115

/ci-test

added ci-test-triggered label

mentioned in merge request !1041 (merged)

added 2 commits

• b6a7f3fa - Fixed formatting
• 97ba082b - Use a service to hold and configure the Allen ROOTService

Compare with previous version