a grammar for functordesc

@amathad fyi

this enables Gaudi::Property<collection<ThOr::FunctorDesc>>.
before the only possible thing were "plain" Gaudi::Property<ThOr::FunctorDesc>.

tested with std::vector<std::vector<ThOr::FunctorDesc>>.

EDIT: modified with_functor_maps.h to work with the new grammar.

mentioned in merge request Phys!930 (merged)

approved this merge request

mentioned in merge request Analysis!792 (merged)

mentioned in merge request DaVinci!539 (merged)

assigned to @ascarabo

please get the merge workflow started @ascarabo. it goes together with Phys!930 (merged)

added Accelerators label

removed Accelerators label

added lhcb-run3-cleanup label

• [2021-05-05 00:46] Validation started with lhcb-run3-cleanup#202
• [2021-05-06 00:44] Validation started with lhcb-run3-cleanup#203
• [2021-05-07 00:49] Validation started with lhcb-run3-cleanup#204
• [2021-05-08 01:06] Validation started with lhcb-run3-cleanup#205
• [2021-05-10 00:12] Validation started with lhcb-run3-cleanup#206
• [2021-05-11 00:41] Validation started with lhcb-run3-cleanup#207
• [2021-05-12 01:03] Validation started with lhcb-run3-cleanup#208
• [2021-05-13 00:59] Validation started with lhcb-run3-cleanup#209
• [2021-05-14 01:13] Validation started with lhcb-run3-cleanup#210
• [2021-05-16 00:28] Validation started with lhcb-run3-cleanup#211
• [2021-05-17 13:36] Validation started with lhcb-master-mr#2322

added 1 commit

• 6948bbb1 - update with_functor_maps accordingly

Compare with previous version

changed the description

This worked. The PID_K functors now do something, which broke the DaVinciTests ref https://lhcb-nightlies.web.cern.ch/nightly/lhcb-run3-cleanup/203/DaVinci/x86_64-centos7-gcc9-opt/tests#DaVinciTests.functors.test_thor_functors

I'll fix the ref in DaVinci!538 (merged)

Now, I'd like this merged asap. However, I see Moore is broken in lhcb-run3-cleanup (but that's the TMVA removal).

mentioned in issue Moore#273 (closed)

mentioned in issue Moore#278 (closed)

assigned to @axu and unassigned @ascarabo

Hi @nnolte, this Rec test failed in lhcb-run3-cleanup#208. Can you please have a look? Is an update of the reference file necessary?

added 1 commit

• 66207229 - Fix test

Compare with previous version

It was easier to fix the test than to make it available. Check if that makes sense to you @nnolte .

added ci-test-triggered label