Skip to content

Cleanup/fixes for LAr(Cluster)CollisionTime Alg & Monitoring

Most important is the third commit, moving the configuration of the LArCollisionTimeAlg and LArClusterCollisionTimeAlg to the central CaloRec_jobOptions.py. The problem was that the LArClusterCollisionTimeAlg was running after the LArMonManager, leading to a per-event warning message and un-filled histograms. Because of of how the config of the old-style monitoring works its impossible to schedule these algorithms from within the monitoring configuration.

Note that with the athenaMT, this issue is not a problem because the order of algorithms is determined by the scheduler. The Run3 Monitoring framework that is more algorithm-oriented alleviates the algorithm-ordering problem as well. So this fix is mostly for legacy running.

The remaining commits of this MR are just modernizing the code.

Merge request reports