Skip to content

Conversation

@will-moore
Copy link
Member

@will-moore will-moore commented Aug 28, 2025

NB: This requires branch at ome/omero-cli-zarr#180. That needs to be merged (and released) before this.

Screenshot 2025-08-28 at 16 20 38

To test, try with various public URLs, e.g:

NB: OME-Zarr v0.5 not supported yet; running on merge-ci etc doesn't support zarr-python v3.

TODO:

  • Support bioformats2raw.layout
  • Handle zarr thumbnail errors - invalid URL, thumbnail too big etc.
  • Handle import errors
  • Disable URL after Import starts. Make Import button disabled status clearer
  • Enforce Dataset is not None
  • Once this is merged, consider updating ope-with config on merge-ci and at https://github.com/ome/devspace/blob/master/home/jobs/OMERO-web/config.xml

@will-moore will-moore requested a review from dominikl September 1, 2025 09:24
@dominikl
Copy link
Member

dominikl commented Sep 3, 2025

Looks good to me. Are you going to handle plates too in this PR?

@will-moore
Copy link
Member Author

No, plates might need a bit more thought.

@will-moore
Copy link
Member Author

@jburel feedback - Need to handle case where you Open with > Import a Dataset where you don't have write permissions (to create Image links)

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.

2 participants