-
Notifications
You must be signed in to change notification settings - Fork 1
(chore): update benchmarks #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I'd guess Benchmarks with that dataset probably just need to rechunk right at the start. |
Yeah this changed with zarr-developers/zarr-python#2463 for the |
I was looking at that, I will look into it soon hopefully. Trying to prioritize for our next release at the moment (hence checking benchmarks etc.) |
I will run the full thing now, post the results, and then remove them if we're confident that these changes are good. Then you can run the benchmarks later on a machine whose specs you are perhaps more aware of (or we can keep what I post). |
Thanks @ilan-gold!
It'll be interesting to rerun benchmarks when zarr-python has both of these PRs released: |
This is very preliminary and partially blocked by zarr-developers/zarr-python#2971
In any case, on the machine I am using,
zarrs-python
sharding performance in dask seems quite degraded, so wanted to raise that to see if you could reproduce.Some relevatn system info potentially:
This disk is a mounted volume from denbi: https://cloud.denbi.de/wiki/quickstart/#using-cinder-volumes - not sure how much more nfo I can find