Skip to content
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

Dump sample scripts - maybe add to docs #303

Merged
merged 5 commits into from
Jan 20, 2025

Conversation

will-moore
Copy link
Member

@will-moore will-moore commented Sep 5, 2023

Several forum discussions etc have asked for more detailed examples on how to write OME-Zarr.

Added 2 new examples using different strategies to write images that are bigger than memory:

  • Write tiles to disk until we have whole image. Then downsample to build OME-Zarr.
  • Use dask to stitch delayed tiles together, then write_image(dask_data).

@will-moore will-moore marked this pull request as draft September 5, 2023 09:09
@joshmoore
Copy link
Member

Anything you need here, @will-moore ?

@will-moore
Copy link
Member Author

@joshmoore Thanks for the ping. Do you think these look useful and they are in line with "best practice" for how to achieve what was asked for in https://forum.image.sc/t/writing-tile-wise-ome-zarr-with-pyramid-size/85063?

If so, I'll tidy them up, add a few words of intro/explanation etc.

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/weird-convertion-error-about-ngff-converter/101997/2

@imagesc-bot
Copy link

This pull request has been mentioned on Image.sc Forum. There might be relevant details there:

https://forum.image.sc/t/write-sparse-ome-zarr-or-ome-tiff-with-spatial-offset/101906/6

@joshmoore
Copy link
Member

If we're sending people to this from image.sc, shall we get them into the repo?

@will-moore will-moore marked this pull request as ready for review January 16, 2025 15:16
@will-moore
Copy link
Member Author

@josh - just tested and tweaked those 2 scripts... Ready to review/merge now, thx.

@joshmoore
Copy link
Member

Thanks, @will-moore. Let's go ahead and get this merged, but depending on how we move forward with this page, we might want to extract the salient features of those image.sc posts and just describe the snippets here rather than relying on just the link.

@joshmoore joshmoore merged commit 14f407f into ome:master Jan 20, 2025
10 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.

3 participants