New histogram collection: VectorizedHistCollection

[kreczko]

Fixes #39, #171

Introduces VectorizedHistCollection that adds two things w.r.t. BaseHistCollection:

• access multiple histograms at once
• fill multiple histograms at once

Accessing collection[pileup] with pileup = [1, 12, 1, 50] will automatically return the histograms for the correct pileup bins (not values). This part is generalized through the VectorizedBinProxy.

Filling histograms can be done with either scalar, numpy.ndarray or awkward.JaggedArray.

VectorizedHistCollection also introduces some experimental functionality, e.g. execute_before_write.
The execute_before_write parameter can be used to schedule function calls before the collection is written to disk. E.g. this could replace the EfficiencyCollection's _calculateEfficiencies call (i.e. a step towards the generalization of histogram collections). This part is generalized through the VectorizedHistProxy.

Example

import awkward
from cmsl1t.collections import VectorizedHistCollection

puBins = list(range(0, 50, 10)) + [999]
collection = VectorizedHistCollection(innerBins=puBins, innerLabel='pu')
collection.add('test', bins=[35, 90, 120])

pileup = [1, 12, 1, 50]
jet_et = awkward.fromiter([
        [60, 50, 40, 30, 20],
        [32, 23],
        [56, 34, 31],
        [],
    ])

collection[pileup]['test'].fill(jet_et)

Further work needed

• documentation
• more tests & benchmarks (> 2 dimensions, scalar values)
• allowing for different histogram proxies: e.g. special fill for efficiencies

[lgtm-com]

This pull request fixes 1 alert when merging e643ec9 into ead9128 - view on LGTM.com

fixed alerts:

• 1 for Unreachable code

[lgtm-com]

This pull request fixes 1 alert when merging 114f53c into ead9128 - view on LGTM.com

fixed alerts:

• 1 for Unreachable code

[lgtm-com]

This pull request fixes 1 alert when merging 09178f4 into ead9128 - view on LGTM.com

fixed alerts:

• 1 for Unreachable code

[lgtm-com]

This pull request fixes 1 alert when merging 51d44d3 into ead9128 - view on LGTM.com

fixed alerts:

• 1 for Unreachable code

[benkrikler]

Lots of questions and comments for you Luke, hopefully as you wanted!

[kreczko]

ok, @benkrikler, that should be that for now. I am addressing the jetMet_analyzer in a separate branch

[lgtm-com]

This pull request fixes 1 alert when merging cb28ed4 into ead9128 - view on LGTM.com

fixed alerts:

• 1 for Unreachable code