Skip to content
GitLab

Overlay: Add BCM new-style config + fix data overlay

Merged Tadej Novak requested to merge tadej/athena:overlay/bcm into master

This ended-up a bigger MR than expected, but here we go. The goal was to add new-style config for BCM overlay. In the process data overlay for BCM has also been fixed (ATLASSIM-4622).

Some observations:

  • BCM stores multiple BCIDs for data (5 in my tests), one for MC.
  • The order of collections is not deterministic.

Changes:

  • Map the channels to collections as direct loop is not possible.
  • For data: find event BCID from data RDOs and use that one for overlay, do not store the rest if not required.

/cc @janovak @jchapman

Edited by Tadej Novak