Confusing error on `make_histogram(axis::integer(0, 1), dense_storage())`

boostorg / histogram

Fast multi-dimensional generalized histogram with convenient interface for C++14

Boost Software License 1.0

313 stars 74 forks source link

Confusing error on `make_histogram(axis::integer(0, 1), dense_storage())` #350

Open HDembinski opened 2 years ago

HDembinski commented 2 years ago

// c++17
auto h = bh::make_histogram(bh::axis::integer(0, 1), bh::dense_storage());

This does not work (and currently shouldn't!) but fails in mysterious ways that are difficult to comprehend. If possible, we should make this work and deprecate make_histogram_with or fail at the earliest. The code interprets the second argument as an axis, but hopefully we can prevent that with better constraints on the template.