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.

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 support for OME-NGFF ZARR inputs #45

Merged
merged 2 commits into from
Jun 8, 2023

Conversation

blowekamp
Copy link
Collaborator

No description provided.

@blowekamp blowekamp requested a review from zivy June 8, 2023 13:54
README.rst Outdated
@@ -52,7 +52,8 @@ Sample data is available and described on Zenodo:
:target: https://doi.org/10.5281/zenodo.4304786

Any image and transform file format supported by `SimpleITK's IO <https://simpleitk.readthedocs.io/en/master/IO.html>`_
can be use by sitk-ibex. The 3D `nrrd` format, and `txt` transform file extension are recommended.
can be use by sitk-ibex. The NRRD or`NGFF <https://ngff.openmicroscopy.org/latest/>`_ image format, and `txt` transform file
Copy link
Member

Choose a reason for hiding this comment

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

use -> used
format -> formats

README.rst Outdated
.. image:: https://zenodo.org/badge/DOI/10.5281/zenodo.4632320.svg
:target: https://doi.org/10.5281/zenodo.4632320

The sample Imaris files can be converted to OMG-NGFF ZARR with
Copy link
Member

Choose a reason for hiding this comment

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

OMG -> OME

README.rst Outdated

These images will be registered based on the common "Hoechst". The name of the channels are lost in this conversion from
Imaris to OME-NGFF ZARR. Some conversions produce "omera" metadata in the ZARR file which contains channel labels, which
can be used. When then channel label are unavailable, the channel index can be used such as the following commands::
Copy link
Member

Choose a reason for hiding this comment

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

then -> the
are -> is

if isinstance(channel, int):
channel_number = channel
else:
if "omero" not in zarr_group.attrs or "channels" not in zarr_group.attrs["omero"]:
Copy link
Member

Choose a reason for hiding this comment

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

Is the existence of "omero" specific to the converter used to create the OME-Zarr files? Not sure what the relationship is between OMERO (microscopy image storage/management system) and the OME-Zarr format.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The ngff meta-data is documented here: https://ngff.openmicroscopy.org/latest/index.html#omero-md. The "omero" field is optional and "transitional".

OME stands for "Open Microscopy Environment". OMERO is from the same group.

Copy link
Member

Choose a reason for hiding this comment

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

got it.

@blowekamp blowekamp merged commit 4d640ba into niaid:master Jun 8, 2023
@blowekamp blowekamp deleted the add_zarr_input branch June 8, 2023 16:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants