Skip to content

refactored TriggerConfigHLT, now it is a single means to communicate to HLT CF building

Tomasz Bold requested to merge tbold/athena:refactor-TriggerConfigHLT into master

This MR contains refactoring of the chains repository (aka TriggerConfigHLT). The singleton behaviour is simplified. Also now, it is the only method to pass chains information to CF building. Tagging @fpastore Emulations tests are moved to it as well. It required tiny touch of PEB related tests (input makers naming) Tagging @rbielski on it. Unused elements in it are just removed. Tagging @bernius

And menu experts @jmontejo and @aranzazu since it is a cleanup of the menu code in the end. After it is merged we should be able to generate HLT menu JSON in every test we have and can migrate more components to it. Tagging @tamartin @mark & @stelzer

While dooing the tests I found an issue in CF building (or monitoring). Namely when the chains are passed to CF building in different order we end up in a different CF configuration. As a result jet chains have different counts. For the moment I used OrderdDicts so the results are intact. But this is will be something for @fpastore to look at once merged and conveniently reproducible.

Merge request reports