boostorg / boostorg/histogram

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

Open
#350 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
334
Forks
76
PR merge metrics
No merged PRs in 30d

Description

```c++
// 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.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.