Draft: Remove MVADictTools

Following the discussion at https://gitlab.cern.ch/lhcb-dpa/project/-/issues/95#note_4384968, trying to remove MVADictTools to see what happens.

added cleanup lhcb-run3-cleanup labels

/ci-test --merge

marked this merge request as ready

added ci-test-triggered label

• [2021-04-12 16:16] Validation started with lhcb-master-mr#2179
• [2021-04-13 00:44] Validation started with lhcb-run3-cleanup#184
• [2021-04-13 08:52] Validation started with lhcb-master-mr#2184
• [2021-04-14 01:04] Validation started with lhcb-run3-cleanup#185
• [2021-04-15 01:21] Validation started with lhcb-run3-cleanup#186
• [2021-04-16 01:17] Validation started with lhcb-run3-cleanup#187
• [2021-04-17 01:23] Validation started with lhcb-run3-cleanup#188
• [2021-04-19 01:31] Validation started with lhcb-run3-cleanup#189
• [2021-04-20 00:58] Validation started with lhcb-run3-cleanup#190
• [2021-04-21 01:22] Validation started with lhcb-run3-cleanup#191
• [2021-04-22 01:02] Validation started with lhcb-run3-cleanup#192
• [2021-04-23 01:24] Validation started with lhcb-run3-cleanup#193
• [2021-04-24 01:38] Validation started with lhcb-run3-cleanup#194
• [2021-04-26 00:54] Validation started with lhcb-run3-cleanup#195
• [2021-04-27 01:13] Validation started with lhcb-run3-cleanup#196
• [2021-04-28 01:15] Validation started with lhcb-run3-cleanup#197
• [2021-04-29 01:30] Validation started with lhcb-run3-cleanup#198
• [2021-04-30 01:29] Validation started with lhcb-run3-cleanup#199
• [2021-05-01 01:28] Validation started with lhcb-run3-cleanup#200
• [2021-05-04 00:31] Validation started with lhcb-run3-cleanup#201
• [2021-05-05 00:50] Validation started with lhcb-run3-cleanup#202
• [2021-05-06 00:45] Validation started with lhcb-run3-cleanup#203
• [2021-05-07 00:51] Validation started with lhcb-run3-cleanup#204
• [2021-05-08 01:08] Validation started with lhcb-run3-cleanup#205
• [2021-05-10 00:13] Validation started with lhcb-run3-cleanup#206
• [2021-05-11 00:43] Validation started with lhcb-run3-cleanup#207
• [2021-05-12 01:04] Validation started with lhcb-run3-cleanup#208
• [2021-05-13 01:00] Validation started with lhcb-run3-cleanup#209
• [2021-05-14 01:14] Validation started with lhcb-run3-cleanup#210
• [2021-05-16 00:29] Validation started with lhcb-run3-cleanup#211
• [2021-05-18 01:08] Validation started with lhcb-run3-cleanup#212
• [2021-05-19 00:55] Validation started with lhcb-run3-cleanup#213
• [2021-05-20 01:09] Validation started with lhcb-run3-cleanup#214
• [2021-05-21 01:22] Validation started with lhcb-run3-cleanup#215
• [2021-05-22 01:25] Validation started with lhcb-run3-cleanup#216
• [2021-05-25 00:45] Validation started with lhcb-run3-cleanup#217
• [2021-05-26 01:19] Validation started with lhcb-run3-cleanup#218
• [2021-05-27 01:14] Validation started with lhcb-run3-cleanup#219
• [2021-05-28 01:28] Validation started with lhcb-run3-cleanup#220
• [2021-05-29 01:43] Validation started with lhcb-run3-cleanup#221
• [2021-05-30 00:46] Validation started with lhcb-run3-cleanup#222
• [2021-05-31 00:32] Validation started with lhcb-run3-cleanup#223
• [2021-06-01 01:17] Validation started with lhcb-run3-cleanup#224
• [2021-06-02 01:13] Validation started with lhcb-run3-cleanup#225
• [2021-06-03 00:50] Validation started with lhcb-run3-cleanup#226

Not that easy. It breaks Analysis for no good reason. That's easy to fix.

But ParticleFilterWithTMVA https://gitlab.cern.ch/lhcb-nightlies/Moore/blob/772ca54f6725596d708f1ac2cae702ed8e5704a3/Hlt/Hlt2Conf/python/Hlt2Conf/algorithms.py#L23 is used in selections.

mentioned in merge request Analysis!783 (merged)

I'll check tomorrow if this package can be cleaned up without breaking anything.

marked this merge request as draft

Ah, is this the dependency removal that @rmatev mentioned at the PAC meeting? Else the package may be better in LHCb? (Not in Phys given that the latter will be gone this year.)

@rmatev , seen the minutes now. If it's easy to fix, I can give it a try (with instructions ;).

added DPA-WP3 label

mentioned in issue Moore#272 (closed)

This breaks TupleToolIsoGeneric. I'll try removing that.

mentioned in merge request Analysis!725 (merged)

mentioned in merge request Analysis!781 (merged)

mentioned in issue Moore#273 (closed)

mentioned in issue Moore#278 (closed)

This MR (I presume) breaks Moore in https://lhcb-nightlies.web.cern.ch/nightly/lhcb-run3-cleanup/ . What is the procedure to fix that ? @axu