-
Notifications
You must be signed in to change notification settings - Fork 13
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
Support multi-channel OME-Zarr #791
Comments
Quiting @tischi from the forum post: Adding channel to ome.zarr via "ome.zarr": {
"relativePath": "images/ome-zarr/HTM.ome.zarr/A/1/0",
"channel": 0
}, I think that's a good idea; we could do the same with |
In fact, it may be super easy: Just here we would have to fetch the correct
@tibuch this could be your first PR 😉 |
Currently I can "hack" my way around this by creating a proxy-zarr-file with only one channel using symbolic links to point to the multi-channel sub-directories of the hsc-zarr-file:
But what @tischi is recommending sounds much nicer and cleaner! I will have a look (hopefully today) and see if this could become my first PR ;) |
That sounds great! I will also create a small example project for this today, so you would have something easier for development. |
|
|
One more info: The tests in the github actions are not running stable: #742 |
Interesting:
Can one can open this (channel?) separately, i.e. does it contain all the metadata to be opened as an individual OME-Zarr image?! Does that always work or is it just a coincidence that the axis order allows for such a "subsetting-hack'? |
The proxy zarr contains |
I see, so this does not work "just like this". |
Indeed. I wrote a little helper:
|
Ah, and Windows will not support this hack with the links. So better to go the way indicated by @tischi. |
Hey @tibuch, I have added an example project here: https://github.com/mobie/spatial-transcriptomics-example-project Also one note: on the python site we had a check that the name in the ome.zarr metadata ( |
Awesome! I will setup my IDE and start looking into. However, I don't think I will manage to finish today ;) |
Of course! I am available next week, so just let me know if you run into any issues :). |
Hi @constantinpape, Does this dataset work: https://github.com/mobie/spatial-transcriptomics-example-project I think some data is not available. Cheers |
Hey @tischi and @constantinpape, I made some progress and it works for single MoBIE
Maybe I forgot to update something? Or is there something else broken in the branch? Just asking because @tischi mentioned that he hopes that he fixed the branch. My changes are here: https://github.com/fmi-faim/mobie-viewer-fiji/tree/develop Would be awesome if someone of you finds time to have a look and help me carry it over the finish line Cheers! |
Hi @tibuch,
I think all data is available, at least my python validation tool says it's there (can't open it directly in MoBIE, because we need the changes you're working on for this). Do you get some error message when trying to load it?
I assume that the grid somehow needs this information beforehand... I will take a quick look later, but that's probably something @tischi has to check. |
Should I be able to see/download the data from here: https://s3.embl.de/i2k-2020/spatial-transcriptomics-example/pos42/images/ome-zarr |
Sort of, but you need to give the address for an actual file, the "fake directories" in s3 don't work, so you can't actually list anything like this (you would need to do s3 api requests for this). |
(but it shouldn't really matter in MoBIE, all the loading from s3 should work) |
Just wanted to check if I can see it, which I can. MoBIE also starts without errors, but BDV looks pretty black and empty...I guess there are still some bugs/issues. |
There might also be some issue with contrast limits ... I will try to check out your branch later and see if I can build from it and adjust contrast limits. |
If I click |
I opened #797 and mobie/mobie-io#98 to discuss the code changes. There are some changes in mobie-io that I am not so sure about and that might explain some of the issues we see here. |
@tibuch Thanks for giving this a shot ❤️ I am currently on holidays and can help from middle of next week on. |
@tibuch For debugging this you could first try opening the multi-channel OME-Zarr with the |
Sorry, I overlooked that comment. The examples that should work when opening the ome-zarr directly are here: |
I am getting an error:
throws:
This is with Any ideas? |
Is there also an |
In the tests of mobie-io I found: https://s3.embl.de/i2k-2020/ngff-example-data/v0.4/zyx.ome.zarr |
Maybe there's something wrong on the server side. I can check later. |
Hey @tischi, |
I can use this ( This (https://github.com/mobie/spatial-transcriptomics-example-project) does not show me anything. I tried playing around with the contrast limits, but it looks very empty. |
The file name suggests that this only has one channel?! |
If I open https://s3.embl.de/i2k-2020/spatial-transcriptomics-example/pos42/images/ome-zarr/MMStack_Pos42.ome.zarr (the one from the spatial-transcriptomics-example-project) with the OME-Zarr command I can see 6 channels (4 image channels, 2 segmentations). However, I am only able to get some signal for the cell-segmentation channel. Everything else looks empty to me. Could that be? |
OK! Using this code (@tibuch you could also try this) in the
It does open: The channel naming needs fixing in mobie-io: mobie/mobie-io#103 In fact in MoBIE, sources MUST have different names. |
@tibuch to me that seems to suggest that in principle it can work. |
Just fyi, the data at https://s3.embl.de/i2k-2020/spatial-transcriptomics-example/pos42/images/ome-zarr/MMStack_Pos42.ome.zarr contains data for all channels; here's what I see when opening it in napari (using the napari ome zarr plugin): But I think everything might still work correctly, and you just don't see anything due to issues with the contrast limits. |
Hi @tibuch, just a quick update for you: we merged #797 today and pushed it to mobie-beta so I can test it. I will go ahead and create another project with the data from https://s3.embl.de/i2k-2020/ngff-example-data/v0.4/czyx.ome.zarr (which can be opened via BDV->Ome Zarr) for testing. However, to check this I need the fix from #806. |
Ok, so I guess the TODO for me is?!
|
Yes, that would be great! But first, please check and release #806, then I can continue with creating an example project that should work. |
I can help if you wish. |
If you have time that would be very helpful, indeed! Please also have a look here https://github.com/mobie/mobie-io/blob/multi-channel-fix/src/test/java/spimdata/OmeZarrV4S3SpimDataTests.java |
@tischi @tibuch
Opening the project works, and when opening it from the filesystem everything works correctly: But when opening from s3, it seems to always load channel 0: Both images show the viewer when displaying |
@tibuch This works now in the mobie3 branch. If you want to develop/test things you should use the mobie3 branch from now on. Thanks! |
@tibuch: just fyi: the spec has changed a bit as well, the updated version of the python library is here: mobie/mobie-utils-python#85. I will look into merging this into main next week. And there is also a converter functions to get projects to the new spec in there if you need it. |
https://forum.image.sc/t/create-plate-view-for-mobie/67795/4
The text was updated successfully, but these errors were encountered: