ome / omero-guide-ilastik

BSD 2-Clause "Simplified" License
4 stars 5 forks source link

use ome-zarr #21

Closed jburel closed 4 years ago

jburel commented 4 years ago

A similar problem to the one fixed in https://github.com/ome/ome-zarr-py/pull/41 has been raised when loading the zarr data This implies that the zarr notebooks are no longer working. This PR install ome-zarr, psutil now needs to be added since it now appears to be missing when installing from PyPI. Check that the idr0062_pixel_classification_zarr.ipynb is running

cc @jrswedlow @joshmoore @pwalczysko

pwalczysko commented 4 years ago

Built using repo2docker from this branch

idr0062_pixel_classification_zarr notebook definitely works fine, all cells. Nice new viewing features too.

LGTM

pwalczysko commented 4 years ago

Failed travis though

jburel commented 4 years ago

linkcheck passes locally. I am going to turn it of for now

jburel commented 4 years ago

This is strange https://idr.openmicroscopy.org/webclient/?show=project-801 - Exceeded 30 redirects

jburel commented 4 years ago

I use the query instead of the web link and that fixed the travis issue. Merging now