Remove velo, ST, OT
Removing obsolete detectors. See https://gitlab.cern.ch/lhcb-dpa/project/-/issues/37
Changes:
- Vis/SoEvent : as much as possible, velo -> VP, ST -> UT, OT -> FT. Some functionality seems missing. See PK-R3C tags in code.
Loosely depends on LHCb!2804 (merged) (would likely build without it but better merged at the same time).
Merge request reports
Activity
added lhcb-run3-cleanup label
- [2020-10-23 00:11] Validation started with lhcb-run3-cleanup#48
- [2020-10-24 00:11] Validation started with lhcb-run3-cleanup#49
- [2020-10-25 00:10] Validation started with lhcb-run3-cleanup#50
- [2020-10-26 00:12] Validation started with lhcb-run3-cleanup#51
- [2020-10-27 00:11] Validation started with lhcb-run3-cleanup#52
- [2020-10-28 00:12] Validation started with lhcb-run3-cleanup#53
- [2020-10-29 00:15] Validation started with lhcb-run3-cleanup#54
- [2020-10-30 00:14] Validation started with lhcb-run3-cleanup#55
- [2020-10-31 00:17] Validation started with lhcb-run3-cleanup#56
- [2020-11-01 00:10] Validation started with lhcb-run3-cleanup#57
- [2020-11-02 00:15] Validation started with lhcb-run3-cleanup#58
- [2020-11-03 00:12] Validation started with lhcb-run3-cleanup#59
- [2020-11-04 00:18] Validation started with lhcb-run3-cleanup#60
- [2020-11-05 00:16] Validation started with lhcb-run3-cleanup#61
- [2020-11-06 00:11] Validation started with lhcb-run3-cleanup#62
- [2020-11-07 00:12] Validation started with lhcb-run3-cleanup#63
- [2020-11-08 00:11] Validation started with lhcb-run3-cleanup#64
- [2020-11-10 00:12] Validation started with lhcb-run3-cleanup#65
- [2020-11-11 00:58] Validation started with lhcb-run3-cleanup#66
- [2020-11-12 00:13] Validation started with lhcb-run3-cleanup#67
- [2020-11-13 00:11] Validation started with lhcb-run3-cleanup#68
- [2020-11-14 00:13] Validation started with lhcb-run3-cleanup#69
- [2020-11-17 00:15] Validation started with lhcb-run3-cleanup#70
- [2020-11-18 00:27] Validation started with lhcb-run3-cleanup#71
- [2020-11-19 00:13] Validation started with lhcb-run3-cleanup#72
- [2020-11-20 00:41] Validation started with lhcb-run3-cleanup#73
- [2020-11-21 00:29] Validation started with lhcb-run3-cleanup#74
- [2020-11-22 00:22] Validation started with lhcb-run3-cleanup#75
- [2020-11-23 00:21] Validation started with lhcb-run3-cleanup#76
- [2020-11-24 00:24] Validation started with lhcb-run3-cleanup#77
- [2020-11-25 00:22] Validation started with lhcb-run3-cleanup#78
- [2020-11-26 00:21] Validation started with lhcb-run3-cleanup#79
- [2020-11-27 00:28] Validation started with lhcb-run3-cleanup#80
- [2020-11-28 00:21] Validation started with lhcb-run3-cleanup#81
- [2020-11-29 00:16] Validation started with lhcb-run3-cleanup#82
- [2020-11-30 00:13] Validation started with lhcb-run3-cleanup#83
- [2020-12-01 00:10] Validation started with lhcb-run3-cleanup#84
- [2020-12-02 00:26] Validation started with lhcb-run3-cleanup#85
- [2020-12-03 00:23] Validation started with lhcb-run3-cleanup#86
- [2020-12-03 15:35] Validation started with lhcb-master-mr#1691
- [2020-12-03 15:35] Validation started with lhcb-head-2#97
- [2020-12-04 03:12] Validation started with lhcb-run3-cleanup#87
- [2020-12-05 01:14] Validation started with lhcb-run3-cleanup#88
- [2020-12-06 00:26] Validation started with lhcb-run3-cleanup#89
- [2020-12-08 01:44] Validation started with lhcb-run3-cleanup#90
- [2020-12-08 11:23] Validation started with lhcb-head-2#104
- [2020-12-09 08:36] Validation started with lhcb-head-2#105
- [2020-12-09 08:48] Validation started with lhcb-run3-cleanup#91
- [2020-12-09 16:50] Validation started with lhcb-head-2#106
- [2020-12-10 00:47] Validation started with lhcb-run3-cleanup#92
- [2020-12-10 01:09] Validation started with lhcb-head-2#107
- [2020-12-10 07:50] Validation started with lhcb-head-2#108
Edited by Software for LHCbHi @pkoppenb, trying to help a bit - I just checked and this PR creates no issue, hence it can be merged. Now, who has merging rights in Panoramix...?
Thanks @erodrigu always appreciated.
- My last MR here was merged by @dovombru . So I presume it's under RTA control somehow.
- There are issues in the code that I would like to be looked at. For instance the absence of FT here !80 (diffs) is suspicious. Also this !80 (diffs).
mentioned in merge request DaVinci!469 (merged)
added 7 commits
-
c41e0b7a...2d0bcd85 - 6 commits from branch
master
- 8bce5b2e - Remove velo, ST, OT
-
c41e0b7a...2d0bcd85 - 6 commits from branch
added all-slots label
Logically dependent on LHCb!2804 (merged)
removed all-slots label
@pkoppenb, @sponce, shouldn't the CI be triggered here to test the full chain including the related MR to LHCb? Cc'ing also @bcouturi since I believe he has been maintaining Panoramix. In DPA we have many topics with MRs "just about to be mergeable" and it would be great to go that final step, to tick off several things and move on. Help appreciated. Thanks.