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.

Sign up for GitHub

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

Jump to bottom

Clarify a few aspects of the OMERO Zarr pixel buffer integration #15

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Clarify a few aspects of the OMERO Zarr pixel buffer integration #15

wants to merge 2 commits into from

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Feb 20, 2025

  • the lsid must point to the multiscales group
  • provide an example to a Zarr dataset on a public S3 bucket
  • clarify the credentials provider chain and link to the AWS doc

/cc @dominikl @will-moore

@sbesson
Clarify a few aspects of the OMERO Zarr pixel buffer integration
0aba71d 
- the lsid must point to the multiscales group
- provide an example to a  Zarr dataset on a public S3 bucket
- clarify the credentials provider chain and link to the AWS doc
@sbesson sbesson requested a review from erindiel February 20, 2025 10:46
will-moore
will-moore reviewed Feb 20, 2025
View reviewed changes
will-moore
will-moore approved these changes Feb 20, 2025
View reviewed changes
Copy link

@will-moore will-moore left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, thx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@will-moore will-moore will-moore approved these changes

@erindiel erindiel Awaiting requested review from erindiel

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sbesson @will-moore