Skip to content

Fix the ID Trigger naming convention in the ID Trigger configuration code, add test beamspot chain

Mark Sutton requested to merge sutt/athena:crime into master

There were inconsistencies in the way the ID Trigger sequences were configured. These changes partly address thie to tidy the configuration code somewhat. As part of this is a fix to the naming convention in the beamspot chains, together with the implementation of a test beamspot chain in the LS2 menu. This includes a renaming of the beamspot chain in the pp_run3 menu, to conform to the established naming conventin from Run 2, that has been agreed with Menu coordination.

NB: THIS IS THE SAME CHANGE SET AS !32943 (closed) which I have closed because my branch got messed up and I was no longer able to push changes from it.

This latest commit is simply to fix the merge conflict that arose on the reference file. Once the tests have completed, can this request please be merged, as if it was request 32943 ? I would prefer not to spend another 4 days on a code review that has already been commented and reviewed.

Thanks

Merge request reports