Remove L0 related code

L0 is obsolete in Run3 and must be removed from master. It includes:

• Remove L0 event model classes
• Remove L0 and SPD/PRS from Calo code
• Remove L0 from Relation event classes
• Remove L0 functors
• Remove L0/SPD/PRS/PU decoder configuration (and obsolete configuration and tests)
• Remove pile-up decoding

added cleanup label

@cattanem Is that good enough to build on the dedicated slot?

mentioned in merge request !2736 (merged)

mentioned in merge request !2737 (closed)

mentioned in merge request !2738 (closed)

added 1 commit

• 56dc96ba - forgot cmakelist

Compare with previous version

mentioned in merge request !2739 (merged)

mentioned in merge request !2740 (merged)

added 1 commit

• 89694025 - Also add changes in LHCbAlgs here

Compare with previous version

Added a few more MR that depend on it. This also includes !2741 (closed) as the ST code would otherwise not compile. (!2742 (closed) would, but is also done for consistency).

(There are also broken pipelines. I'll check once the compiler is happy.)

• [2020-09-01 00:06] Validation started with lhcb-run3-cleanup#2
• [2020-09-02 00:11] Validation started with lhcb-run3-cleanup#3
• [2020-09-03 00:12] Validation started with lhcb-run3-cleanup#4
• [2020-09-04 00:07] Validation started with lhcb-run3-cleanup#5
• [2020-09-05 00:10] Validation started with lhcb-run3-cleanup#6
• [2020-09-06 00:06] Validation started with lhcb-run3-cleanup#7

The nightly setup worked. Thanks @cattanem . A few compilation errors in LHCb (all with Calo, !2736 (merged)). And plenty of other errors in dependent projects, as expected.

Question to @cattanem , @rmatev , @bcouturi : lhcb-run3-cleanup#2 is not mapped to cvmfs it seems. Can it be? That would be useful to build the dependent projects.

mentioned in merge request Boole!299 (merged)

mentioned in merge request Phys!772 (merged)

mentioned in merge request Phys!773 (merged)

mentioned in merge request Phys!774 (merged)

mentioned in merge request Rec!2194 (closed)

mentioned in merge request Rec!2195 (merged)

mentioned in merge request Rec!2196 (merged)

mentioned in merge request Rec!2197 (merged)

mentioned in merge request Analysis!687 (merged)

mentioned in merge request Analysis!688 (merged)

mentioned in merge request Panoramix!77 (merged)

mentioned in merge request Alignment!141 (merged)

mentioned in merge request Rec!2198 (merged)

mentioned in merge request Castelao!78 (closed)

• [2020-09-08 00:06] Validation started with lhcb-run3-cleanup#8
• [2020-09-09 00:13] Validation started with lhcb-run3-cleanup#9
• [2020-09-10 00:06] Validation started with lhcb-run3-cleanup#10
• [2020-09-11 00:10] Validation started with lhcb-run3-cleanup#11
• [2020-09-12 00:10] Validation started with lhcb-run3-cleanup#12
• [2020-09-14 00:10] Validation started with lhcb-run3-cleanup#13
• [2020-09-15 00:50] Validation started with lhcb-run3-cleanup#14

The title of this MR is a bit misleading, as all the other MRs also target master. But I understand that this was the major cleanup and the other MRs followed? Could this be reflected in the title please @pkoppenb ?

changed title from Remove L0 from master to Remove L0 event classes from master

changed the description

changed title from Remove L0 event classes from master to Remove L0 event classes

@dovombru I tried making it clearer.

assigned to @vrenaudi

It seems that with this MR we are also removing most instances of "slave" in our physics software slack. @olafs @fdordei @sponce

mentioned in issue Moore#206 (closed)

mentioned in merge request !2752 (merged)

added 1 commit

• 5083f757 - Remove L0 from CMakeList.txt

Compare with previous version

Added some forgotten CMakeList.txt fixes

resolved all threads

Finally all projects compile on https://lhcb-nightlies.web.cern.ch/nightly/lhcb-run3-cleanup/build/11/ . I'd like to move all MR tagged lhcb-run3-cleanup to all-slots and see what happens there. There are some failing tests in LHCb, Rec, Boole, Brunel, DaVinci that I need to look into. @cattanem , @dovombru , @vrenaudi . Is it OK to go ahead?

mentioned in merge request !2757 (merged)

mentioned in merge request !2758 (closed)

mentioned in merge request Boole!308 (closed)

assigned to @gunther and unassigned @vrenaudi

Most of the Brunel tests are very easy (PVChecker no longer gives L0 stats), while two of them are related to changes in tracking. That needs further investigation.

mentioned in merge request Brunel!1075 (merged)