Skip to content

Use the public storage_type attribute of boost-histogram - #46

Open
davidwalter2 wants to merge 1 commit into
bendavid:mainfrom
davidwalter2:260830_storageTypeApi
Open

Use the public storage_type attribute of boost-histogram#46
davidwalter2 wants to merge 1 commit into
bendavid:mainfrom
davidwalter2:260830_storageTypeApi

Conversation

@davidwalter2

Copy link
Copy Markdown
Contributor

boost-histogram renamed _storage_type to storage_type. The leading
underscore marked it private, so reading it was always a bet on an
implementation detail, and current versions no longer provide it at all.

make_array_interface_view and hist_to_pyroot_boost therefore raise
AttributeError before doing any work, which makes histogram production fail
on the current wmassdevrolling container.

Four call sites, mechanical: _storage_type -> storage_type.

🤖 Generated with Claude Code

https://claude.ai/code/session_01HrsrmERvJF2Vafn7wE1B3v

boost-histogram renamed _storage_type to storage_type; the leading
underscore marked it private, so relying on it was always a bet on an
implementation detail. The current version no longer provides it, and
make_array_interface_view and hist_to_pyroot_boost raise AttributeError
before doing any work.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant