search

OpenMS / pyopenms-docs

pyOpenMS readthedocs documentation, additional utilities, addons, scripts, and examples.
https://pyopenms.readthedocs.io
Other
42 stars 52 forks source link

pyopenms binder integration broken #275

Closed timosachsenberg closed 2 years ago

timosachsenberg commented 2 years ago

2022-05-10 07:33:19 (11.8 MB/s) - ‘Linux-wheels.zip?status=completed’ saved [225398166]

Archive: Linux-wheels.zip inflating: pyopenms_nightly-3.0.0.dev20220507-cp37-cp37m-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
inflating: pyopenms_nightly-3.0.0.dev20220507-cp38-cp38-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
inflating: pyopenms_nightly-3.0.0.dev20220507-cp39-cp39-manylinux_2_17_x86_64.manylinux2014_x86_64.whl
WARNING: Requirement 'pyopenms_nightly--cp37-cp37m-manylinux2014_x86_64.whl' looks like a filename, but the file does not exist Processing ./pyopenms_nightly--cp37-cp37m-manylinux2014_x86_64.whl ERROR: Could not install packages due to an OSError: [Errno 2] No such file or directory: '/home/jovyan/pyopenms_nightly-*-cp37-cp37m-manylinux2014_x86_64.whl'

axelwalter commented 2 years ago

Seems like the file name of the pyopenms wheel has changed recently. Just changed that in the postBuild script.

timosachsenberg commented 2 years ago

Hmm we should think about:

  1. if we can detect automatically if the binder page breaks.
  2. if we can trigger regular rebuilds of the notebooks to detect errors I will make an issue for that
jpfeuffer commented 2 years ago

This would not have helped if the install mechanism in CI is different than in Binder. So just make sure that they are the same.

jpfeuffer commented 2 years ago

Is this resolved @axelwalter . Please close then or reference in your commit/PR.

axelwalter commented 2 years ago

Yes it's resolved.