search

ome / omero-guide-ilastik

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

Bump version #29

Closed jburel closed 3 years ago

jburel commented 3 years ago

This PR bumps the version of ilastik and uses the meta-package. This is the only way I managed to get it to work again h5py is capped otherwise it does not work with binder The way the input array is passed has been modified in both scripts and notebooks cc @pwalczysko

github-actions[bot] commented 3 years ago

Binder :point_left: Launch a binder notebook on branch _bumpversion

joshmoore commented 3 years ago

Did h5py drop support or break an API or something?

jburel commented 3 years ago

No this is due to mamba that is used during the build process by repo2docker see https://github.com/h5py/h5py/issues/1880

pwalczysko commented 3 years ago

MyBinder, idr0062_pixel_classification_zarr.ipynb notebook

Error loading notebook

Unreadable Notebook: /home/jovyan/notebooks/idr0062_pixel_classification_zarr.ipynb NotJSONError('Notebook does not appear to be JSON: \'{\\n "cells": [\\n {\\n "cell_type": "m...')

Repeated this error also locally - repo2docker

jburel commented 3 years ago

thanks @pwalczysko. I was cleaning the notebooks and probably cleaned too much :-)

pwalczysko commented 3 years ago

the other two notebooks worked fine.

jburel commented 3 years ago

@pwalczysko it should be fixed now

pwalczysko commented 3 years ago

@pwalczysko it should be fixed now

Yes, tested on MyBinder, and the notebook works as expected now. Interestingly, I noticed a warning (see screenshot below).

Screenshot 2021-04-30 at 17 06 22

Looks good to me.

jburel commented 3 years ago

Merging. I will push a new tag and update the value in itr cc @sbesson