Josh Moore
Josh Moore
Can you please upload the dataset to https://zenodo.org/communities/bio-formats/records?q=&l=list&p=1&s=10 so that a license and DOI can be clearly assigned? Thanks!
:+1: for the fix of xmlvalid in in #4316, but do we know if the issue will effect others?
> In [ome/ome-zarr-py#174 (files)](https://github.com/ome/ome-zarr-py/pull/174/files) it looks like this only handles local OME.xml files with `root = ET.parse(filename)` Then that would have been a bug. I assume a method like `get_json()`...
Better to just use the SyncMixing for now: https://github.com/zarr-developers/zarr-python/blob/4455726a0dbdeebb00d146b3b7a4bfa4e63374b5/src/zarr/core/sync.py#L181
Some minor pre-commit issues here, @psobolewskiPhD: ``` napari_ome_zarr/_reader.py:113:13: W503 line break before binary operator napari_ome_zarr/_reader.py:114:13: W503 line break before binary operator ```
Gotcha. (Poor thing!) > Maybe #93 would help resolve that? Any ideas how to get it passing? ``` E AttributeError: 'QtViewer' object has no attribute 'view' ```
Pushed an update from the mainline.
Still failed with the isort/black issue. No objection to moving to `ruff` but let's do that separately. I've pushed an isort config fix for the moment.
I think in this case it's that there was no change, only the error message. i.e., isort marked a change, black then reverted **but reported a change**.
Anything else from your side, @will-moore?