Add channelID to TTVELOClusters

Extend TupleToolVELOClusters to save not just the sensor ID but the more useful channelID information. See https://lhcb-doxygen.web.cern.ch/lhcb-doxygen/lhcb/v42r10/d8/de1/class_l_h_cb_1_1_velo_channel_i_d.html

assigned to @johndan

added 1 commit

• eae4b19c - Fixed formatting

Compare with previous version

added legacy label

added all-slots label

• [2022-07-14 00:58] Validation started with lhcb-run2-patches-dev4#1117
• [2022-07-14 00:58] Validation started with lhcb-run2-gaudi-head#355
• [2022-07-14 01:00] Validation started with lhcb-run2-patches#1168
• [2022-07-15 00:56] Validation started with lhcb-run2-gaudi-head#356
• [2022-07-15 00:58] Validation started with lhcb-run2-patches-dev4#1118
• [2022-07-15 00:59] Validation started with lhcb-run2-patches#1169
• [2022-07-16 00:43] Validation started with lhcb-run2-gaudi-head#357
• [2022-07-16 00:46] Validation started with lhcb-run2-patches-dev4#1119
• [2022-07-16 00:48] Validation started with lhcb-run2-patches#1170
• [2022-07-17 00:19] Validation started with lhcb-run2-gaudi-head#358
• [2022-07-17 00:20] Validation started with lhcb-run2-patches-dev4#1120
• [2022-07-17 00:26] Validation started with lhcb-run2-patches#1171
• [2022-07-18 00:09] Validation started with lhcb-run2-patches-dev4#1121

@johndan Is this ready for merging?

If you're happy with the pipelines, I'm happy with the merge going ahead

requested review from @masmith

approved this merge request

mentioned in commit f0343b79

merged

Hi @masmith , thank you! Do you know when this will be released (and when the next DaVinci will go out?). Also a question for @pkoppenb , I suppose?

resolved all threads