Skip to content

Fix semantics of Gaudi::Accumulators::Counter

Marco Clemencic requested to merge 144-remove-nonsensical-operator-from-gaudi-accumulators-counter into master

As described in #144 (closed), operator+= in the current Gaudi::Accumulators::Counter increments by 1 the storage regardless of the value passed. This is confusing and may lead to bugs.

With this MR I introduce a new accumulator so that Gaudi::Accumulators::Counter and bins of Gaudi::Accumulators::Histogram have a semantics closer to that of integral numbers.

Note that I had to fix a problem in the separation between axis and storage types of HistogramAccumulator to be able to use the new accumulator.

Closes #144 (closed)

Edited by Marco Clemencic

Merge request reports