Remove HighPtJets, ClearDaughters and RelatedInfoTools

One more step toward #358

assigned to @pkoppenb

mentioned in issue #358

marked this merge request as draft

added RTA label

added 1 commit

• a61cbdf0 - Reinstate WeightedRelationTable.cpp

Compare with previous version

added 1 commit

• 1035777d - Restore MuonProbeToLongMatcher.cpp

Compare with previous version

marked this merge request as ready

added DPA-WP3 cleanup lhcb-run3-cleanup labels

mentioned in merge request !3433 (closed)

• [2023-05-24 19:35] Validation started with lhcb-master-mr#8006
• [2023-05-24 20:47] Validation started with lhcb-master-mr#8007
• [2023-05-24 21:00] Validation started with lhcb-master-mr#8007
• [2023-05-24 21:12] Validation started with lhcb-master-mr#8007
• [2023-05-25 00:27] Validation started with lhcb-run3-cleanup#837
• [2023-05-25 08:22] Validation started with lhcb-master-mr#8021
• [2023-05-26 00:31] Validation started with lhcb-run3-cleanup#838
• [2023-05-27 00:26] Validation started with lhcb-run3-cleanup#839
• [2023-05-28 00:21] Validation started with lhcb-run3-cleanup#840
• [2023-05-29 00:29] Validation started with lhcb-run3-cleanup#841
• [2023-05-30 00:27] Validation started with lhcb-run3-cleanup#842
• [2023-05-31 00:27] Validation started with lhcb-run3-cleanup#843
• [2023-06-01 00:29] Validation started with lhcb-run3-cleanup#844
• [2023-06-02 00:29] Validation started with lhcb-run3-cleanup#845
• [2023-06-03 00:24] Validation started with lhcb-run3-cleanup#846
• [2023-06-03 22:09] Validation started with lhcb-master-mr#8146
• [2023-06-03 22:27] Validation started with lhcb-master-mr#8146
• [2023-06-04 00:26] Validation started with lhcb-run3-cleanup#847
• [2023-06-05 00:27] Validation started with lhcb-run3-cleanup#848
• [2023-06-05 15:23] Validation started with lhcb-master-mr#8169
• [2023-06-06 00:28] Validation started with lhcb-run3-cleanup#849

/ci-test --platforms=x86_64_v3-centos7-gcc11-opt+g

added ci-test-triggered label

resolved all threads

/ci-test --platforms=x86_64_v3-centos7-gcc11-opt+g

/ci-test

added hlt2-throughput-decreased label

added 32 commits

• 1035777d...8ecf32f4 - 31 commits from branch master
• 8685e531 - Merge branch 'master' into pkoppenb-RM

Compare with previous version

added 2 commits

• 33348cae - Restore RelationTables.cpp
• b1b2c4be - Merge branch 'pkoppenb-RM' of ssh://gitlab.cern.ch:7999/lhcb/Rec into pkoppenb-RM

Compare with previous version

marked this merge request as draft

/ci-test --platforms=x86_64_v3-centos7-gcc11-opt+g

/ci-test --platforms=x86_64_v2-centos7-gcc11+detdesc-opt

