-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add neuroglancer as a service for .ome.zarr files #1192
Comments
FWIW - a rough way to implement it I see if this logic is expressed in JS/Vue, although possible show stopper: ATM I think the place where we use these EXTERNAL_SERVICES would not be able to even see all the files in that dandiset folder but only that "page" :-/ We could add regex to match those specific zarrs (with
where |
Another idea with another show stopper: (ab)use assets metadata and populate with some URLs for external services "offline" (cron job), so that then web UI could pick them up if any given asset has that metadata. Show stopper: I don't think that listing of assets requests metadata for them, so that would require requesting metadata for each asset -- to expensive. No show stopper idea mixed into prior idea of |
ok -- last idea, I promise, and quick to implement: implement on server side as |
i don't think it's a show stopper, but i don't think it would do what you may have thought it would do. at present it would simply enable the service for any file that matched the regex, since services are tied to assets/files rather than groups of things. |
the whole grouping and other things is indeed not implemented in anyway and would come up for json+nonjson, nwb+patchednwb, etc.,. |
btw, the whole grouping thing is available in this script: https://github.com/dandi/example-notebooks/blob/a84f4259d20c96742f25a4de6441a3af9bff27d7/000108/chunglab/demo/neuroglancer-stitched.py |
in addition to #1067, we should still add neuroglancer as a service for the ome.zarr files.
this piece of code is for multiple zarrs, but could be easily adapted i think in javascript for a single s3 url for a zarr when asset is public (i don't know how embargoed zarr assets would be dealt with, but i don't think we have those at this point).
also we can strip some of the parameters of dictionary for now.
Originally posted by @satra in #906 (comment)
The text was updated successfully, but these errors were encountered: