Skip to content

fixes for LoadDictionaries in EL::Worker

Nils Erik Krumnack requested to merge krumnack/athena:el_fix_30Sep into 21.2

This does two things:

  • It fixes my update from last week which did execute the LoadDictionaries too late to do much good.
  • It adds a hot-fix of just always loading the dictionaries in distributed running, until a more proper fix can be implemented. This is inspired by a discussion with @akraszna in !36868 (closed), but adds it in a more appropriate place (roughly where it should occur when the proper fix is in place), marks it clearly as a temporary hot-fix, and probably most important adds it to the execution paths for both batch and grid running. The last was probably just an oversight in the previous MR. I didn't add it to xAODMerge, since @akraszna called that unnecessary.

Not sure I put the proper fix in immediately, once the hot-fix is in place, but as noted, for unrelated reasons that restructuring should happen.

Merge request reports