Add counters for random PV position to lumi summary

Uses the index of a lumi event within its slice modulo the number of PVs in the event to select a pseudo-random PV, and saves the co-ordinates of that PV as lumi counters. All co-ordinates are saved to 10-bit counters (X and Y in um, Z in mm) with the counter offset by 512 to allow negative values in the unsigned counters.

FYI @sxian @egraveri @edallocc @balagura

added 1 commit

• 5a30133f - Fixed formatting

Compare with previous version

added RTA label

added 1 commit

• 766a4e35 - Fix lumi summary schema

Compare with previous version

added 1 commit

• 6701cf31 - Fixed formatting

Compare with previous version

changed the description

marked this merge request as ready

changed the description

mentioned in merge request !1083 (merged)

@axu please can you run the CI tests for this MR

assigned to @axu

/ci-test

requested review from @sxian, @egraveri, @edallocc, and @balagura

added ci-test-triggered label

• [2022-12-17 11:02] Validation started with lhcb-master-mr#6612

requested review from @dcampora

requested review from @raaij

Can you mention also in this description that this MR has conflict with !1083 (merged)?

mentioned in issue Moore#506 (closed)

added needs ref update label

assigned to @sponce

unassigned @axu

changed the description

@dcraik Probably should go to draft it is waits for !1083 (merged)?

mentioned in issue Moore#518 (closed)

mentioned in issue Moore#527 (closed)

mentioned in issue Moore#530 (closed)

marked this merge request as draft

mentioned in issue Moore#540 (closed)

added 213 commits

• 6701cf31...d8fe0870 - 212 commits from branch master
• 77d9e6b1 - Merge remote-tracking branch 'origin/master' into dcraik_add_rand_vertex_to_lumi

Compare with previous version

added 1 commit

• efc780e6 - Fixed formatting

Compare with previous version

marked this merge request as ready

changed the description

@frodrigu please can you re-run the CI tests now that this is ready to merge? An update to the references is expected.

changed the description

resolved all threads

/ci-test

assigned to @frodrigu

• [2023-03-21 14:14] Validation started with lhcb-master-mr#7399
• [2023-03-22 00:06] Validation started with lhcb-gaudi-head#3510

added lhcb-gaudi-head label

resolved all threads

approved this merge request

resolved all threads

merged

mentioned in commit 0ab744d0

unassigned @frodrigu

removed lhcb-gaudi-head label