-
Notifications
You must be signed in to change notification settings - Fork 51
Sorcha/dev/zarr3 compaction #1450
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
…ental metadata writes. (#1593)
grassesi
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it would be better if zarr_store would be under general.zarr_store but this is just nitpicking at this point. Please do remove the print statements tho.
Sorry, think the commit was not the most recent! prints all gone :) i did think about putting it under a different category but the |
|
Which is a known issue with pryefly |
Description
Draft PR to showcase how to enable sharding (experimentation underway to optimise performance)
Issue Number
DRAFT
Closes #1384
Checklist before asking for review
./scripts/actions.sh lint./scripts/actions.sh unit-test./scripts/actions.sh integration-testlaunch-slurm.py --time 60