-
Notifications
You must be signed in to change notification settings - Fork 2
Initial support for Cellmap-style N5 data sets #276
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
base: main
Are you sure you want to change the base?
Conversation
|
I'm just going to link a bunch of cross references here:
|
|
WebKnossos has stricter parsing than Neuroglancer:
|
|
COSEM metadata refers to a |
|
List of CellMap N5 datasets:
Ran in |
|
Thanks, @mkitti ! I tried the 4 examples you listed and they all seem to work. The Neuroglancer docs you linked say that "The pixelResolution attribute is not recommended". As we discussed, I'm not sure it's a good idea for Fileglancer to attempt to support all possible metadata schemas for N5. There is an easy fallback, which is just viewing the metadata directly (we'll add JSON formatting to make that easier.) Please let me know if there are remaining other "must have" requirements for N5 for this feature to be useful. |
|
I do agree we should not support everything, but I am looking at datasets modified timestamps from a week ago that use I just spoke with @trautmane, and he just confirmed for me that the active render pipeline writes There are also seems to be markdown documentation which is distinct from the rST documentaiton. Both the render pipeline and and the Neuroglancer documentation have the following keys in common:
There seems to redundancy between I think the priority is then as follows:
|
|
I think it'll be great if we add validation for metadata required for neuroglancer, and n5-viewer. I also agree with @mkitti's comment about pixel resolution. I am not sure about Cosem metadata though. In addition to the cosem metadata, all cosem/cellmap n5 datasets have all attributes ( |
|
I described a couple of "dialects" that have been used in the building awhile ago: Agreed its not worth supporting them all. |
|
My proposal is basically we should support (set of keys supported by Neuroglancer) ∩ (set of keys actively used at Janelia). At the moment, that intersection seems to be equivalent to (set of keys supported by Neuroglancer). The set of keys supported by Neuroglancer is currently a strict subset of the keys used at Janelia. This is in part because we have made changes so that we export keys supported by Neuroglancer. |
This PR adds some support for Cellmap-style N5 data sets. In particular, we assume something is a compatible data set if it has an
attributes.jsonand there is as0child directory. We then read theattributes.jsonands0/attributes.jsonand display relevant information in the metadata tables similar to the Zarr implementation. Links to Neuroglancer and "Copy data URL" are provided for now.Feedback is welcome from anyone with some data sets to try. I'll ping everyone again once we have this up on the dev server, if you don't want to bother with building it locally.
@mkitti @JaneliaSciComp/fileglancer @StephanPreibisch @yuriyzubov