Make default value of event_variables None

Change the default value of event_variables to None instead of an empty FunctorCollection().

requested review from @erodrigu and @pkoppenb

assigned to @amathad

added 1 commit

• d3a05e03 - fix linting

Compare with previous version

@pkoppenb : Small fix. Could you please trigger a test and merge?

added 1 commit

• 2fb6a213 - Fixed formatting

Compare with previous version

Hey @amathad, is there really a bug? I thought the functions get_... were behaving as expected?

>>> from FunTuple.FunctorCollection import FunctorCollection
>>> v = FunctorCollection()
>>> v
<FunctorCollection: n_thor_fctors=0, n_loki_fctors=0>
>>> v.get_thor_functors()
{}

And if not, I would rather fix at source, meaning at the level of the called functions. Apologies if I'm missing something, just that I got surprised. ["Detail": is there some unit test missing somewhere? Bugs should get tests to be avoided in the future.]

added 1 commit

• 7400787c - small edit

Compare with previous version

added 1 commit

• 74572a75 - move preamble

Compare with previous version

approved this merge request

/ci-test

added ci-test-triggered label

• [2022-07-31 09:25] Validation started with lhcb-master-mr#5273

resolved all threads

mentioned in commit 716b8c3b

merged

added cleanup label