• [2023-06-06 09:12] Validation started with lhcb-master-mr#8175
• [2023-06-07 00:27] Validation started with lhcb-run3-cleanup#850
• [2023-06-08 00:28] Validation started with lhcb-run3-cleanup#851
• [2023-06-09 00:31] Validation started with lhcb-run3-cleanup#852
• [2023-06-10 00:29] Validation started with lhcb-run3-cleanup#853
• [2023-06-11 00:22] Validation started with lhcb-run3-cleanup#854
• [2023-06-12 00:24] Validation started with lhcb-run3-cleanup#855
• [2023-06-13 00:31] Validation started with lhcb-run3-cleanup#856
• [2023-06-14 00:29] Validation started with lhcb-run3-cleanup#857
• [2023-06-15 00:27] Validation started with lhcb-run3-cleanup#858
• [2023-06-16 00:29] Validation started with lhcb-run3-cleanup#859
• [2023-06-17 00:30] Validation started with lhcb-run3-cleanup#860
• [2023-06-18 00:26] Validation started with lhcb-run3-cleanup#861
• [2023-06-19 00:24] Validation started with lhcb-run3-cleanup#862
• [2023-06-20 00:27] Validation started with lhcb-run3-cleanup#863
• [2023-06-21 00:30] Validation started with lhcb-run3-cleanup#864
• [2023-06-22 00:24] Validation started with lhcb-run3-cleanup#865
• [2023-06-23 00:29] Validation started with lhcb-run3-cleanup#866
• [2023-06-24 00:24] Validation started with lhcb-run3-cleanup#867
• [2023-06-25 00:24] Validation started with lhcb-run3-cleanup#868
• [2023-06-26 00:31] Validation started with lhcb-run3-cleanup#869
• [2023-06-27 00:27] Validation started with lhcb-run3-cleanup#870
• [2023-06-28 00:28] Validation started with lhcb-run3-cleanup#871
• [2023-06-28 13:57] Validation started with lhcb-master-mr#8396
• [2023-06-29 00:26] Validation started with lhcb-run3-cleanup#872
• [2023-06-30 00:22] Validation started with lhcb-run3-cleanup#873
• [2023-07-01 00:24] Validation started with lhcb-run3-cleanup#874
• [2023-07-02 00:23] Validation started with lhcb-run3-cleanup#875
• [2023-07-03 00:27] Validation started with lhcb-run3-cleanup#876
• [2023-07-03 17:45] Validation started with lhcb-master-mr#8449
• [2023-07-04 00:29] Validation started with lhcb-run3-cleanup#877
• [2023-07-05 00:24] Validation started with lhcb-run3-cleanup#878
• [2023-07-05 10:15] Validation started with lhcb-master-mr#8459

marked this merge request as ready

resolved all threads

added 19 commits

• b1b2c4be...ba04d732 - 18 commits from branch master
• 7c7ec1cd - Merge branch 'master' into pkoppenb-RM

Compare with previous version

mentioned in issue Moore#598 (closed)

mentioned in issue Moore#599 (closed)

Hi @pkoppenb, is this ready to be tested?

I think so, @axu

removed hlt2-throughput-decreased label

mentioned in issue #411

added 32 commits

• 7c7ec1cd...c993b8cd - 28 commits from branch master
• 6dcf8027 - move forward
• 7696e3fb - Reinstate WeightedRelationTable.cpp
• d8b26f35 - Restore MuonProbeToLongMatcher.cpp
• 58351cb6 - Restore RelationTables.cpp

Compare with previous version

assigned to @axu

unassigned @pkoppenb

assigned to @rmatev and unassigned @axu

• [2023-07-06 00:31] Validation started with lhcb-run3-cleanup#879
• [2023-07-07 00:32] Validation started with lhcb-run3-cleanup#880
• [2023-07-08 00:29] Validation started with lhcb-run3-cleanup#881
• [2023-07-09 00:24] Validation started with lhcb-run3-cleanup#882
• [2023-07-10 00:28] Validation started with lhcb-run3-cleanup#883
• [2023-07-11 00:25] Validation started with lhcb-run3-cleanup#884
• [2023-07-12 00:26] Validation started with lhcb-run3-cleanup#885
• [2023-07-13 00:23] Validation started with lhcb-run3-cleanup#886
• [2023-07-14 00:14] Validation started with lhcb-run3-cleanup#887

resolved all threads

resolved all threads

merged

mentioned in commit c036841d

mentioned in commit Moore@4b4c83b3

mentioned in merge request Moore!2515 (merged)

mentioned in commit Moore@dbbe674d

Hi @pkoppenb, I just realized that this MR that got merge affects this file where there are defined some algorithms adopted for Run 3 software. Should it be the case? The algorithm SelectionFromWeightedRelationTable was introduced by !3434 (merged) to get a selection of particle that is a persistable object.

Hi @tfulghes . Oh that must have been an unintentional effect of a conflict resolution. Apologies. Can you please re-add it in a new MR? And also make sure there is a test of this algorithm somewhere. If this MR got merged it must have been that no test broke.

It was tested in some HLT2 lines with Moore!2370 (merged). With the last commit it was reverted back to the previous version, however it was a workaround necessary to solve the problem pointed in this thread here: Moore!2207 (comment 6749361). Should I recreate MR reverting the last changes?

mentioned in commit Moore@8c4885d6

mentioned in commit Moore@8710bf45

mentioned in issue Moore#623 (closed)

mentioned in commit Moore@e85f1122