Skip to content

Frozen binnings - #143

Merged
janpipek merged 17 commits into
mainfrom
frozen-binnings
Aug 24, 2026
Merged

janpipek merged 17 commits into
mainfrom
frozen-binnings

Conversation

@janpipek

@janpipek janpipek commented Jul 18, 2026 •

Copy link
Copy Markdown
Owner

Refactor binning instances to frozen attrs models.

This has a few consequences:

  • they cannot change in-place, thus all changes in an adaptive histogram bins is performed as replacing one binning with another one
  • instead of "adaptation", we call the operation "coercing" (with two different implementations for values and for another histogram)
  • the API of BinningBase is slightly changed
  • the semantics of includes_right_edge, align, range, ..., etc. in fixed_width_binning is slightly changed

Some other changes:

  • the fixed width as the only adaptable

Testing:

  • all tests pass
  • all documentation notebooks render reasonably

@janpipek
janpipek marked this pull request as ready for review August 24, 2026 14:38
@janpipek
janpipek merged commit 742e5d6 into main Aug 24, 2026
9 checks passed
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