-
Notifications
You must be signed in to change notification settings - Fork 26
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
ome-ngff header for zarr streaming #6226
Conversation
webknossos-datastore/conf/com.scalableminds.webknossos.datastore.routes
Outdated
Show resolved
Hide resolved
…re.routes Co-authored-by: Norman Rzepka <[email protected]>
…bknossos into ome-ngff-zarr-streaming
# Conflicts: # webknossos-datastore/app/com/scalableminds/webknossos/datastore/controllers/ZarrStreamingController.scala
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.
LGTM :)
I assume this is not supported for volume annotations at the moment? Would that be difficult to add? If not, it would be great if you could create another PR for that (feel free to merge this one already)
It shouldn't be hard to add, I'll open a PR. |
OmeNgffHeader
ClassOmeNgffHeader
class doesn't implement all fields from the standard but only the ones that we currently needZarrHeader
classURL of deployed dev instance (used for testing):
Steps to test:
GET /data/zarr/:organization/:dataset/:layer/.zattrs
, e.g.GET /data/zarr/sample_organization/l4_sample/color/.zattrs
Issues: