search

dask-contrib / dask-histogram

Histograms with task scheduling.
https://dask-histogram.readthedocs.io
BSD 3-Clause "New" or "Revised" License
23 stars 4 forks source link

[DO NOT MERGE] testing PR 38 changes after 37 got merged #40

Closed douglasdavis closed 1 year ago

douglasdavis commented 1 year ago

@lgray I'm not sure if there's anything else you'd like to do with #38, so I'm making this do-not-merge PR just to show that everything should be working with #37 + #38 (after a one tiny change in the test you wrote, on line 116)

- h.compute()
+ h = h.compute()
lgray commented 1 year ago

Yeah - let me see what it does to https://github.com/scikit-hep/hist/pull/471 as well. I'll rebase and make all the changes over on #38.

lgray commented 1 year ago

Everything is settled now! Closing this PR.